Skip to content

Add OpenTelemetry baggage observability - #144

Merged
vrslev merged 3 commits into
mainfrom
feature/VA-7612-observability-baggage
Aug 25, 2026
Merged

Add OpenTelemetry baggage observability#144
vrslev merged 3 commits into
mainfrom
feature/VA-7612-observability-baggage

Conversation

@vrslev

Copy link
Copy Markdown
Contributor

Summary

  • add an opt-in span processor that maps allowlisted W3C baggage keys to server and consumer span attributes
  • add opt-in Sentry enrichment for searchable baggage tags and URL-encoded links
  • preserve existing trace propagation and custom before-send callbacks
  • document the generic settings and their output contract

Verification

  • 56 focused OpenTelemetry and Sentry tests passed on Python 3.13
  • Ruff format check passed
  • Ruff lint check passed
  • independent review: No material findings
  • full local mypy was blocked by the pre-existing environment missing current optional FastMCP dependencies and an unrelated Litestar hook signature mismatch; the pull-request matrix is authoritative

Compatibility

All new settings default to empty collections, so existing consumers are unchanged until they opt in.

Delivery dependency

Downstream consumer updates remain pending until this package change is approved and released.

@vrslevvrslev self-assigned this Aug 25, 2026
@codecov

codecovBot commented Aug 25, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

FlagCoverage Δ
unittests98.77% <100.00%> (+0.61%)⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing linesCoverage Δ
...obootstrap/instruments/opentelemetry_instrument.py98.48% <100.00%> (+0.19%)⬆️
microbootstrap/instruments/sentry_instrument.py97.70% <100.00%> (+10.37%)⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@vrslevvrslev changed the title VA-7612 Add OpenTelemetry baggage observabilityAdd OpenTelemetry baggage observabilityAug 25, 2026
@vrslev
vrslev marked this pull request as ready for review August 25, 2026 10:21
@vrslev
vrslev enabled auto-merge (squash) August 25, 2026 10:23
@vrslev
vrslev disabled auto-merge August 25, 2026 10:26
@vrslev
vrslev merged commit 02405e8 into mainAug 25, 2026
18 checks passed
@vrslev
vrslev deleted the feature/VA-7612-observability-baggage branch August 25, 2026 10:26
Sign up for freeto 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.

1 participant

@vrslev