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

install/kubernetes: set the right option for expectAzureVnet #14449

Merged
merged 1 commit into from
Dec 18, 2020

Conversation

aanm
Copy link
Member

@aanm aanm commented Dec 18, 2020

Fixes: 011546b ("helm/azure: Fatal error for CNI Azure installation")
Signed-off-by: André Martins andre@cilium.io

Fix behavior of the helm option `--set nodeinit.expectAzureVnet=true`

Fixes: 011546b ("helm/azure: Fatal error for CNI Azure installation")
Signed-off-by: André Martins <andre@cilium.io>
@aanm aanm requested review from a team as code owners December 18, 2020 13:55
@aanm aanm requested review from a team, qmonnet and kaworu December 18, 2020 13:55
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Dec 18, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot added this to In progress in 1.10.0 Dec 18, 2020
@aanm aanm added needs-backport/1.9 release-note/bug This PR fixes an issue in a previous release of Cilium. labels Dec 18, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Dec 18, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.9.2 Dec 18, 2020
@aanm
Copy link
Member Author

aanm commented Dec 18, 2020

node need to run the CI for this change

@qmonnet qmonnet removed their assignment Dec 18, 2020
@joestringer joestringer merged commit ff24fe7 into cilium:master Dec 18, 2020
@christarazi christarazi removed their assignment Dec 21, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport pending to v1.9 in 1.9.2 Jan 6, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport pending to v1.9 in 1.9.2 Jan 6, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.9 to Backport done to v1.9 in 1.9.2 Jan 14, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.9 to Backport done to v1.9 in 1.9.2 Jan 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/bug This PR fixes an issue in a previous release of Cilium.
Projects
No open projects
1.9.2
Backport done to v1.9
Development

Successfully merging this pull request may close these issues.

None yet

5 participants