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

Incorrect parameter reference for azure-monitor-metrics #26600

Closed
pichuang opened this issue Jun 4, 2023 · 5 comments
Closed

Incorrect parameter reference for azure-monitor-metrics #26600

pichuang opened this issue Jun 4, 2023 · 5 comments
Assignees
Labels
AKS az aks/acs/openshift Auto-Assign Auto assign by bot bug This issue requires a change to an existing behavior in the product in order to be resolved. CXP Attention This issue is handled by CXP team.

Comments

@pichuang
Copy link

pichuang commented Jun 4, 2023

Describe the bug

There is a parameter reference error in az aks for enabling Azure Monitor metrics

Related command

Incorrect Command

az aks create --enable-azure-monitor-metrics

Correct Command

az aks create --enable-azuremonitormetrics

Errors

Line 90: az aks create -n poc1-aks -g rg-poc1-aks -l EastUS --kubernetes-version 1.26.3 --enable-cluster-autoscaler --azure-monitor-workspace-resource-id /subscriptions/xxx-xxx-xxx-xxx/resourceGroups/rg-poc1-aks/providers/microsoft.operationalinsights/workspaces/amw-poc1-aks --grafana-resource-id /subscriptions/xxx-xxx-xxx-xxx/resourceGroups/rg-poc1-aks/providers/microsoft.grafana/grafana/grafana-poc1-aks --max-count 2 --min-count 1 --tier standard --max-pods 250 --auto-upgrade-channel stable --dns-name-prefix poc1-aks --enable-managed-identity --node-count 2 --node-vm-size Standard_B4ms --network-plugin azure --network-policy azure --vnet-subnet-id /subscriptions/xxx-xxx-xxx-xxx/resourceGroups/rg-poc1-aks/providers/Microsoft.Network/virtualNetworks/vnet-poc1-aks/subnets/subnet-nodepool --pod-subnet-id /subscriptions/xxx-xxx-xxx-xxx/resourceGroups/rg-poc1-aks/providers/Microsoft.Network/virtualNetworks/vnet-poc1-aks/subnets/subnet-podpool --enable-addons ingress-appgw --enable-azure-monitor-metrics --appgw-name agic-poc1-aks --appgw-subnet-id /subscriptions/xxx-xxx-xxx-xxx/resourceGroups/rg-poc1-aks/providers/Microsoft.Network/virtualNetworks/vnet-poc1-aks/subnets/subnet-agic

unrecognized arguments: --enable-azure-monitor-metrics

Issue script & Debug output

unrecognized arguments: --enable-azure-monitor-metrics

Expected behavior

--enable-azure-monitor-metrics should work, or all docs need to fix to --enable-azuremonitormetrics

Environment Summary

$ az version
{
"azure-cli": "2.49.0",
"azure-cli-core": "2.49.0",
"azure-cli-telemetry": "1.0.8",
"extensions": {
"aks-preview": "0.5.141",
"amg": "1.2.5",
"arcdata": "1.5.0",
"azure-firewall": "0.14.7",
"bastion": "0.2.4",
"connectedk8s": "1.3.20",
"customlocation": "0.1.3",
"k8s-configuration": "1.7.0",
"k8s-extension": "1.4.1",
"ssh": "1.1.6",
"virtual-wan": "0.2.17"
}
}

Additional context

No response

@pichuang pichuang added the bug This issue requires a change to an existing behavior in the product in order to be resolved. label Jun 4, 2023
@ghost ghost added AKS az aks/acs/openshift CXP Attention This issue is handled by CXP team. Auto-Assign Auto assign by bot labels Jun 4, 2023
@yonzhan
Copy link
Collaborator

yonzhan commented Jun 4, 2023

Thank you for opening this issue, we will look into it.

@pichuang
Copy link
Author

pichuang commented Jun 4, 2023

@navba-MSFT
Copy link
Contributor

@pichuang Apologies for the late reply. Thanks for reaching out to us and reporting this issue. The above PR has been filed to fix the docs.

@navba-MSFT
Copy link
Contributor

@pichuang Please note another PR was filed above to fix the aks-preview CLI extension. Azure/azure-cli-extensions#6379

This PR was to Deprecate option names --enable-azuremonitormetrics and --disable-azuremonitormetrics, use --enable-azure-monitor-metrics and --disable-azure-monitor-metrics instead, so as to be consistent with the option names in official azure-cli.

@navba-MSFT
Copy link
Contributor

@pichuang The above PR is now merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AKS az aks/acs/openshift Auto-Assign Auto assign by bot bug This issue requires a change to an existing behavior in the product in order to be resolved. CXP Attention This issue is handled by CXP team.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants