Skip to content

[chore] [exporter/googlemanagedprometheus] Fix invalid filter processor syntax in README example#49425

Open
vishalmore90 wants to merge 1 commit into
open-telemetry:mainfrom
vishalmore90:fix-issue-49414
Open

[chore] [exporter/googlemanagedprometheus] Fix invalid filter processor syntax in README example#49425
vishalmore90 wants to merge 1 commit into
open-telemetry:mainfrom
vishalmore90:fix-issue-49414

Conversation

@vishalmore90

@vishalmore90 vishalmore90 commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Context

Resolves #49414
The troubleshooting section of the googlemanagedprometheus exporter README contained a syntactically invalid example for configuring the filter processor. It incorrectly used a YAML list for metrics instead of a map, causing startup failures when users copy-pasted the config.

Changes

  • Updated the filter processor example to use the modern OTTL metric_conditions format.
  • Replaced the deprecated metrics: map syntax with metric_conditions: and explicitly used metric.name != ....

Verification

  • Verified that the new configuration matches the current filterprocessor documentation and OTTL standards.

Signed-off-by: vishal <httpsvishal07@gmail.com>
@vishalmore90 vishalmore90 requested review from a team, braydonk, dashpole and jsuereth as code owners July 2, 2026 03:36
@github-actions github-actions Bot added the exporter/googlemanagedprometheus Google Managed Prometheus exporter label Jul 2, 2026
@vishalmore90 vishalmore90 changed the title Fix invalid filter processor syntax in README example [chore] [exporter/googlemanagedprometheus] Fix invalid filter processor syntax in README example Jul 2, 2026
@vishalmore90 vishalmore90 marked this pull request as draft July 2, 2026 03:38
@vishalmore90 vishalmore90 marked this pull request as ready for review July 2, 2026 03:38
@vishalmore90 vishalmore90 reopened this Jul 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

exporter/googlemanagedprometheus Google Managed Prometheus exporter

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Documentation issue: bad example of processor filter

2 participants