-
Notifications
You must be signed in to change notification settings - Fork 3.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
expose broker bundles metrics to prometheus #12366
expose broker bundles metrics to prometheus #12366
Conversation
/pulsarbot run-failure-checks |
1 similar comment
/pulsarbot run-failure-checks |
@BewareMyPower PTAL |
@gaozhangmin thanks for your contribution! Please do not forget to add docs accordingly to allow users to know your great code changes. Feel free to ping me if you need a doc review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
...r-broker/src/main/java/org/apache/pulsar/broker/loadbalance/impl/ModularLoadManagerImpl.java
Outdated
Show resolved
Hide resolved
d9d54e5
to
d18cc77
Compare
@hangc0276 PTAL again |
Hi @gaozhangmin any progress on docs? Thanks |
Doc is added #12825 |
(cherry picked from commit 0f9bcbc)
…) (merge request !158) Squash merge branch 'release-2.8.1-bundle-metric' into 'release-2.8.1.4' Fixes #<xyz> ### Motivation --story=878795699 扩展bundle级别的metric指标apache#12366 (merge request !157) TAPD: --story=878795699
…) (merge request !158) Squash merge branch 'release-2.8.1-bundle-metric' into 'release-2.8.1.4' Fixes #<xyz> ### Motivation --story=878795699 扩展bundle级别的metric指标apache#12366 (merge request !157) TAPD: --story=878795699
Expose broker bundles metrics to prometheus.
Documentation
Docs will update in another pr.