Skip to content

{ACR} add step in "check-health" command to ensure the identity used for CMK does exist#16403

Merged
fengzhou-msft merged 2 commits intoAzure:devfrom
yugangw-msft:cmk_check
Jan 14, 2021
Merged

{ACR} add step in "check-health" command to ensure the identity used for CMK does exist#16403
fengzhou-msft merged 2 commits intoAzure:devfrom
yugangw-msft:cmk_check

Conversation

@yugangw-msft
Copy link
Copy Markdown
Contributor

Description
Add CMK configuration check to deflect support case


This checklist is used to make sure that common guidelines for a pull request are followed.

@yugangw-msft yugangw-msft changed the title [acr] add cmk check [acr] add step in "check-health" command to ensure the identity used for CMK does exist Dec 31, 2020
@yugangw-msft
Copy link
Copy Markdown
Contributor Author

CC @adewaleo @shizhMSFT @jikuma

@yonzhan
Copy link
Copy Markdown
Collaborator

yonzhan commented Dec 31, 2020

acr


CMK_MANAGED_IDENTITY_ERROR = ErrorClass(
"CMK_ERROR",
"The identity used for registry '{}' encryption doesn't exists."
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
"The identity used for registry '{}' encryption doesn't exists."
"The identity used for registry '{}' encryption doesn't exist."

@fengzhou-msft fengzhou-msft merged commit f5a88f1 into Azure:dev Jan 14, 2021
@fengzhou-msft fengzhou-msft changed the title [acr] add step in "check-health" command to ensure the identity used for CMK does exist {ACR} add step in "check-health" command to ensure the identity used for CMK does exist Jan 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants