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.

Sign up for GitHub

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

Jump to bottom

API: Use Activity as a context propagation mechanism #60

Closed
SergeyKanzhelev opened this issue Jun 3, 2019 · 0 comments · Fixed by #140
Closed

API: Use Activity as a context propagation mechanism #60

SergeyKanzhelev opened this issue Jun 3, 2019 · 0 comments · Fixed by #140
Assignees
@lmolkova
Labels
pkg:OpenTelemetry.Api Issues related to OpenTelemetry.Api NuGet package

Comments

@SergeyKanzhelev
Copy link
Member

Activity from DiagnosticsSource package is already used to propagate a context. Activity is being set and restore by the framework so OpenTelemetry can rely on it.

It will be great to switch OpenTelemetry to use Activity as a way to propagate state (current Span, SpanContext and tags) instead of simply relying on AsyncLocal
@SergeyKanzhelev SergeyKanzhelev added the pkg:OpenTelemetry.Api Issues related to OpenTelemetry.Api NuGet package label Jun 3, 2019
@SergeyKanzhelev SergeyKanzhelev added this to the API complete milestone Jun 3, 2019
@lmolkova lmolkova mentioned this issue Jul 9, 2019
Merged
Yun-Ting pushed a commit to Yun-Ting/opentelemetry-dotnet that referenced this issue Oct 13, 2022
@cijothomas
Remove Azure instrumentation (open-telemetry#60)
25ac501 
* Remove Azure instrumentation

* sanity check fails
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lmolkova lmolkova

Labels
pkg:OpenTelemetry.Api Issues related to OpenTelemetry.Api NuGet package
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Support .NET Activities lmolkova/opentelemetry-dotnet
2 participants
@lmolkova @SergeyKanzhelev