[Propagators] OTLP trace export carries ot.th/rv[APMAPI-2170] - #9062
[Propagators] OTLP trace export carries ot.th/rv[APMAPI-2170]#9062MilanGarnier wants to merge 8 commits into
ot.th/rv[APMAPI-2170]#9062Conversation
ot.th/rv (APMAPI-2170)
ot.th/rv (APMAPI-2170)ot.th/rv[APMAPI-2170]
Execution-Time Benchmarks Report ⏱️Execution-time results for samples comparing This PR (9062) and master. ✅ No regressions detected |
BenchmarksBenchmark execution time: 2026-08-24 17:32:34 Comparing candidate commit 8913f1b in PR branch Found 0 performance improvements and 48 performance regressions! Performance is the same for 24 metrics, 0 unstable metrics, 111 known flaky benchmarks, 15 flaky benchmarks without significant changes.
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: bf379941ce
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
2138077 to
4b6a5e8
Compare
zacharycmontoya
left a comment
There was a problem hiding this comment.
This set of changes LGTM
9c3e961 to
6dc2ca0
Compare
6dc2ca0 to
a32bde3
Compare
…rializer.cs Co-authored-by: Zach Montoya <zach.montoya@datadoghq.com>
a32bde3 to
a29c8a7
Compare
Summary of changes
Export the
ot=tracestate member in OTLP JSON and protobuf spans. (contains tracestate sampling datathandrv).Implementation details
Serialize the existing
OtelTraceStatevalue when present.Test coverage
Added focused JSON and protobuf serializer tests for present and absent values.
Other details
Stacked on #8983 which brings the support for otel tracestate sampling, and should be reviewed first.
System tests: TODO