-
Notifications
You must be signed in to change notification settings - Fork 217
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
OTel Context Missing #596
Comments
Why did we re-open? |
Because it's not fixed. We still need to move the txn name away from the scope. |
Ah ok - thought we would open up a new issue for this, but this is fine for me as well 👍 |
So this is about "OTel Context Missing", and as we currently always get an empty Hub here /~https://github.com/getsentry/sentry-go/blob/master/otel/span_processor.go#L120-L123, so even with #599 being merged, it won't work just yet. |
Alright I will push up a PR then! |
Summary
Events sent to Sentry which were converted from OTel spans, should have an OTel context with attributes and other data sourced from OTel SDKs. It appears that some events do not have this.
Python event with context
Go event without context
Develop Spec: Add OpenTelemetry Context
Steps To Reproduce
Expected Behavior
OTel SDK data populated in the OTel context and appearing in the product on the transaction detail page
SDK
sentry-go
with OTel Go SDKsThe text was updated successfully, but these errors were encountered: