Skip to content

(docs/internal-telemetry): add example for labels#7257

Merged
tiffany76 merged 6 commits intoopen-telemetry:mainfrom
timojohlo:example-internal-telemetry
Jul 7, 2025
Merged

(docs/internal-telemetry): add example for labels#7257
tiffany76 merged 6 commits intoopen-telemetry:mainfrom
timojohlo:example-internal-telemetry

Conversation

@timojohlo
Copy link
Contributor

This PR adds an example on how to add additional labels to internal metrics generated by the Collector. I stumbled across this problem and could not yet find an example that explained how to do it.

I also saw a major change to this document with #7035. If agreed, this commit could be included there.

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Jul 4, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

@opentelemetrybot opentelemetrybot requested review from a team and evan-bradley and removed request for a team July 4, 2025 11:47
@timojohlo timojohlo marked this pull request as ready for review July 7, 2025 06:55
@timojohlo timojohlo requested a review from a team as a code owner July 7, 2025 06:55
@opentelemetrybot opentelemetrybot requested a review from a team July 7, 2025 06:55
@tiffany76
Copy link
Member

/fix:format

@opentelemetrybot
Copy link
Contributor

fix:format applied successfully in this run.

@tiffany76
Copy link
Member

Thank you, @timojohlo! We appreciate your help making the docs better.

This PR might be faster to review and merge than #7035, which is making bigger changes. So let's have the @open-telemetry/collector-approvers take a look. Thanks!

Copy link
Member

@songy23 songy23 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@tiffany76 tiffany76 left a comment

Choose a reason for hiding this comment

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

I left a couple copyedit suggestions, but otherwise it LGTM!

@opentelemetrybot opentelemetrybot requested a review from a team July 7, 2025 16:25
@tiffany76 tiffany76 added this pull request to the merge queue Jul 7, 2025
Merged via the queue into open-telemetry:main with commit 58e6917 Jul 7, 2025
19 checks passed
@opentelemetrybot
Copy link
Contributor

Thank you for your contribution @timojohlo! 🎉 We would like to hear from you about your experience contributing to OpenTelemetry by taking a few minutes to fill out this survey.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants