Skip to content

Conversation

@fr0stylo
Copy link
Contributor

Description

Add feature gate for extra scape metrics in prometheus receiver along side current configuration. Change should not change any current behavior, just adds new additional input stream, which needs to be migrated later

Link to tracking issue

Fixes #44181

Testing

Updated tests to support mainly new input - feature gate, due to the fact of deprecation of configuration setup

@fr0stylo fr0stylo requested review from a team, ArthurSens and dashpole as code owners November 25, 2025 21:18
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Nov 25, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

@github-actions github-actions bot added the first-time contributor PRs made by new contributors label Nov 25, 2025
@github-actions
Copy link
Contributor

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

Important reminders:

A maintainer will review your pull request soon. Thank you for helping make OpenTelemetry better!

@github-actions github-actions bot added the receiver/prometheus Prometheus receiver label Nov 25, 2025
Copy link
Member

@ArthurSens ArthurSens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thanks for working on this. Should we also update our README file?

fr0stylo and others added 2 commits November 26, 2025 16:09
@fr0stylo
Copy link
Contributor Author

Resolved comments and updated README.md

@fr0stylo fr0stylo requested a review from ArthurSens November 26, 2025 20:40
Copy link
Member

@ArthurSens ArthurSens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We are almost ready; we also need to update a few additional places[1][2] in our README to mention that the existing configuration option is deprecated and that people should use the feature flag instead.

… in favor of `EnableReportExtraScrapeMetrics` feature gate
@fr0stylo
Copy link
Contributor Author

README.md updated

@fr0stylo fr0stylo requested a review from ArthurSens November 27, 2025 20:55
Copy link
Member

@ArthurSens ArthurSens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! LGTM

@atoulme atoulme merged commit e3474d7 into open-telemetry:main Nov 28, 2025
189 checks passed
@otelbot
Copy link
Contributor

otelbot bot commented Nov 28, 2025

Thank you for your contribution @fr0stylo! 🎉 We would like to hear from you about your experience contributing to OpenTelemetry by taking a few minutes to fill out this survey. If you are getting started contributing, you can also join the CNCF Slack channel #opentelemetry-new-contributors to ask for guidance and get help.

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 receiver/prometheus Prometheus receiver

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PrometheusReceiver Configuration - Deprecate ReportExtraScrapeMetrics

3 participants