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

az configure | spelling error in parameter name is not throwing an error #16334

Open
dbradish-microsoft opened this issue Dec 21, 2020 · 3 comments
Assignees
Milestone

Comments

@dbradish-microsoft
Copy link
Contributor

dbradish-microsoft commented Dec 21, 2020

Issue #1: It looks like a customer can enter any value as a default. "subscription" isn't even a supported config value according to https://docs.microsoft.com/en-us/cli/azure/azure-cli-configuration?view=azure-cli-latest#cli-configuration-with-az-configure.

image

Issue #2: We allow for resource group defaults using az config set, az configure --defaults, and az config param-persist. This actually then changes the default subscription. Yet, if a customer does a "list" or "show", they only see the resource group value. This is very confusing when they have set their subscription value, then over-write themselves with a resource group default.

@ghost ghost added the needs-triage This is a new issue that needs to be triaged to the appropriate team. label Dec 21, 2020
@yonzhan yonzhan added the Configure az configure/config label Dec 21, 2020
@ghost ghost removed the needs-triage This is a new issue that needs to be triaged to the appropriate team. label Dec 21, 2020
@yonzhan
Copy link
Collaborator

yonzhan commented Dec 21, 2020

error handling in az config

@yonzhan yonzhan modified the milestones: S181, S182 Dec 21, 2020
@yonzhan yonzhan modified the milestones: S182, S183 Feb 6, 2021
@yonzhan yonzhan modified the milestones: S183 - For Ignite, S184 Feb 26, 2021
@yonzhan yonzhan modified the milestones: S184, S185 Mar 14, 2021
@yonzhan yonzhan modified the milestones: S185, S186 Apr 2, 2021
@yonzhan yonzhan modified the milestones: S186, S187 Apr 25, 2021
@jiasli jiasli modified the milestones: S187, Backlog May 13, 2021
@jiasli
Copy link
Member

jiasli commented May 13, 2021

az config set is designed as a util to modify ~/.azure/config, same as editing it manually with a text editor. Currently there is not check on the config option name.

az configure --defaults is deprecated, please use az config set defaults.group=xxx. Meanwhile, the correct command to change selected subscription is another command az account set --subscription xxx.

@dbradish-microsoft
Copy link
Contributor Author

@jiasli, The following article, Manage Azure subscriptions with the Azure CLI was published to address the config confusion, I am not confident that az configure --defaults has been removed from all docs, but I'll create a work item and take a look.

@dbradish-microsoft dbradish-microsoft changed the title spelling error in az config is not throwing an error az configure | spelling error in parameter name is not throwing an error Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants