-
Notifications
You must be signed in to change notification settings - Fork 2.6k
[AzureCLIV2] Replaces az --version
with az version
#19039
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
Conversation
`az version` doesn't perform an update check and is faster. Fixes: microsoft#19006
Hi @jessehouwing , thanks for creating this PR, will take a look. |
az --version
with az version
az --version
with az version
Can I ask where this one is at currently? I can see the simple version check is incredibly slow, for example :
Even if we were able to opt out of running the version check every single time, that would be incredibly helpful. |
Be sure to check out
https://jessehouwing.net/recommendations-for-using-azure-cli-in-your-workflow/
…On Fri, Apr 19, 2024, 02:26 Wade Gausden ***@***.***> wrote:
Can I ask where this one is at currently? I can see the simple version
check is incredibly slow, for example :
2024-04-19T00:13:37.4098274Z [command]C:\Windows\system32\cmd.exe /D /S /C ""C:\Program Files\Microsoft SDKs\Azure\CLI2\wbin\az.cmd" --version"
2024-04-19T00:14:13.2414888Z azure-cli 2.59.0
Even if we were able to opt out of running the version check every single
time, that would be incredibly helpful.
—
Reply to this email directly, view it on GitHub
<#19039 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA724S7FKOTLJ2V3MG3VWTTY6BQCFAVCNFSM6AAAAAA5LATBUGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRVGUZTCMZSGA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
I did :) how do you think I found this! Unfortunately most of the other tips aren’t as applicable to my use case. Whereas because we use the az cli across a number of different (small) jobs, this particular issue kills us. |
If you were able to set the
|
Can we merge this PR? |
@rvairavelu @manolerazvan @v-mohithgc You are marked as approvers on this and it's been a year. When a pipeline is made up of many smaller Powershell tasks, this absolutely kills a pipeline. Adding 30 seconds to each step. This really needs to be merged. |
Hi |
Hi @jessehouwing, we have made the necessary code changes. Please find the PR below. I’ll let you know once the deployments are complete. |
az version
doesn't perform an update check and is faster.Fixes: #19006
Task name: AzureCLI
Description: Replaces the call to
az --version
withaz version
.Documentation changes required: N
Added unit tests: N
Attached related issue: #19006
Checklist: