-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[AKS] Mark AAD-legacy properties deprecated #5670
Conversation
AKS |
5261bb7
to
f91a4c0
Compare
f91a4c0
to
31c9d5d
Compare
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.
CI failed due to vm command bump SDK and default API version. Will fix failed test case in some other PR.
Fix PR #5830, please rebase from main after the fix is merged. |
f0ec7d0
to
31c9d5d
Compare
31c9d5d
to
df7ef36
Compare
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.
LGTM
This PR marks the parameters
--aad-client-app-id
,--aad-server-app-id
and--aad-server-app-secret
deprecated. These parameters will be hidden from the output ofaz aks create -h
.