Skip to content

[connector/signaltometrics] Add feature gate to change default error_mode to ignore#48434

Draft
singhvishalkr wants to merge 1 commit into
open-telemetry:mainfrom
singhvishalkr:fix-48419-signaltometrics-error-mode-default
Draft

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

Conversation

@singhvishalkr
Copy link
Copy Markdown

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

This PR adds a feature gate connector.signaltometrics.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.

The feature gate allows gradual rollout of this breaking change.

Fixes #48419

…mode to ignore

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

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

@github-actions github-actions Bot requested a review from lahsivjar May 16, 2026 08:29
@singhvishalkr singhvishalkr marked this pull request as draft May 17, 2026 12:56
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/signaltometrics] Change default error_mode to ignore

2 participants