-
Notifications
You must be signed in to change notification settings - Fork 395
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
fix(w3c): ensure tracecontext headers take precedence over datadog #9254
Conversation
Datadog ReportBranch report: ✅ 0 Failed, 141918 Passed, 29870 Skipped, 7h 44m 31.7s Total duration (1h 51m 29.09s time saved) ⌛ Performance Regressions vs Default Branch (1)
|
BenchmarksBenchmark execution time: 2024-05-20 20:22:46 Comparing candidate commit 8f7eea9 in PR branch Found 0 performance improvements and 1 performance regressions! Performance is the same for 208 metrics, 9 unstable metrics. scenario:httppropagationextract-datadog_tracecontext_tracestate_propagated_on_trace_id_match
|
9f1b4a9
to
f9168af
Compare
f9168af
to
710804e
Compare
ca5dbaa
to
9471153
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #9254 +/- ##
===========================================
- Coverage 76.02% 10.28% -65.75%
===========================================
Files 1294 1264 -30
Lines 122816 120999 -1817
===========================================
- Hits 93370 12439 -80931
- Misses 29446 108560 +79114 ☔ View full report in Codecov by Sentry. |
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.
isn't the point of the configuration value for the inject/extract styles to denote precedence order of headers to look at?
we talked about this off GitHub. "yes", but there is an edge case where that behavior is not desired. this change is really a bug fix for a known edge case when things break. |
Even those this is a fix we are not going to backport it right now. There are other changes needed on intake to make this fix work properly. Waiting until 2.10.0 release will be acceptable for this. We can choose to backport this in the future once the intake pieces are ready. |
Description
Ensures when trace headers with the same trace_id but different span_ids are received by an application the tracecontext headers are treated as the source of truth.
This behavior will be validated by the following system-tests: DataDog/system-tests#2385
Checklist
changelog/no-changelog
is set@DataDog/apm-tees
.Reviewer Checklist