-
Notifications
You must be signed in to change notification settings - Fork 3.1k
[exporter/datadogexporter] Add source tag to metrics client #31528
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
[exporter/datadogexporter] Add source tag to metrics client #31528
Conversation
songy23
left a comment
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 test the source tag in integration_test as well?
opentelemetry-collector-contrib/exporter/datadogexporter/integrationtest/integration_test.go
Line 92 in 2091a35
| assert.Equal(t, "datadogexporter-otelcol-tests", spl.AgentVersion) |
Co-authored-by: Pablo Baeyens <[email protected]>
Chatted with @songy23 and decided this isn't necessary because these metrics are Prometheus metrics and not APM stats. |
…emetry#31528) **Description:** <Describe what has changed.> <!--Ex. Fixing a bug - Describe the bug and how this fixes the issue. Ex. Adding a feature - Explain what this achieves.--> Add `source:{datadogexporter|datadogconnector}` tag to trace agent telemetry metrics. **Link to tracking Issue:** <Issue number if applicable> **Testing:** <Describe what testing was performed and which tests were added.> **Documentation:** <Describe the documentation added.> --------- Co-authored-by: Pablo Baeyens <[email protected]>
Description:
Add
source:{datadogexporter|datadogconnector}tag to trace agent telemetry metrics.Link to tracking Issue:
Testing:
Documentation: