-
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
Renamed cost allocation tags to cost tags and added schema for get list of all available tag keys #2862
Renamed cost allocation tags to cost tags and added schema for get list of all available tag keys #2862
Changes from 27 commits
7b927cf
c641158
0e276b2
7be4364
bff7aa7
011c9f0
002a6db
020911c
502d70e
b41bd12
7956270
346ce83
a0072f9
ee9bfc3
6df7a9e
7a5eab1
623ff00
6388784
a8d47d1
3ccf1b4
884aabe
3d1c7c1
def6325
ef019c5
1f2de0f
5699a84
3bd6450
7f53393
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 |
---|---|---|
@@ -0,0 +1,47 @@ | ||
{ | ||
"parameters": { | ||
"api-version": "2018-03-31", | ||
"billingAccountId": "12345" | ||
}, | ||
"responses": { | ||
"200": { | ||
"body": { | ||
"id": "providers/Microsoft.CostManagement/billingAccounts/{billingaccount-id}/providers/Microsoft.Consumption/tags/tags1", | ||
"name": "tags1", | ||
"type": "Microsoft.Consumption/tags", | ||
"eTag": "\"1d34d012214157f\"", | ||
"properties": { | ||
"tags": [ | ||
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. The property name here should be "costTags", as defined in 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. Thank you! for mentioning this. Actually it was wrong at the other place. I am corrected it there. |
||
{ | ||
"key": "Department" | ||
}, | ||
{ | ||
"key": "CostCenter" | ||
}, | ||
{ | ||
"key": "Portal" | ||
}, | ||
{ | ||
"key": "OrgName" | ||
}, | ||
{ | ||
"key": "Namespace" | ||
}, | ||
{ | ||
"key": "resourceType" | ||
}, | ||
{ | ||
"key": "Subsystem" | ||
}, | ||
{ | ||
"key": "Environment" | ||
}, | ||
{ | ||
"key": "clusterName" | ||
} | ||
] | ||
} | ||
} | ||
} | ||
} | ||
} |
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.
One more thing : Please avoid using property name as the description. Try to add a little more info :)
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.
Sure! I have made the changes. Will keep this in mind for future as well.