Skip to content

Conversation

@rosanch
Copy link
Contributor

@rosanch rosanch commented Jan 15, 2021

Description
ACR_SYNC_TOKEN_USERNAME is being deprecated from the install command group. The new value will be ACR_SYNC_TOKEN_NAME

Testing Guide
az acr connected-registry install info -n -r
az acr connected-registry install renew-credentials -n -r

History Notes
[ACR] az acr connected-registry install info: Add new key ACR_SYNC_TOKEN_NAME with the same value as ACR_SYNC_TOKEN_USERNAME. A warning that the latter will be deprecated is displayed.
[ACR] az acr connected-registry install renew-credentials: Add new key ACR_SYNC_TOKEN_NAME with the same value as ACR_SYNC_TOKEN_USERNAME. A warning that the latter will be deprecated is displayed.


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

@toddysm
Copy link
Member

toddysm commented Jan 15, 2021

Can we update the notes as follows?

History Notes

[ACR connected-registry] az acr connected-registry install info: Added new key ACR_SYNC_TOKEN_NAME with the same value as ACR_SYNC_TOKEN_USERNAME. A warning that the latter will be deprecated is displayed.
[ACR connected-registry] az acr connected-registry install renew-credentials: Added new key ACR_SYNC_TOKEN_NAME with the same value as ACR_SYNC_TOKEN_USERNAME. A warning that the latter will be deprecated is displayed.

@rosanch
Copy link
Contributor Author

rosanch commented Jan 19, 2021

@fengzhou-msft can you help review the pr please? It's ready for merge

@yonzhan
Copy link
Collaborator

yonzhan commented Jan 19, 2021

ACR

@northtyphoon
Copy link
Member

@yonzhan can you merge the pr?

@yonzhan
Copy link
Collaborator

yonzhan commented Jan 26, 2021

@fengzhou-msft to help with this PR.

@fengzhou-msft fengzhou-msft merged commit 7e2f29a into Azure:dev Jan 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants