Skip to content

Remove spanId from outbound tracestate header#1771

Open
hmstepanek wants to merge 4 commits into
mainfrom
remove-spanid-from-tracestate
Open

Remove spanId from outbound tracestate header#1771
hmstepanek wants to merge 4 commits into
mainfrom
remove-spanid-from-tracestate

Conversation

@hmstepanek

Copy link
Copy Markdown
Contributor

Overview

Remove spanId from outbound tracestate header. This is done in order to decrease header size. The spanId is already present in the traceparent header.

@github-actions

github-actions Bot commented Jun 18, 2026

Copy link
Copy Markdown

MegaLinter analysis: Success

Descriptor Linter Files Fixed Errors Warnings Elapsed time
✅ ACTION actionlint 9 0 0 1.02s
✅ MARKDOWN markdownlint 7 0 0 0 1.3s
✅ PYTHON ruff 1039 0 0 0 1.21s
✅ PYTHON ruff-format 1039 0 0 0 0.45s
✅ YAML prettier 23 0 0 0 1.7s
✅ YAML v8r 23 0 0 6.55s
✅ YAML yamllint 23 0 0 0.8s

Notices

📣 MegaLinter 9.5.0 is out! Discover the new features and security recommendations in the release announcement. (Skip this info by defining SECURITY_SUGGESTIONS: false)

See detailed reports in MegaLinter artifacts

MegaLinter is graciously provided by OX Security
Show us your support by starring ⭐ the repository

@mergify mergify Bot added the tests-failing Tests failing in CI. label Jun 18, 2026
@codecov-commenter

codecov-commenter commented Jun 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.02%. Comparing base (853d8cd) to head (295b2ad).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1771      +/-   ##
==========================================
- Coverage   82.07%   82.02%   -0.05%     
==========================================
  Files         215      215              
  Lines       26589    26589              
  Branches     4213     4213              
==========================================
- Hits        21823    21810      -13     
- Misses       3315     3326      +11     
- Partials     1451     1453       +2     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 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.

@hmstepanek hmstepanek marked this pull request as ready for review June 24, 2026 02:57
@hmstepanek hmstepanek requested a review from a team as a code owner June 24, 2026 02:57
@mergify mergify Bot removed the tests-failing Tests failing in CI. label Jun 24, 2026
@mergify

mergify Bot commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

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