-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[Identity] Fix the resource deployment step for MI tests #31751
Conversation
API change check API changes are not detected in this pull request. |
9843f58
to
d654270
Compare
d654270
to
74a8f86
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.
Hey @KarishmaGhiya, noticing that there is some overlap with #31335. Perhaps, we should press to get that one merged and have the azure-cli change be separate?
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.
🚀
Thanks for this! I chatted with Ben, he's ok with me merging this for now. |
westus2
Reason:
We need to deploy vms in the TME tenant, the default region of westus doesnt work. Have to set the Location to westus2, as it has quota for vms/aks. Otherwise we get the error like this :