14589: Bugfix to support boolean flags in prometheus-consul-exporter

Signed-off-by: Jan Fiedler <[email protected]>
helm · Jun 11, 2019 · 899d8c8 · 899d8c8
1 parent 5ccaad3
commit 899d8c8
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/stable/prometheus-consul-exporter/templates/deployment.yaml b/stable/prometheus-consul-exporter/templates/deployment.yaml
@@ -27,10 +27,8 @@ spec:
           command: ["consul_exporter"]
           args: [
             "--consul.server={{ .Values.consulServer }}",
-            {{- range $option := .Values.options }}
-              {{- range $k, $v := . }}
-            "--{{ $k }}={{ $v }}", 
-              {{- end }}
+            {{- range $key, $value := .Values.options }}
+            "--{{ $key }}{{ if $value }}={{ $value }}{{ end }}",
             {{- end }}
           ]
           ports:

diff --git a/stable/prometheus-consul-exporter/values.yaml b/stable/prometheus-consul-exporter/values.yaml
@@ -29,7 +29,9 @@ consulServer: host:port
 
 # Flags - for a list visit https://github.com/prometheus/consul_exporter#flags
 options: {}
-
+  # no-consul.health-summary:
+  # kv.filter=foobar
+
 service:
   type: ClusterIP
   port: 9107