-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Support OpenTelemetry SDK v2 #15737
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
Comments
This is currently a blocker for us, we hope to see a compatible release soon |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
Today, the Node SDK uses & supports the OpenTelemetry SDK in version 1.x.
Since v2 was released, we should look into supporting this as well.
This is, as of now, a breaking change, as anybody using
@sentry/node
with custom stuff that depends on OTEL v1 will have breakage once we update this.Because of this, we will likely cut a v10 release soon-ish that only bumps OTEL to v2.
There are a few pieces to this:
@sentry/opentelemetry
to allow both v1 and v2 (if possible - this can already happen on v9!)@sentry/node
to v2 of the OTEL SDKsa. We need to make sure that all instrumentations we use (e.g. prisma, remix, ...) are also on v2 first!
@sentry/node-core
SDK, which does not include any OTEL dependencies (but instead has peerDependency ranges defined). This SDK ships without any instrumentations out of the box. It can be used both by users that want a custom OTEL setup, as well as for scenarios where you only need manual instrumentation (e.g. CLIs or similar). See Implement Node Core SDK #15213Experimental work for this is being done here: #15518
The text was updated successfully, but these errors were encountered: