-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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
User identities in a new dictionary property #3183
Conversation
Moving user identities support from the property "identityId" list to a new dictionary property named "userAssignedIdentities".
Automation for azure-sdk-for-nodeA PR has been created for you based on this PR content. Once this PR will be merged, content will be added to your service PR: |
Automation for azure-libraries-for-javaA PR has been created for you based on this PR content. Once this PR will be merged, content will be added to your service PR: |
Automation for azure-sdk-for-pythonA PR has been created for you based on this PR content. Once this PR will be merged, content will be added to your service PR: |
Automation for azure-sdk-for-rubyA PR has been created for you based on this PR content. Once this PR will be merged, content will be added to your service PR: |
Automation for azure-sdk-for-goEncountered a Subprocess error: (azure-sdk-for-go)
Command: dep ensure The following issues were found in Gopkg.toml:
✗ unable to deduce repository and source type for "golang.org/x/tools": unable to read metadata: go-import metadata not found
ProjectRoot name validation failed |
@ravbhatnagar, this is to support merge-patch for user assigned identities. //cc @johanste |
Looks good |
status [as of June/13]: this api version is not yet deployed to production. |
@anuchandy This API version has now been deployed. |
@nkuchta can you please sync with @dochung4 [Dongwook Chung] on this comment #3302 (comment)? I assume those changes has to be made to 2018-06-01 as well. // cc @lmazuel |
@anuchandy, this change should be independent of that effort. I've asked @dochung4 to follow up with his team to get his changes reconciled into this API version. |
@nkuchta thank you!, merging this PR. |
Automation for azure-sdk-for-javaNothing to generate for azure-sdk-for-java |
Adding proxy resource for VM guest agent provisioning...
Add files azure-sdk-config.yaml
Moving user identities API surface from the list property "identityId" to a new dictionary property named "userAssignedIdentities".
This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information
api-version
in the path should match theapi-version
in the spec).Quality of Swagger