-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
[AutoPR] containerregistry/resource-manager #6915
[AutoPR] containerregistry/resource-manager #6915
Conversation
…licy schema (#6873) * Generated from 1afeec1884ccf139e6c1bda82ab81227b3df9d2b update retention policy schema * Generated from db881033ac05775a68e36a1e098baef3e851732c update retention policy schema * Generated from c136d1fcf1a161f68944223dad032c4407154c6f remove unnecessary breaking change * Generated from 287fcbfdc1fc8b0926588963393574052e0bb9ed fix typo
(message created by the CI based on PR content) Installation instructionPackage azure-mgmt-containerregistryYou can install the package You can build a wheel to distribute for test using the following command: If you have a local clone of this repository, you can also do:
Or build a wheel file to distribute for testing:
Direct downloadYour files can be directly downloaded here: |
[ACR] Adding scopemap spec file for new SDK build For releasing ScopeMap and Token features, we need to retrieve models from the scopemap spec file. The existing SDK version doesn't incorporate changes from the scopemap spec file. Requesting a new SDK build. Please suggest changes if deemed necessary.
@@ -231,6 +251,9 @@ | |||
UpdateTriggerPayloadType, | |||
SourceRegistryLoginMode, | |||
SecretObjectType, | |||
TokenCertificateName, | |||
TokenPasswordName, | |||
Status, |
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.
seems introduction of this enum, caused renaming existing "Status" model to "Status1".
I think we should figure out how to change rest api spec to make sure it's not renamed....
@@ -307,7 +334,7 @@ | |||
'SourceTriggerUpdateParameters', | |||
'SourceUpdateParameters', | |||
'SourceUploadDefinition', | |||
'Status', | |||
'Status1', |
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.
this probably shouldn't be renamed to Status1
fix: Status to TokenStatus
Created to accumulate context: containerregistry/resource-manager