Skip to content

Azure Monitor Exporter - Modified ExtractRoleInfo to read roleinstance and rolename once #16551

Merged
rajkumar-rangaraj merged 2 commits intoAzure:masterfrom
rajkumar-rangaraj:rajrang/resourcefinetune
Nov 4, 2020
Merged

Azure Monitor Exporter - Modified ExtractRoleInfo to read roleinstance and rolename once #16551
rajkumar-rangaraj merged 2 commits intoAzure:masterfrom
rajkumar-rangaraj:rajrang/resourcefinetune

Conversation

@rajkumar-rangaraj
Copy link
Copy Markdown
Member

  • Introduced two internal properties RoleName and RoleInstance to avoid repeatedly reading role name and instance.

Copy link
Copy Markdown

@TimothyMothra TimothyMothra left a comment

Choose a reason for hiding this comment

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

LGTM

@rajkumar-rangaraj rajkumar-rangaraj merged commit 7f62f11 into Azure:master Nov 4, 2020
annelo-msft pushed a commit to annelo-msft/azure-sdk-for-net that referenced this pull request Feb 17, 2021
…e and rolename once (Azure#16551)

* SetResource

* Rename ExtractRoleInfo to InitRoleInfo
openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-net that referenced this pull request Oct 29, 2021
[Hub Generated] Review request for Microsoft.Confluent to add version preview/2021-09-01-preview (Azure#16048)

* Adds base for updating Microsoft.Confluent from version preview/2021-03-01-preview to version 2021-09-01-preview

* Updates readme

* Updates API version in new specs and examples

* Fixing spellcheck (Azure#16551)
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.

2 participants