-
Notifications
You must be signed in to change notification settings - Fork 72
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
Fix azure cli version parsing #1295
Conversation
e87cdc2
to
c22912c
Compare
@microsoft-github-policy-service agree company="Red Hat" |
7b06a91
to
f63a436
Compare
Please rebase pull request. |
I was blocked to deploy personal integrated dev env with error Tested these changes locally and its working now. |
Can you update the description to match the current proposed change? Thank you! |
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.
Thank you!
Signed-off-by: jubittajohn <jujohn@redhat.com> Co-authored-by: Jean-Francois Chevrette <jchevret@redhat.com>
f63a436
to
c284c69
Compare
@janboll I have rebased this PR and updated the description. Could you help to re-approve the changes. Thank you |
What this PR does
the above command which is used to retrieve the azure cli version also returns the keyword location in addition to the expected version, because the
az --version
also contains the keywordazure-cli
in the path of the returned Python location.Below is the output of the above command.
Output of the
az --version
command:This change parses the version correctly
version returned affter this change:
Jira:
Link to demo recording:
Special notes for your reviewer