Add TCP Options Context Propagation Support - #914
Merged
Conversation
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #914 +/- ##
==========================================
+ Coverage 55.08% 55.26% +0.17%
==========================================
Files 253 253
Lines 21756 21769 +13
==========================================
+ Hits 11985 12031 +46
+ Misses 8946 8911 -35
- Partials 825 827 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
rafaelroquetto
force-pushed
the
tcp_opts
branch
4 times, most recently
from
November 22, 2025 00:47
c48bb1b to
4613686
Compare
rafaelroquetto
commented
Nov 22, 2025
rafaelroquetto
changed the title
rafaelroquetto
force-pushed
the
tcp_opts
branch
from
November 22, 2025 02:51
1dc31c5 to
f48c390
Compare
rafaelroquetto
marked this pull request as ready for review
November 22, 2025 03:08
mmat11
reviewed
Nov 24, 2025
mmat11
left a comment
mmat11
left a comment
Contributor
There was a problem hiding this comment.
left some comments, overall lgtm! nice work on the context propagation docs 👏🏻
Merged
marctc
pushed a commit
to grafana/opentelemetry-ebpf-instrumentation
that referenced
this pull request
Dec 9, 2025
* Introduce TCP context propagation * Update test matrix * Test format * Fix test * Add devdoc * Remove deprecated option * Fix docs linter * Remove magic number * Fix tests * Remove stray comment * Reuse encode_hex * Remove unused aliases * Update docs * Fix formatting
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add TCP Options Context Propagation Support
This PR adds support for injecting distributed tracing context via TCP options (kind 25), providing a transport-layer fallback for context propagation when HTTP headers are unavailable (e.g., SSL/TLS traffic or non-HTTP protocols):
tpinjectornow injects TCP options in addition to HTTPContextPropagationfield supporting combinations likeheaders,tcp,ip, orallviaOTEL_EBPF_BPF_CONTEXT_PROPAGATION(backwards compatible)devdocs/context-propagation.mdexplaining architecture, execution order, and coordination mechanismsBreaking Changes
Removed deprecated
ContextPropagationEnabledboolean field - users must migrate tocontext_propagation: "all"or specific combinations like"headers,tcp"