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

Document the level of support exist for OpenTracing JS #2219

Closed
carlosalberto opened this issue Dec 12, 2021 · 4 comments
Closed

Document the level of support exist for OpenTracing JS #2219

carlosalberto opened this issue Dec 12, 2021 · 4 comments
Assignees

Comments

@carlosalberto
Copy link
Contributor

OpenTracing JS never implemented ScopeManager, which supports in-process context propagation. This means that only explicit context/Span propagation is supported, even if OpenTelemetry offers this.

cc @yurishkuro @mwear @open-telemetry/javascript-approvers

@bogdandrutu
Copy link
Member

@dyladan ?

@dyladan
Copy link
Member

dyladan commented Jan 4, 2022

This appears to be true. There was an issue in opentracing to add a scope manager, but it doesn't look like it was ever implemented opentracing/opentracing-javascript#112

What is our action item here? Should we just document on the shim that there is no scope manager so we have no shim for it?

@yurishkuro
Copy link
Member

Yes, we simply need to be compatible with existing status of OT

@carlosalberto
Copy link
Contributor Author

Closing, this was all addressed by #2230 (now we only need to document this in each related SIG, e.g. Javascript SIG)

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

No branches or pull requests

5 participants