Skip to content

Adding units to the Metrics API, adding one attribute to the DiagnosticSettings API, updating documentation#7074

Merged
erich-wang merged 4 commits intoAzure:masterfrom
gucalder:master
Aug 5, 2019
Merged

Adding units to the Metrics API, adding one attribute to the DiagnosticSettings API, updating documentation#7074
erich-wang merged 4 commits intoAzure:masterfrom
gucalder:master

Conversation

@gucalder
Copy link
Contributor

@gucalder gucalder commented Aug 1, 2019

  • Adding some metric unit names to the Metrics API 2018-01-01. The complete list now includes the following: "Count", "Bytes", "Seconds", "CountPerSecond", "BytesPerSecond", "Percent", "MilliSeconds", "ByteSeconds", "Unspecified", "Cores", "MilliCores", "NanoCores", "BitsPerSecond"

  • Adding support for a new attribute in the DiagnosticSettings object (Diagnostic Settings API 2017-05-01-preview,) i.e.: the attribute is "logAnalyticsDestinationType," which is a string that accepts the following values: Dedicated or null

  • Minor modification to the documentation

@gucalder gucalder requested a review from erich-wang as a code owner August 1, 2019 04:45
@@ -0,0 +1,12 @@
2019-08-01 03:08:36 UTC
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gucalder, this file should be under eng\mgmt\mgmtmetadata

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, moving the file.
BTW: this file was created automatically. There has to be some kind of bug or I am running the wrong command to generate the code.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gucalder , you need to first pull latest code from master, then follow the step 2 https://github.com/Azure/adx-documentation-pr/blob/master/engineering/adx_netsdk_process.md#sdk-generation-from-updated-spec
msbuild eng/mgmt.proj /t:Util /p:UtilityName=InstallPsModules

@erich-wang erich-wang self-assigned this Aug 1, 2019
@gucalder
Copy link
Contributor Author

gucalder commented Aug 1, 2019

Ping

@erich-wang
Copy link
Member

@gucalder , overall LGTM, could you provide the swagger spec PR link related to this PR? I need to confirm whether the spec is merged?

@gucalder
Copy link
Contributor Author

gucalder commented Aug 2, 2019

These are the links to the PRs for the last specs changes:
Azure/azure-rest-api-specs#6485
Azure/azure-rest-api-specs#6454

These have not been generated before since there was some kind of issue with the automatic generation of code, and when I tried manually, there were some bugs in the autorest library that made it fail.
This time I refreshed Node.js and the manual generation worked fine.

@gucalder
Copy link
Contributor Author

gucalder commented Aug 2, 2019

Ping

@erich-wang erich-wang merged commit 62f751e into Azure:master Aug 5, 2019
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.

3 participants