dajac opened a new pull request #11273:
URL: https://github.com/apache/kafka/pull/11273


   This PR adds the `ActiveBrokerCount` and the `FencedBrokerCount` metrics to 
the ZK controller. Note that `FencedBrokerCount` is always set to zero in the 
ZK controller.
   
   `testControllerMetrics` has been extended to ensure that the metrics are 
exposed. However, I was not able to build a test with multiple brokers to 
verify the counts due to the shared common yammer metrics registry which is 
used. I have tested the metrics manually as well.
   
   ### Committer Checklist (excluded from commit message)
   - [ ] Verify design and implementation 
   - [ ] Verify test coverage and CI build status
   - [ ] Verify documentation (including upgrade notes)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to