Skip to content

[Azure Monitor Exporter] Export Metric attributes#22886

Merged
hectorhdzg merged 5 commits intoAzure:mainfrom
hectorhdzg:hectorhdzg/metricattr
Aug 18, 2022
Merged

[Azure Monitor Exporter] Export Metric attributes#22886
hectorhdzg merged 5 commits intoAzure:mainfrom
hectorhdzg:hectorhdzg/metricattr

Conversation

@hectorhdzg
Copy link
Copy Markdown
Member

Added metric attributes
Create an envelope per metric data point

@ghost ghost added the Monitor Monitor, Monitor Ingestion, Monitor Query label Aug 12, 2022
@hectorhdzg hectorhdzg requested a review from MSNev August 12, 2022 19:06

# PRLabel: %Monitor
/sdk/monitor/ @hectorhdzg
/sdk/monitor/ @hectorhdzg @JacksonWeber
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@jsquire @ronniegeraghty do you know how Jackson can request for write permissions?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@azure-sdk
Copy link
Copy Markdown
Collaborator

API change check

APIView has identified API level changes in this PR and created following API reviews.

azure-monitor-opentelemetry-exporter

Copy link
Copy Markdown
Member

@JacksonWeber JacksonWeber left a comment

Choose a reason for hiding this comment

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

LGTM

@hectorhdzg hectorhdzg merged commit 6c33763 into Azure:main Aug 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Monitor Monitor, Monitor Ingestion, Monitor Query

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants