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

Fix demo sample recognition by fixing the demo tag #972

Merged
merged 1 commit into from
Jan 9, 2024
Merged

Conversation

tombruijn
Copy link
Member

@tombruijn tombruijn commented Jan 9, 2024

When demo_sample was passed as an attribute to the createOpenTelemetrySpan extension function it wasn't recognized as a tag for AppSignal. The UI only recognizes tags and not root level attributes for the demo samples box.

Use the attribute format described in our docs to transform the attribute into a tag.
https://docs.appsignal.com/opentelemetry/custom-instrumentation/attributes.html#tag

Fixes https://github.com/appsignal/appsignal-server/issues/10389

When `demo_sample` was passed as an attribute to the
`createOpenTelemetrySpan` extension function it wasn't recognized as a
tag for AppSignal. The UI only recognizes tags and not root level
attributes for the demo samples box.

Use the attribute format described in our docs to transform the
attribute into a tag.
https://docs.appsignal.com/opentelemetry/custom-instrumentation/attributes.html#tag
@tombruijn tombruijn added the bug label Jan 9, 2024
@tombruijn tombruijn requested a review from unflxw January 9, 2024 11:18
@tombruijn tombruijn self-assigned this Jan 9, 2024
@tombruijn tombruijn merged commit c49dbf3 into main Jan 9, 2024
1 check passed
@tombruijn tombruijn deleted the fix-demo-tags branch January 9, 2024 12:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants