Uh oh!
There was an error while loading. Please reload this page.
feat(otel): Add initial package for otel - #6002
Conversation
87aa39f to
5cd1925Compare
Lms24
left a comment
There was a problem hiding this comment.
LGTM, I like the idea of having this in its own package. Just had a two remarks/questions.
Another question: Do you see any problems with the approach of the Otel package requiring sentry/tracing? I'm thinking for example about our long-term plan of splitting up or integrating the tracing package into browser/node so that users only have to install one package anymore.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
AbhiPrasad
commented
Oct 21, 2022
Discussed this offline - the answer is: yes it might have consequences, but for now not going to worry about it until v8. |
size-limit report 📦
|
ref: #6000
Add boilerplate package for OpenTelemetry Node.
I would like to make this a separate package because we are dependent on the https://github.com/open-telemetry/opentelemetry-js-api, and I don't want to include that as a dependency for everyone else.
The nice thing is that we can require the usage of
@sentry/tracing, so users only have to install@sentry/opentelemetry-node! Aka either use@sentry/tracingOR use@sentry/opentelemetry-node.As such, installation looks like so: