Fix sentry values management

FlowFuse · Oct 13, 2023 · 3f8179e · 3f8179e
1 parent 3995837
commit 3f8179e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/helm/flowforge/templates/configmap.yaml b/helm/flowforge/templates/configmap.yaml
@@ -129,7 +129,7 @@ data:
       http: info
     telemetry:
       enabled: {{ .Values.forge.telemetry.enabled }}
-      {{ if or (.Values.forge.telemetry.plausible) (.Values.forge.telemetry.posthog) (hasKey .Values.forge.telemetry.sentry "frontend_dsn") }}
+      {{ if or (.Values.forge.telemetry.plausible) (.Values.forge.telemetry.posthog) (and (hasKey .Values.forge.telemetry "sentry") (hasKey .Values.forge.telemetry.sentry "frontend_dsn")) }}
       frontend:
         {{ if .Values.forge.telemetry.plausible -}}
         plausible:

diff --git a/helm/flowforge/templates/deployment.yaml b/helm/flowforge/templates/deployment.yaml
@@ -58,7 +58,7 @@ spec:
         - name: NODE_EXTRA_CA_CERTS
           value: /usr/local/ssl-certs/chain.pem
         {{- end }}
-        {{- if .Values.forge.telemetry.sentry.environment }}
+        {{- if and (hasKey .Values.forge.telemetry "sentry") (hasKey .Values.forge.telemetry.sentry "environment") }}
         - name: SENTRY_ENV
           value: {{ .Values.forge.telemetry.sentry.environment }}
         {{- end }}