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

Migrate OpenTracing implementation to OpenTelemetry #969

Open
Blacksmoke16 opened this issue Sep 10, 2021 · 0 comments
Open

Migrate OpenTracing implementation to OpenTelemetry #969

Blacksmoke16 opened this issue Sep 10, 2021 · 0 comments

Comments

@Blacksmoke16
Copy link

The implementation from #860 was built against the OpenTracing standard. This standard is considered deprecated in favor of OpenTelemetry (OTLP) source. Rollar's implementation should be updated to adhere to the more modern OTLP spec, mainly setTag among other changes.

In the meantime, for anyone coming across this issue, OTLP's JS package provides https://github.com/open-telemetry/opentelemetry-js/tree/main/packages/opentelemetry-shim-opentracing that can be leveraged to allow using an OTLP tracer with Rollbar's existing OpenTracing implementation. This can of course be removed in favor of the raw OTLP tracer when rollbar's implementation is updated to support it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants