Skip to content

[tailsamplingprocessor] Add gate for default OTTL error_mode ignore#48424

Draft
MukundaKatta wants to merge 1 commit into
open-telemetry:mainfrom
MukundaKatta:codex/tailsampling-default-error-mode-ignore
Draft

[tailsamplingprocessor] Add gate for default OTTL error_mode ignore#48424
MukundaKatta wants to merge 1 commit into
open-telemetry:mainfrom
MukundaKatta:codex/tailsampling-default-error-mode-ignore

Conversation

@MukundaKatta
Copy link
Copy Markdown

Description

Adds an alpha feature gate for the tail sampling processor to switch the default ottl_condition.error_mode from propagate to ignore, matching the rollout requested in #48420. Explicit error_mode values are unchanged, and omitted values keep using propagate until processor.tailsamplingprocessor.defaultErrorModeIgnore is enabled.

Also updates the config docs/schema and covers both default paths in a unit test.

Link to tracking issue

Fixes #48420

Testing

  • go test ./... from processor/tailsamplingprocessor
  • git diff --check

@MukundaKatta MukundaKatta requested review from a team and jmacd as code owners May 15, 2026 20:07
@github-actions github-actions Bot added the first-time contributor PRs made by new contributors label May 15, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Welcome, contributor! Thank you for your contribution to opentelemetry-collector-contrib.

Important reminders:

  • Read our Contributing Guidelines.
  • Sign the CLA if you haven't already.
  • First-time contributors should have at most one PR not marked as draft until their first PR is merged.
  • If your change isn't one of our priority components, reviews may take more time.
  • Give reviewers at least a few days before pinging them for feedback.
  • If you need help or struggle to move your PR forward, raise the topic on #otel-collector-dev or a Collector SIG meeting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

first-time contributor PRs made by new contributors processor/tailsampling Tail sampling processor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[processor/tailsampling] Change default error_mode to ignore

2 participants