-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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] az aks create/update
: Update region map for default region creation with new Azure Monitor Workspace regions
#28236
Conversation
… Workspace regions
️✔️AzureCLI-FullTest
|
Hi @bragi92, |
️✔️AzureCLI-BreakingChangeTest
|
AKS |
@zhoxing-ms , @FumingZhang please have a look at this PR whenever you have a chance. |
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
btw, do you have a chance to fix the issue related to a CRI before? I think the following line should be deleted.
azure-cli/src/azure-cli/azure/cli/command_modules/acs/managed_cluster_decorator.py
Line 7424 in 248054e
self.context.raw_param.get("disable_addons") is not None |
@FumingZhang I created another PR to fix the CRI issue here : https://github.com/Azure/azure-cli/pull/28414/files. Can you please approve and merge it in if it looks good? |
@FumingZhang or @zhoxing-ms can you help me with merging this PR in? I do not have write permissions to the repo. |
Related command
az aks create -n kaveeshcli22 -g kaveeshcli --location westeurope --enable-azure-monitor-metrics --grafana-resource-id "{full_id}"
az aks update -n kaveeshcli22 -g kaveeshcli --enable-azure-monitor-metrics --enable-windows-recording-rules
Description
Azure Monitor Workspace now supports new regions so I am updating the default mapping for Azure Monitor Workspace to support these new regions in case the customer does not provide one while onboarding to the Managed Prometheus addon.
Testing Guide
Scenario (region map update):
az cloud set --name AzureCloud
az aks create -n kaveeshcli -g kaveeshcli --location canadaeast--enable-azure-monitor-metrics
This checklist is used to make sure that common guidelines for a pull request are followed.
The PR title and description has followed the guideline in Submitting Pull Requests.
I adhere to the Command Guidelines.
I adhere to the Error Handling Guidelines.