-
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
Added new version in Microsoft.RecoveryServices and s360 fixups #14752
Changes from 13 commits
8eebfec
d025a8f
b86d9bb
758faaf
bee9a9d
5b203ab
f4de6a2
74b611e
b0d4034
86155e9
af97365
cf41749
1b5fcc5
192e866
87d09ff
3b05561
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -1924,6 +1924,7 @@ tileset | |
tilesets | ||
timeasc | ||
timedesc | ||
timedout | ||
timeframe | ||
timegrain | ||
timeseries | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -1375,8 +1375,8 @@ | |
"type": "object", | ||
"description": "A resource identity that is managed by the user of the service.", | ||
"properties": { | ||
"principalId": { | ||
"description": "The principal ID of the user-assigned identity.", | ||
"objectId": { | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yes i will be changing it asap and reverting it back to principal id There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Updated |
||
"description": "The object ID of the user-assigned identity.", | ||
"type": "string", | ||
"readOnly": true | ||
}, | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,19 @@ | ||
{ | ||
"parameters": { | ||
"subscriptionId": "77777777-b0c6-47a2-b37c-d8e65a629c18", | ||
"resourceGroupName": "resGroupFoo", | ||
"api-version": "2021-04-01", | ||
"location": "westus", | ||
"input": { | ||
"name": "swaggerExample", | ||
"type": "Microsoft.RecoveryServices/Vaults" | ||
} | ||
}, | ||
"responses": { | ||
"200": { | ||
"body": { | ||
"nameAvailable": true | ||
} | ||
} | ||
} | ||
} |
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.
The previous property name was correct. User assigned contract is defined here: https://armwiki.azurewebsites.net/authorization/managed_identities/MSIOnboardingUserAssigned.html#resource-definition-response
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.
Can you please reply to Chris' comment? according to our docs, we use principalId not objectId.
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.
Updated