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

Ambient distributed tracing context #5926

Closed
SergeyKanzhelev opened this issue Jun 12, 2018 · 4 comments
Closed

Ambient distributed tracing context #5926

SergeyKanzhelev opened this issue Jun 12, 2018 · 4 comments
Assignees
Labels
area-hosting Includes Hosting area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions enhancement This issue represents an ask for new feature or an enhancement to an existing one Needs: Design This issue requires design work before implementating.

Comments

@SergeyKanzhelev
Copy link

Motivation

Propagate W3C headers to dependant components and make distributed tracing context available even when monitoring is not enabled.

Propagation of headers will ensure that distributed trace will correlate services even when unmonitored ASP.NET app is deployed in-the-middle. Let's take service A that injects distributed tracing context and calls ASP.NET app B. Service B calls service C that expects distributed tracing context in incoming request.

If B is not being monitored for distributed traces - it should propagate distributed tracing context unmodified. When analysing telemetry from services A and C it will look like A calls C directly. Once distributed tracing monitoring will be enabled for B - B have to start modifying context according to W3C specification. So telemetry analysis will show A calls B which calls C.

In both scenarios any other telemetry like ILogger traces or stack dumps can be attributed with the distributed tracing context. This will allow to correlate failures of service B with the distributed trace those failures affected.

CC: @seth-capistron, @vancem, @jacpull, @lmolkova, @glennc, @davidfowl

@aspnet-hello aspnet-hello transferred this issue from aspnet/Hosting Dec 18, 2018
@aspnet-hello aspnet-hello added this to the 3.0.0 milestone Dec 18, 2018
@aspnet-hello aspnet-hello added area-hosting enhancement This issue represents an ask for new feature or an enhancement to an existing one labels Dec 18, 2018
@muratg
Copy link
Contributor

muratg commented Jan 9, 2019

cc @shirhatti @davidfowl @Tratcher

@analogrelay
Copy link
Contributor

Epic #8924

@analogrelay analogrelay added the Needs: Design This issue requires design work before implementating. label Apr 3, 2019
@analogrelay
Copy link
Contributor

@davidfowl to investigate the design here to enable Activity tracking by default.

@davidfowl
Copy link
Member

Closing this

@ghost ghost locked as resolved and limited conversation to collaborators Dec 3, 2019
@amcasey amcasey added the area-hosting Includes Hosting label Jun 1, 2023
@amcasey amcasey added area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions and removed area-runtime labels Aug 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-hosting Includes Hosting area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions enhancement This issue represents an ask for new feature or an enhancement to an existing one Needs: Design This issue requires design work before implementating.
Projects
None yet
Development

No branches or pull requests

7 participants