Skip to content

CORE-946 + CORE-945: headsampling tracestate + dry run#193

Merged
natyharary merged 4 commits intomainfrom
core-946-headsampling-tracestate
May 3, 2026
Merged

CORE-946 + CORE-945: headsampling tracestate + dry run#193
natyharary merged 4 commits intomainfrom
core-946-headsampling-tracestate

Conversation

@natyharary
Copy link
Copy Markdown
Contributor

@natyharary natyharary commented May 3, 2026

What this PR does / why we need it:

  1. Adds the tracestate attribute (w3c-coded), mirroring the nodejs instrumentation.

Tracestate spec:
https://github.com/odigos-io/odigos/tree/main/collector/processors/odigostracestateprocessor

  1. This PR also adds the support for the dry run config, which not only makes traces be recorded irregardless of the sampling rule (hence the "dry" run) but also adds an indication on the tracestate.
  2. Bonus: adds ruff (linter) + ty (typing linter) support with pyproject, CI and git pre-commit hooks.

Changelog entry: Does this PR introduce a user-facing bug fix, feature, dependency update, or breaking change??

Tracestate + dry run support

@natyharary natyharary self-assigned this May 3, 2026
@natyharary natyharary force-pushed the core-946-headsampling-tracestate branch from 6df6ccb to 6e24e84 Compare May 3, 2026 12:12
@natyharary natyharary force-pushed the core-946-headsampling-tracestate branch from 6e24e84 to 0d75439 Compare May 3, 2026 12:15
@natyharary natyharary requested a review from tamirdavid1 May 3, 2026 12:18
@natyharary natyharary marked this pull request as ready for review May 3, 2026 12:19
@blumamir blumamir dismissed a stale review May 3, 2026 12:46

not new

@natyharary natyharary merged commit d581b03 into main May 3, 2026
23 of 24 checks passed
@natyharary natyharary deleted the core-946-headsampling-tracestate branch May 3, 2026 19:24
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