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(@opentelemetry/instrumentation-user-interaction): Add custom span attributes #1803

Conversation

adamlyka
Copy link

Which problem is this PR solving?

#1770

Short description of the changes

Added the ability to add custom attributes on a span
I have set it up to be configured with a UserInteractionCustomAttributeFunction which is in the same fashion as the other plugins for consistency.

@adamlyka adamlyka requested a review from a team November 16, 2023 10:58
Copy link

linux-foundation-easycla bot commented Nov 16, 2023

CLA Signed

The committers listed above are authorized under a signed CLA.

@adamlyka adamlyka force-pushed the feat/user-interaction-add-custom-span-attributes branch 3 times, most recently from 9ffce41 to 1d5f45e Compare November 16, 2023 11:41
@adamlyka adamlyka force-pushed the feat/user-interaction-add-custom-span-attributes branch from 1d5f45e to a1368ae Compare November 16, 2023 11:55
@pkanal
Copy link
Contributor

pkanal commented Dec 1, 2023

As I mentioned on the issue, this is actually currently possible through the shouldPreventSpanCreation so I think it would be preferable to document that better.

@pichlermarc
Copy link
Member

Closing this as per @pkanal 's comment. Please let us know in a new issue if the option documented in #1840 does not work for you.

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.

3 participants