[prometheus-kafka-exporter] Add kafka broker version (prometheus-comm…

…unity#896) * prometheus-kafka-exporter: add broker version Signed-off-by: Evyatar Cohen <coevyatar@gmail.com> * prometheus-kafka-exporter: fix trailing spaces Signed-off-by: Evyatar Cohen <coevyatar@gmail.com>
holiday-kim · Apr 28, 2021 · fa75fe2 · fa75fe2
1 parent fc70a3e
commit fa75fe2
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 1 deletion.
diff --git a/charts/prometheus-kafka-exporter/Chart.yaml b/charts/prometheus-kafka-exporter/Chart.yaml
@@ -3,7 +3,7 @@ appVersion: "v1.2.0"
 description: A Helm chart to export the metrics from Kafka in Prometheus format using the kafka-exporter from https://github.com/danielqsj/kafka_exporter
 name: prometheus-kafka-exporter
 home: https://github.com/danielqsj/kafka_exporter
-version: 1.0.0
+version: 1.1.0
 sources:
   - https://gkarthiks.github.io/helm-charts/charts/prometheus-kafka-exporter
 keywords:

diff --git a/charts/prometheus-kafka-exporter/templates/deployment.yaml b/charts/prometheus-kafka-exporter/templates/deployment.yaml
@@ -27,6 +27,9 @@ spec:
           {{- range $server := .Values.kafkaServer }}
             - '--kafka.server={{ $server }}'
           {{- end }}
+          {{- if .Values.kafkaBrokerVersion }}
+            - '--kafka.version={{ .Values.kafkaBrokerVersion }}'
+          {{- end }}
           {{- if .Values.tls.enabled }}
             - '--tls.enabled'
             - '--tls.ca-file={{ .Values.tls.mountPath }}/ca.crt'

diff --git a/charts/prometheus-kafka-exporter/values.yaml b/charts/prometheus-kafka-exporter/values.yaml
@@ -8,6 +8,9 @@ replicaCount: 1
 kafkaServer:
   - kafka-server:9092
 
+# Specifies broker version to use, leave empty for default
+kafkaBrokerVersion:
+
 rbac:
   # Specifies whether RBAC resources should be created
   create: true