Skip to content

ROX-28151: Prometheus registry for custom metrics#15807

Closed
parametalol wants to merge 1 commit intomichael/ROX-28151-minimal-aggregatorfrom
michael/ROX-28151-minimal-custom-registry
Closed

ROX-28151: Prometheus registry for custom metrics#15807
parametalol wants to merge 1 commit intomichael/ROX-28151-minimal-aggregatorfrom
michael/ROX-28151-minimal-custom-registry

Conversation

@parametalol
Copy link
Copy Markdown
Contributor

@parametalol parametalol commented Jun 23, 2025

Description

A custom Prometheus registry serves the aggregated metrics, and is separated from the default registry that serves other general central metrics.

User-facing documentation

Testing and quality

  • the change is production ready: the change is GA, or otherwise the functionality is gated by a feature flag
  • CI results are inspected

Automated testing

  • added unit tests
  • added e2e tests
  • added regression tests
  • added compatibility tests
  • modified existing tests

How I validated my change

CI

Current dependencies on/for this PR:

@openshift-ci
Copy link
Copy Markdown

openshift-ci bot commented Jun 23, 2025

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@rhacs-bot
Copy link
Copy Markdown
Contributor

rhacs-bot commented Jun 23, 2025

Images are ready for the commit at c957700.

To use with deploy scripts, first export MAIN_IMAGE_TAG=4.9.x-515-gc957700c5f.

@codecov
Copy link
Copy Markdown

codecov bot commented Jun 23, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 49.10%. Comparing base (30cee42) to head (c957700).

Additional details and impacted files
@@                          Coverage Diff                          @@
##           michael/ROX-28151-minimal-aggregator   #15807   +/-   ##
=====================================================================
  Coverage                                 49.09%   49.10%           
=====================================================================
  Files                                      2644     2645    +1     
  Lines                                    195830   195855   +25     
=====================================================================
+ Hits                                      96151    96182   +31     
+ Misses                                    92163    92159    -4     
+ Partials                                   7516     7514    -2     
Flag Coverage Δ
go-unit-tests 49.10% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@parametalol parametalol requested a review from stehessel August 13, 2025 08:19
@parametalol parametalol force-pushed the michael/ROX-28151-minimal-aggregator branch from 018086d to 43dd7da Compare August 13, 2025 08:20
@parametalol parametalol force-pushed the michael/ROX-28151-minimal-custom-registry branch from 5bdddbb to b1fff5d Compare August 13, 2025 08:33
@parametalol parametalol marked this pull request as ready for review August 13, 2025 08:33
@parametalol parametalol force-pushed the michael/ROX-28151-minimal-custom-registry branch 2 times, most recently from 4129676 to aa7a747 Compare August 14, 2025 11:51
@parametalol parametalol requested review from a team, adisos, janisz and zivnevo as code owners August 14, 2025 11:51
@parametalol parametalol requested review from a team and shireenf-ibm as code owners August 14, 2025 11:51
@parametalol parametalol requested review from GrimmiMeloni and removed request for a team August 14, 2025 11:51
@parametalol parametalol force-pushed the michael/ROX-28151-minimal-aggregator branch from 30d5b00 to 7df7ad8 Compare August 14, 2025 11:51
@parametalol parametalol force-pushed the michael/ROX-28151-minimal-aggregator branch from 7df7ad8 to 30cee42 Compare August 18, 2025 09:58
@parametalol parametalol force-pushed the michael/ROX-28151-minimal-custom-registry branch from aa7a747 to c957700 Compare August 18, 2025 09:58
@parametalol
Copy link
Copy Markdown
Contributor Author

Merged within #15797.

@parametalol parametalol closed this Sep 8, 2025
@parametalol parametalol deleted the michael/ROX-28151-minimal-custom-registry branch September 16, 2025 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants