updated server-statefulset.yaml to correct ad.datadoghq.com/consul.lo…

…gs annotation to valid single quote string
hashicorp · Feb 9, 2024 · 96692a2 · 96692a2
1 parent f88bf52
commit 96692a2
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/charts/consul/templates/server-statefulset.yaml b/charts/consul/templates/server-statefulset.yaml
@@ -143,7 +143,7 @@ spec:
         {{- end }}
         {{- if .Values.global.metrics.datadog.enabled }}
         "ad.datadoghq.com/tolerate-unready": "true"
-        "ad.datadoghq.com/consul.logs": {{ .Values.global.metrics.datadog.dogstatsd.dogstatsdTags | squote }}
+        "ad.datadoghq.com/consul.logs": {{ .Values.global.metrics.datadog.dogstatsd.dogstatsdTags | toJson | squote }}
         {{- if .Values.global.metrics.datadog.openMetricsPrometheus.enabled }}
         "ad.datadoghq.com/consul.checks": |
           {

diff --git a/charts/consul/test/unit/server-config-configmap.bats b/charts/consul/test/unit/server-config-configmap.bats
@@ -1364,7 +1364,7 @@ load _helpers
   local actual=$(helm template \
       -s templates/server-config-configmap.yaml  \
       --set 'global.metrics.enabled=true'  \
-      --set 'global.metrics.enableAgentMetrics=true'  \
+      --set 'global.metrics.enableAgentMetrics=true' \
       --set 'global.metrics.datadog.enabled=true'  \
       --set 'global.metrics.datadog.dogstatsd.enabled=true'  \
       . | tee /dev/stderr |