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

feat: add bool to control CNI config installation using Helm #2150

Conversation

vasrem
Copy link
Contributor

@vasrem vasrem commented Jan 3, 2025

Description

This feature enables more controlled installation of CNI configs in setups where more than one CNI is used. It helps prevent race conditions during initial node bootstrapping.

Todos

  • Tests
  • Documentation
  • Release note

Release Note

Add bool to control CNI config installation using Helm

This feature enables more controlled installation of CNI configs in
setups where more than one CNI is used. It helps prevent race conditions
during initial node bootstrapping.

Signed-off-by: Vasilis Remmas <[email protected]>
@vasrem vasrem changed the title feature: add bool to control CNI config installation using Helm feat: add bool to control CNI config installation using Helm Jan 3, 2025
@vasrem
Copy link
Contributor Author

vasrem commented Jan 9, 2025

@thomasferrandiz it would be great if you could have a look at this one. 🙏

@thomasferrandiz thomasferrandiz merged commit ffc4749 into flannel-io:master Jan 10, 2025
8 checks passed
@vasrem vasrem deleted the feature/allow-skipping-cni-conf-installation branch January 14, 2025 09:54
@vasrem
Copy link
Contributor Author

vasrem commented Jan 14, 2025

Thanks for merging this one @thomasferrandiz! Do you have an ETA for another release that will include this feature just to keep it in mind?

@thomasferrandiz
Copy link
Contributor

@vasrem
We don't have a set schedule for flannel releases but the next release should be early February at the latest.

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.

3 participants