From 88d3496a991fbdd1dd610cbb1d2ea26ab57c0d89 Mon Sep 17 00:00:00 2001
From: Dean Sheather <dean@deansheather.com>
Date: Mon, 19 Dec 2022 04:48:08 +1000
Subject: [PATCH] fix: fix helm prometheus block causing failures (#5458)

---
 helm/templates/coder.yaml | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/helm/templates/coder.yaml b/helm/templates/coder.yaml
index 44847032a0..85b01254a5 100644
--- a/helm/templates/coder.yaml
+++ b/helm/templates/coder.yaml
@@ -89,12 +89,20 @@ spec:
               protocol: TCP
             {{- end }}
             {{- range .Values.coder.env }}
-            {{- if and (eq .name "CODER_PROMETHEUS_ENABLE") (eq .value "true") }}
+            {{- if eq .name "CODER_PROMETHEUS_ENABLE" }}
+            {{/*
+            This sadly has to be nested to avoid evaluating the second part
+            of the condition too early and potentially getting type errors if
+            the value is not a string (like a `valueFrom`). We do not support
+            `valueFrom` for this env var specifically.
+            */}}
+            {{- if eq .value "true" }}
             - name: "prometheus-http"
               containerPort: 6060
               protocol: TCP
             {{- end }}
             {{- end }}
+            {{- end }}
           readinessProbe:
             httpGet:
               path: /api/v2/buildinfo