Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

E2E for Cilium CNI without Kubeproxy #310

Merged
merged 1 commit into from
Oct 25, 2023

Conversation

adiantum
Copy link
Contributor

@adiantum adiantum commented Aug 28, 2023

For tech preview we are planning to using cilium without kube-proxy.
To get some validation these changes add E2E tests for Cilium CNI without kube-proxy to make it able to validate the configuration.
Changes:

  • add manifest scripts for Cilium CNI with kube-proxy replacement
  • add CAPX manifests with skip phase kube-proxy install
  • change makefile: add target for Cilium CNI with kube-proxy replacement E2E tests

Tests:
E2E tested on selected K8S versions. All tests pass except HostPort related tests which is well-known issue for Cilium CNI.

NONE

@nutanix-cn-prow-bot
Copy link

@adiantum: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-ncn-1-calico-k8s-v1.26.1 943946c link false /test e2e-ncn-1-calico-k8s-v1.26.1

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@adiantum adiantum changed the title [WIP] E2E for Cilium CNI without Kubeproxy E2E for Cilium CNI without Kubeproxy Aug 28, 2023
@adiantum
Copy link
Contributor Author

/assign @thunderboltsid @tuxtof

@adiantum adiantum force-pushed the cil-no-proxy branch 2 times, most recently from b867fb9 to 7d17219 Compare September 21, 2023 16:14
@codecov-commenter
Copy link

Codecov Report

Merging #310 (29cb985) into main (32d9ba5) will not change coverage.
The diff coverage is n/a.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@          Coverage Diff          @@
##            main    #310   +/-   ##
=====================================
  Coverage   2.61%   2.61%           
=====================================
  Files          4       4           
  Lines        995     995           
=====================================
  Hits          26      26           
  Misses       969     969           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Contributor

@tuxtof tuxtof left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

templates/* need to be removed

@adiantum
Copy link
Contributor Author

templates/* need to be removed

Fixed

 - add manifest scripts
 - change makefile
Copy link
Contributor

@tuxtof tuxtof left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tuxtof tuxtof merged commit b4e4a20 into nutanix-cloud-native:main Oct 25, 2023
3 of 4 checks passed
@adiantum adiantum deleted the cil-no-proxy branch October 25, 2023 17:08
thunderboltsid pushed a commit that referenced this pull request Apr 30, 2024
- add manifest scripts
 - change makefile
tuxtof added a commit that referenced this pull request May 2, 2024
…3.0 (#424)

* Fix credentials delete logic (#301)

- check if no delete attempt occured
 - check if no secret present in cluster
 - add integration test case in e2e suite
 - change Makefile test-e2e target to make able to run selected
   tests using GINKGO_FOCUS env variable
 - code grooming

* Bump docker/metadata-action from 4 to 5 (#317)

Bumps [docker/metadata-action](https://github.com/docker/metadata-action) from 4 to 5.
- [Release notes](https://github.com/docker/metadata-action/releases)
- [Upgrade guide](https://github.com/docker/metadata-action/blob/master/UPGRADE.md)
- [Commits](docker/metadata-action@v4...v5)

---
updated-dependencies:
- dependency-name: docker/metadata-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump docker/login-action from 2 to 3 (#316)

Bumps [docker/login-action](https://github.com/docker/login-action) from 2 to 3.
- [Release notes](https://github.com/docker/login-action/releases)
- [Commits](docker/login-action@v2...v3)

---
updated-dependencies:
- dependency-name: docker/login-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump aquasecurity/trivy-action from 0.11.2 to 0.12.0 (#313)

Bumps [aquasecurity/trivy-action](https://github.com/aquasecurity/trivy-action) from 0.11.2 to 0.12.0.
- [Release notes](https://github.com/aquasecurity/trivy-action/releases)
- [Commits](aquasecurity/trivy-action@0.11.2...0.12.0)

---
updated-dependencies:
- dependency-name: aquasecurity/trivy-action
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump actions/checkout from 3 to 4 (#314)

Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to 4.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v3...v4)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* fix(manager): remove CPU limit and increase memory limit (#319)

* E2E for Cilium CNI without Kubeproxy (#310)

- add manifest scripts
 - change makefile

* Bump aquasecurity/trivy-action from 0.12.0 to 0.13.0 (#324)

Bumps [aquasecurity/trivy-action](https://github.com/aquasecurity/trivy-action) from 0.12.0 to 0.13.0.
- [Release notes](https://github.com/aquasecurity/trivy-action/releases)
- [Commits](aquasecurity/trivy-action@0.12.0...0.13.0)

---
updated-dependencies:
- dependency-name: aquasecurity/trivy-action
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump aquasecurity/trivy-action from 0.13.0 to 0.13.1 (#326)

Bumps [aquasecurity/trivy-action](https://github.com/aquasecurity/trivy-action) from 0.13.0 to 0.13.1.
- [Release notes](https://github.com/aquasecurity/trivy-action/releases)
- [Commits](aquasecurity/trivy-action@0.13.0...0.13.1)

---
updated-dependencies:
- dependency-name: aquasecurity/trivy-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump aquasecurity/trivy-action from 0.13.1 to 0.14.0 (#327)

Bumps [aquasecurity/trivy-action](https://github.com/aquasecurity/trivy-action) from 0.13.1 to 0.14.0.
- [Release notes](https://github.com/aquasecurity/trivy-action/releases)
- [Commits](aquasecurity/trivy-action@0.13.1...0.14.0)

---
updated-dependencies:
- dependency-name: aquasecurity/trivy-action
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* fix CVE-2023-39325 (#329)

* update kube-vip to v0.6.3 (#320)

* Add codecov.yml to ensure adequate patch coverage (#328)

This sets the minimum patch coverage required for changes to repo to 75%.

* Add last git commit hash to build flags (#332)

This ensures we have the git hash of the changes in our CI logs
so we can correlate the code changes being executed.

* Cleanup for passing the linter (#336)

* Delete unused code from pkg/context

* Run gofumpt on the code

* Add lint fixes to test package

* Skip linting the conversion functions

The conversion functions are infinitely recursive and cause
linting errors.

* Remove unnecessary statement in lint target

* Revert file-scope licenses

* Bump aquasecurity/trivy-action from 0.14.0 to 0.15.0 (#337)

Bumps [aquasecurity/trivy-action](https://github.com/aquasecurity/trivy-action) from 0.14.0 to 0.15.0.
- [Release notes](https://github.com/aquasecurity/trivy-action/releases)
- [Commits](aquasecurity/trivy-action@0.14.0...0.15.0)

---
updated-dependencies:
- dependency-name: aquasecurity/trivy-action
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Update controller-gen to v0.14.0

Peg go version in build job to go1.21
Update manifests
Run `make manifests`

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Ilya Alekseyev <ilya.alekseyev@nutanix.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Frederic M <43849398+fad3t@users.noreply.github.com>
Co-authored-by: Christophe Jauffret <christophe.jauffret@nutanix.com>
tuxtof pushed a commit that referenced this pull request May 2, 2024
- add manifest scripts
 - change makefile
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants