-
-
Notifications
You must be signed in to change notification settings - Fork 161
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
Propagate tracing span attributes in sentry events #617
Comments
I don’t think this is currently possible. Other members of the team are prototyping ways to rewamp the performance monitoring / tracing APIs to make such things easier in the future, but it is still very early in that process. |
Cool, looking forward to seeing what happens there. Let me know if you want any feedback/testers, we'd be open to trying stuff out. |
Hello. I just made the implementation of this feature. Would be glad if someone look at it and maybe suggest some improvements. |
Unfortuantely, this feature appears to be tied to a Could we move the sampling of the span to the |
Reverts #7379. Unfortunately, this doesn't actually work because those fields are only recorded as part of spans that get sampled, see getsentry/sentry-rust#617 (comment). If we were to start recording all spans, we'd have a massive overhead and send lots of spans to Sentry.
Reverts #7379. Unfortunately, this doesn't actually work because those fields are only recorded as part of spans that get sampled, see getsentry/sentry-rust#617 (comment). If we were to start recording all spans, we'd have a massive overhead and send lots of spans to Sentry.
The way to achieve this is to use a |
Hey,
Thanks for providing Rust support. We're happily using
sentry
andsentry-tracing
crates. Mostly all working well.We expose a ton of helpful debugging information through span attributes. As far as I can tell, these are captured for performance monitoring, but aren't provided as context when capturing events (via
tracing::error!
for example).Is there any way that any span data captured could also be included in events emitted by
sentry-tracing
?The text was updated successfully, but these errors were encountered: