Skip to content

Remove config 'report_extra_scrape_metrics'#46725

Merged
songy23 merged 1 commit into
open-telemetry:mainfrom
ArthurSens:remove-reportextrascrapemetrics
Mar 17, 2026
Merged

Remove config 'report_extra_scrape_metrics'#46725
songy23 merged 1 commit into
open-telemetry:mainfrom
ArthurSens:remove-reportextrascrapemetrics

Conversation

@ArthurSens
Copy link
Copy Markdown
Member

Description

Completely removes the config options and obsolete feature gates

Link to tracking issue

Fixes #44181

Testing

Documentation

@ArthurSens ArthurSens requested review from a team and dashpole as code owners March 6, 2026 16:55
@github-actions github-actions Bot added the receiver/prometheus Prometheus receiver label Mar 6, 2026
@github-actions github-actions Bot requested review from Aneurysm9 and krajorama March 6, 2026 16:56
@ArthurSens
Copy link
Copy Markdown
Member Author

What I'm still in doubt about is if this PR needs to wait for 0.148.0 before getting merged 🤔

Copy link
Copy Markdown
Contributor

@dashpole dashpole left a comment

Choose a reason for hiding this comment

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

I do think this should wait until after the v1.48.0 release.

@ArthurSens ArthurSens added the on hold This is blocked by another PR/issue label Mar 6, 2026
@ArthurSens ArthurSens removed the on hold This is blocked by another PR/issue label Mar 17, 2026
Signed-off-by: Arthur Silva Sens <arthursens2005@gmail.com>
@ArthurSens ArthurSens force-pushed the remove-reportextrascrapemetrics branch from c0a5bbc to 8c69db6 Compare March 17, 2026 17:20
@ArthurSens ArthurSens added the ready to merge Code review completed; ready to merge by maintainers label Mar 17, 2026
@ArthurSens
Copy link
Copy Markdown
Member Author

v0.148.0 is out, this should be good to go

@songy23 songy23 merged commit dd9bb15 into open-telemetry:main Mar 17, 2026
207 of 208 checks passed
@ArthurSens ArthurSens deleted the remove-reportextrascrapemetrics branch March 17, 2026 21:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready to merge Code review completed; ready to merge by maintainers receiver/prometheus Prometheus receiver

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PrometheusReceiver Configuration - Deprecate ReportExtraScrapeMetrics

5 participants