Skip to content

manifests: use CNI 0.4.0#300

Merged
squat merged 1 commit intomainfrom
use_cni_0.4.0
Apr 21, 2022
Merged

manifests: use CNI 0.4.0#300
squat merged 1 commit intomainfrom
use_cni_0.4.0

Conversation

@squat
Copy link
Copy Markdown
Owner

@squat squat commented Apr 20, 2022

As mentioned in the Kilo Slack [0], Kubernetes supports CNI 0.4.0 and
does not yet support 1.0.0. Correspondingly, this commit downgrades the
declared CNI version in the configuration to 0.4.0 and crucially updates
the configuration used in the e2e tests to exercise this new CNI
version.

[0] https://kubernetes.slack.com/archives/C022EB4R7TK/p1650455432970199?thread_ts=1650368553.132859&cid=C022EB4R7TK

Signed-off-by: Lucas Servén Marín lserven@gmail.com

As mentioned in the Kilo Slack [0], Kubernetes supports CNI 0.4.0 and
does not yet support 1.0.0. Correspondingly, this commit downgrades the
declared CNI version in the configuration to 0.4.0 and crucially updates
the configuration used in the e2e tests to exercise this new CNI
version.

[0] https://kubernetes.slack.com/archives/C022EB4R7TK/p1650455432970199?thread_ts=1650368553.132859&cid=C022EB4R7TK

Signed-off-by: Lucas Servén Marín <lserven@gmail.com>
@squat squat requested a review from leonnicolas April 20, 2022 13:00
@squat
Copy link
Copy Markdown
Owner Author

squat commented Apr 20, 2022

cc @READ10

@READ10
Copy link
Copy Markdown

READ10 commented Apr 20, 2022

Deployed to a test cluster and pods come up ok, so looks good to me.

@squat squat merged commit 8afe1be into main Apr 21, 2022
@squat squat deleted the use_cni_0.4.0 branch April 21, 2022 06:26
@squat
Copy link
Copy Markdown
Owner Author

squat commented Apr 21, 2022

Merged with some urgency to fix the main branch

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.

2 participants