Skip to content

[otap-dataflow] propagate headers in otlp exporter#2585

Merged
lquerel merged 19 commits intoopen-telemetry:mainfrom
c1ly:otlp-exporter-grpc-headers
Apr 8, 2026
Merged

[otap-dataflow] propagate headers in otlp exporter#2585
lquerel merged 19 commits intoopen-telemetry:mainfrom
c1ly:otlp-exporter-grpc-headers

Conversation

@c1ly
Copy link
Copy Markdown
Contributor

@c1ly c1ly commented Apr 7, 2026

Change Summary

Updated the otlp exporter to check if there is a propagation policy and if the received OtapPdata contains any headers in the context, if yes then exporter will apply the policy to the headers and create a tonic MetadataMap containing the propagated headers. Update the exporter to insert the MetadataMap in the grpc request

What issue does this PR close?

How are these changes tested?

unit tests

Are there any user-facing changes?

no

@c1ly c1ly requested a review from a team as a code owner April 7, 2026 20:10
@github-actions github-actions bot added the rust Pull requests that update Rust code label Apr 7, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 7, 2026

Codecov Report

❌ Patch coverage is 94.32990% with 11 lines in your changes missing coverage. Please review.
✅ Project coverage is 88.37%. Comparing base (beb4fa2) to head (65d9249).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff            @@
##             main    #2585    +/-   ##
========================================
  Coverage   88.37%   88.37%            
========================================
  Files         620      620            
  Lines      228395   228577   +182     
========================================
+ Hits       201850   202011   +161     
- Misses      26021    26042    +21     
  Partials      524      524            
Components Coverage Δ
otap-dataflow 90.21% <94.32%> (-0.01%) ⬇️
query_abstraction 80.61% <ø> (ø)
query_engine 90.74% <ø> (ø)
syslog_cef_receivers ∅ <ø> (∅)
otel-arrow-go 52.45% <ø> (ø)
quiver 92.27% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@lquerel
Copy link
Copy Markdown
Contributor

lquerel commented Apr 7, 2026

@albertlockett could you take a look at this one also :-)

Copy link
Copy Markdown
Contributor

@lquerel lquerel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lquerel lquerel enabled auto-merge April 8, 2026 17:45
@lquerel lquerel added this pull request to the merge queue Apr 8, 2026
@albertlockett
Copy link
Copy Markdown
Member

@albertlockett could you take a look at this one also :-)

sorry for the late reply @lquerel ! I did look at this yesterday, thought I had approved but I guess I forgot to click the button 🤦‍♂️

@lquerel
Copy link
Copy Markdown
Contributor

lquerel commented Apr 8, 2026

@albertlockett could you take a look at this one also :-)

sorry for the late reply @lquerel ! I did look at this yesterday, thought I had approved but I guess I forgot to click the button 🤦‍♂️

Thank you again for all the reviews you did yesterday!

Merged via the queue into open-telemetry:main with commit b7f1f6a Apr 8, 2026
67 of 69 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rust Pull requests that update Rust code

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Propagate headers in the OTLP Exporter

4 participants