Merge pull request #20 from nsqio/prom_port_anno

update metrics service prometheus port annotation

charts/nsq/Chart.yaml

@@ -2,7 +2,7 @@ apiVersion: v2
 name: nsq
 description: A realtime distributed messaging platform
 type: application
-version: 0.0.10
+version: 0.0.11
 appVersion: 1.2.1
 home: https://github.com/nsqio/helm-chart/tree/master/charts/nsq
 icon: https://nsq.io/static/img/nsq_blue.png

charts/nsq/values.yaml

@@ -177,8 +177,12 @@ metrics:
 
   service:
     annotations:
+      # Prometheus does not connect to this service directly,
+      # instead it uses this port to connect to each target
+      # of the service (each nsqd).
+      # TODO: remove service and annotate pods instead?
       prometheus.io/scrape: "true"
-      prometheus.io/port: "{{ .Values.metrics.service.ports.metrics }}"
+      prometheus.io/port: "{{ .Values.metrics.containerPort }}"
       prometheus.io/path: "/metrics"
     type: ClusterIP
     ports: