-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
[Core] BREAKING CHANGE: az extension add/update
and az upgrade
: Set default --allow-preview
value from True
to be False
#28913
base: dev
Are you sure you want to change the base?
Conversation
️✔️AzureCLI-FullTest
|
Hi @AllyW, |
️✔️AzureCLI-BreakingChangeTest
|
Core |
--allow-preview
value from True
to be False
for az extension add/update
and az upgrade
--allow-preview
value from True
to be False
for az extension add/update
and az upgrade
@@ -34,7 +34,7 @@ for ext in $output; do | |||
fi | |||
|
|||
echo "Verifying extension:" $ext | |||
az extension add -n $ext | |||
az extension add -n $ext --allow-preview True |
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.
How many extensions do not follow the latest version number convention?
The test script should use the default config, as most users do.
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.
The latest version schema and cli core code is compatible with the version number of all extension modules.
There are 94 extension modules that only has preview releases (which would be failed when customers use new cli to install extension, and if it fails, the message like following will be prompted to users.)
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.
This's a huge number.
Lots of users will be affected.
The PR title does not follow the convention. We need to put command name at the beginning of the title. |
--allow-preview
value from True
to be False
for az extension add/update
and az upgrade
az extension add/update
and az upgrade
: Set default --allow-preview
value from True
to be False
c039d60
to
d29fc7d
Compare
Related command
Description
Testing Guide
History Notes
[Extension] BREAKING CHANGE:
az extension add/update
: Set default--allow-preview
value fromTrue
to beFalse
[Upgrade] BREAKING CHANGE:
az upgrade
: Set default--allow-preview
value fromTrue
to beFalse
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.