-
Notifications
You must be signed in to change notification settings - Fork 374
fix: update deprecated command to new command #2686
Conversation
Fix az deployment group in create.ps1
|
Thanks for the PR! This command is not in the az cli in the version I have installed (
@boydc2014 @luhan2017 do you have any thoughts? |
|
@a-b-r-o-w-n thanks for your comment and posting the issue. |
I will vote for 2, supporting one approach which is the latest looks good enough to me. And thanks @nt-7 , will you be able to add a version guard at top to prompt user to update az-cli, if the version is below certain version? |
| # Deploy Azure services | ||
| Write-Host "> Validating Azure deployment ..." | ||
| $validation = az group deployment validate ` | ||
| $validation = az deployment group validate ` |
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.
Can you also add a version guard on top of the file? maybe pick the version az-cli switch to this new syntax? @zidaneymar any suggestions on the version here?
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.
@boydc2014 thanks for your reply!
As follows, it seems that az CLI version (az--version) cannot be parsed programmatically...
Azure/azure-cli#11425
Meanwhile, we can get the version programmatically with the following new features, az version command. However, it is still a preview feature.
Azure/azure-cli#11680
Do you have any good suggestions about this?
|
@nt-7 we have deprecated the powershell script in favor of a cross-platform friendly node script. Thanks for the PR and for bringing this to our attention. |
Description
"az group deployment" command is deprecated in the most recent version of the azure cli.
This will replace it with the new "az deployment group" command.
So, it should fix from deprecated "az group deployment" to new "az deployment group" in create.ps1.
Results (Screenshots)
Reference
fix: use deployment group instead of group deployment
microsoft/accessibility-insights-service#662
refs #2687