Skip to content

Fix deferred export by properly formatting OTLP headers for curl (#2820)#2832

Merged
plengauer merged 1 commit into
release/v5.42from
backport/6afeca6776ba43c36a2df86151bbe49128daee14
Jan 28, 2026
Merged

Fix deferred export by properly formatting OTLP headers for curl (#2820)#2832
plengauer merged 1 commit into
release/v5.42from
backport/6afeca6776ba43c36a2df86151bbe49128daee14

Conversation

@plengauer
Copy link
Copy Markdown
Owner

Automated backport of commit 6afeca6

Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
@plengauer plengauer requested a review from moflwi as a code owner January 22, 2026 17:40
Copilot AI review requested due to automatic review settings January 22, 2026 17:40
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes deferred export functionality by correcting how OTLP headers are formatted before being passed to curl. The issue was that headers were being written to a file in their raw environment variable format (comma-separated key=value pairs) rather than in the format curl expects (one header per line with "key: value" format).

Changes:

  • Removed the initial blanket write of $OTEL_EXPORTER_OTLP_HEADERS to the headers file
  • Updated header processing for logs, metrics, and traces to properly combine and format headers using tr, grep, and sed to convert from comma-separated key=value format to line-separated key: value format

Comment thread actions/instrument/workflow/main.sh
@plengauer plengauer enabled auto-merge (squash) January 22, 2026 17:41
@plengauer plengauer merged commit 563b4e1 into release/v5.42 Jan 28, 2026
1105 of 1107 checks passed
@plengauer plengauer deleted the backport/6afeca6776ba43c36a2df86151bbe49128daee14 branch January 28, 2026 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants