-
Notifications
You must be signed in to change notification settings - Fork 773
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
[Http] Fix propagation issues #3828
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #3828 +/- ##
==========================================
- Coverage 87.42% 87.13% -0.30%
==========================================
Files 280 280
Lines 10767 10765 -2
==========================================
- Hits 9413 9380 -33
- Misses 1354 1385 +31
|
src/OpenTelemetry.Instrumentation.Http/Implementation/HttpHandlerDiagnosticListener.cs
Outdated
Show resolved
Hide resolved
src/OpenTelemetry.Instrumentation.Http/Implementation/HttpHandlerDiagnosticListener.cs
Outdated
Show resolved
Hide resolved
test/OpenTelemetry.Instrumentation.Http.Tests/HttpClientTests.Basic.cs
Outdated
Show resolved
Hide resolved
test/OpenTelemetry.Instrumentation.Http.Tests/HttpClientTests.Basic.cs
Outdated
Show resolved
Hide resolved
test/OpenTelemetry.Instrumentation.Http.Tests/HttpClientTests.Basic.cs
Outdated
Show resolved
Hide resolved
} | ||
else | ||
{ | ||
Assert.Single(exportedItems); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For test case sample == true && createParentActivity == true
shouldn't we expect nothing exported?
That is, the parent activity that is created is not sampled, right? So per the default ParentBasedSampler
LocalParentNotSampled
sampler defaults to AlwaysOff
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That is, the parent activity that is created is not sampled, right?
It is fake sampled 🤣
parent = new Activity("parent")
.SetIdFormat(ActivityIdFormat.W3C)
.Start();
parent.TraceStateString = "k1=v1,k2=v2";
parent.ActivityTraceFlags = ActivityTraceFlags.Recorded; // <- Here
Changes
I was working on tests for #3793 and it seems we have a propagation bug:
What this change does is...
TODOs
CHANGELOG.md
updated for non-trivial changes