Merge pull request #47 from XenitAB/metrics-port

add metrics port to helm chart
XenitAB · Sep 21, 2021 · 9485a18 · 9485a18
2 parents 43d4007 + 83b16fe
commit 9485a18
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/charts/mqtt-log-stdout/templates/deployment.yaml b/charts/mqtt-log-stdout/templates/deployment.yaml
@@ -41,13 +41,19 @@ spec:
               value: "{{ .Values.mqtt_settings.host_3 }}"
             {{- end }}
             - name: MQTT_PORT
-              value: "{{ .Values.mqtt_settings.port}}"
+              value: "{{ .Values.mqtt_settings.port }}"
             - name: LOG_TOPIC
               value: "{{ required "A valid .Values.mqtt_settings.topic entry required!" .Values.mqtt_settings.topic}}"
+            - name: METRICS_PORT
+              value: "{{ .Values.metrics.port }}"
           securityContext:
             {{- toYaml .Values.securityContext | nindent 12 }}
           image: "{{ .Values.image.repository }}:{{ .Values.image.tag | default .Chart.AppVersion }}"
           imagePullPolicy: {{ .Values.image.pullPolicy }}
+          ports:
+            - containerPort: {{ .Values.metrics.port }}
+              name: metrics
+              protocol: TCP
       {{- with .Values.nodeSelector }}
       nodeSelector:
         {{- toYaml . | nindent 8 }}

diff --git a/charts/mqtt-log-stdout/values.yaml b/charts/mqtt-log-stdout/values.yaml
@@ -55,3 +55,6 @@ mqtt_settings:
   host_3: ""
   port: 1883
   topic: ""
+
+metrics:
+  port: 8080