-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
operator: remove deprecated Azure cloud name flag #17765
Conversation
ci-aks |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good thank you!
Two minor nits below, but they're non-blocking.
Commit b29389d ("Auto-detect Azure cloud name via IMDS") marked the flag as deprecated in 1.10 and to be removed in 1.11. Signed-off-by: Tobias Klauser <tobias@cilium.io>
4a4cb43
to
1b88b2e
Compare
ci-aks |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for this @tklauser !
All reviews are in and the AKS test passed, no need to run full CI. Marking as ready to merge. |
Commit b29389d ("Auto-detect Azure cloud name via IMDS") marked the
flag as deprecated in 1.10 and to be removed in 1.11.