You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pulsar.apache.org by GitBox <gi...@apache.org> on 2020/07/23 00:32:24 UTC

[GitHub] [pulsar-helm-chart] jharris- opened a new pull request #44: Add zookeeper metrics port and PodMonitors

jharris- opened a new pull request #44:
URL: https://github.com/apache/pulsar-helm-chart/pull/44


   Fixes #<xyz>
   
   ### Motivation
   
   This change allows Prometheus-Operator (external to pulsar deployment) to scrape bookkeeper, broker, proxy, and zookeeper targets. 
   
   ### Modifications
   
   - Specified the 'http' container port for zookeeper pods which makes it aligned with the other container types for Prometheus metric scraping
   - Added PodMonitors for bookkeeper, broker, proxy, and zookeeper pods
   
   ### Verifying this change
   
   - Did not see any CI checks...
   - Verified http container port is added in helm template command
   - Verified PodMonitor enable switch for template rendering works via helm template
   


----------------------------------------------------------------
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.

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



[GitHub] [pulsar-helm-chart] wolfstudy merged pull request #44: Add zookeeper metrics port and PodMonitors

Posted by GitBox <gi...@apache.org>.
wolfstudy merged pull request #44:
URL: https://github.com/apache/pulsar-helm-chart/pull/44


   


----------------------------------------------------------------
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.

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