Skip to content
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

feat(otelglobal): add support for Span SetName #590

Merged

Conversation

asecondo
Copy link
Contributor

@asecondo asecondo commented Jan 10, 2024

Adds support for setting a span name via manual instrumentation, as described in #542.

NOTE: this is my first PR in this repo. Open to any and all suggestions 🙂

Copy link

linux-foundation-easycla bot commented Jan 10, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

@asecondo asecondo force-pushed the 542-manual-instrumentation-set-name branch 4 times, most recently from e95e8a2 to ec328e2 Compare January 10, 2024 04:56
@asecondo asecondo marked this pull request as ready for review January 10, 2024 04:57
@asecondo asecondo requested a review from a team January 10, 2024 04:57
@asecondo asecondo changed the title [WIP]: feat(otelglobal): add support for Span SetName feat(otelglobal): add support for Span SetName Jan 10, 2024
@asecondo asecondo force-pushed the 542-manual-instrumentation-set-name branch 2 times, most recently from b8e3135 to 539c2a5 Compare January 10, 2024 05:34
@asecondo asecondo changed the title feat(otelglobal): add support for Span SetName [WIP]: feat(otelglobal): add support for Span SetName Jan 10, 2024
@asecondo asecondo force-pushed the 542-manual-instrumentation-set-name branch from daa9268 to 0860a84 Compare January 11, 2024 01:33
@asecondo asecondo force-pushed the 542-manual-instrumentation-set-name branch from 0860a84 to 3c8c55f Compare January 11, 2024 01:34
@asecondo asecondo changed the title [WIP]: feat(otelglobal): add support for Span SetName feat(otelglobal): add support for Span SetName Jan 11, 2024
Copy link
Contributor

@RonFed RonFed left a comment

Choose a reason for hiding this comment

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

Looks great

Copy link
Member

@MikeGoldsmith MikeGoldsmith left a comment

Choose a reason for hiding this comment

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

Looks good - just one minor suggestion for function name 👍🏻

@MrAlias MrAlias merged commit 9748c86 into open-telemetry:main Jan 14, 2024
16 checks passed
@MrAlias MrAlias added this to the v0.11.0-alpha milestone Mar 28, 2024
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.

4 participants