This is an automated email from the ASF dual-hosted git repository.

sijie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar-helm-chart.git


The following commit(s) were added to refs/heads/master by this push:
     new 65dc686  ZooKeeper HTTP port should be exposed by service so we can 
use prometheus (#143)
65dc686 is described below

commit 65dc68654b33f7e7987f6f9590a319deae245238
Author: Frank Kelly <[email protected]>
AuthorDate: Thu Aug 26 02:13:47 2021 -0400

    ZooKeeper HTTP port should be exposed by service so we can use prometheus 
(#143)
    
    Fixes #142
    
    ### Motivation
    
    Expose HTTP Port on ZooKeeper service so we can use Prometheus
    
    ### Modifications
    
    Bug fix to expose HTTP port on ZooKeeper service
---
 charts/pulsar/templates/zookeeper-service.yaml | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/charts/pulsar/templates/zookeeper-service.yaml 
b/charts/pulsar/templates/zookeeper-service.yaml
index 7ee9890..a71849b 100644
--- a/charts/pulsar/templates/zookeeper-service.yaml
+++ b/charts/pulsar/templates/zookeeper-service.yaml
@@ -31,6 +31,9 @@ metadata:
 {{ toYaml .Values.zookeeper.service.annotations | indent 4 }}
 spec:
   ports:
+    # prometheus needs to access /metrics endpoint
+    - name: http
+      port: {{ .Values.zookeeper.ports.http }}
     - name: follower
       port: {{ .Values.zookeeper.ports.follower }}
     - name: leader-election

Reply via email to