Skip to content

[routingconnector] Add gate for default error_mode ignore#48422

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

[routingconnector] Add gate for default error_mode ignore#48422
MukundaKatta wants to merge 1 commit into
open-telemetry:mainfrom
MukundaKatta:codex/routing-default-error-mode-ignore

Conversation

@MukundaKatta
Copy link
Copy Markdown

Description

Adds an alpha feature gate for the routing connector to switch the default error_mode from propagate to ignore, matching the rollout requested in #48418. The existing default remains propagate until connector.routing.defaultErrorModeIgnore is enabled.

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

Link to tracking issue

Fixes #48418

Testing

  • go test ./... from connector/routingconnector
  • git diff --check

@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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[connector/routing] Change default error_mode to ignore

2 participants