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.

Sign up for GitHub

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

Jump to bottom

pin azure cli to 2.22.0 #201

Merged
merged 1 commit into from
May 7, 2021
Merged

pin azure cli to 2.22.0 #201

merged 1 commit into from
May 7, 2021

Conversation

glennmusa
Copy link
Contributor

Description

Pins the Azure CLI to version 2.22.0 as that was the last known version that successfully manages KeyVaults in AzureUsGovernment.

Issue reference

The issue this PR will close: #200

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles or validates correctly
  • BASH scripts have been validated using shellcheck
  • All tests pass (manual and automated)
  • The documentation is updated to cover any new or changed features
  • Markdown files have been linted using the recommended linter. (See .vscode/extensions.json.)
  • Relevant issues are linked to this PR

@glennmusa glennmusa requested a review from Phydeauxman May 7, 2021 19:56
@glennmusa glennmusa enabled auto-merge (squash) May 7, 2021 19:56
Phydeauxman
Phydeauxman approved these changes May 7, 2021
View reviewed changes
Copy link
Contributor

@Phydeauxman Phydeauxman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the quick turn Glenn...works as expected.

@glennmusa glennmusa merged commit 26a0277 into main May 7, 2021
@glennmusa glennmusa deleted the glennmusa/pinazcliverision branch May 7, 2021 20:28
Phydeauxman
Phydeauxman reviewed May 7, 2021
View reviewed changes
Copy link
Contributor

@Phydeauxman Phydeauxman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Phydeauxman Phydeauxman Phydeauxman approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Creating MLZ Configuration Resources fails in AzureUSGovernment because KeyVault resources cannot be managed
2 participants
@glennmusa @Phydeauxman