Skip to content

[connector/routing] Add feature gate to change default error_mode to ignore#48433

Open
singhvishalkr wants to merge 1 commit into
open-telemetry:mainfrom
singhvishalkr:fix-48418-routing-error-mode-default
Open

[connector/routing] Add feature gate to change default error_mode to ignore#48433
singhvishalkr wants to merge 1 commit into
open-telemetry:mainfrom
singhvishalkr:fix-48418-routing-error-mode-default

Conversation

@singhvishalkr
Copy link
Copy Markdown

The error_mode in the routing connector determines how errors during OTTL condition processing are handled. The current default is propagate, which can drop valid data when a condition error occurs.

This PR adds a feature gate connector.routing.defaultErrorModeIgnore that when enabled changes the default to ignore. In this mode, errors are logged for visibility but valid data is preserved and processing continues with the next condition.

The feature gate allows gradual rollout of this breaking change.

Fixes #48418

…ignore

Add connector.routing.defaultErrorModeIgnore feature gate that when enabled
changes the default error_mode from propagate to ignore. This improves
resiliency by logging OTTL condition errors and continuing processing
rather than dropping valid data.

Fixes open-telemetry#48418
@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