From d03881d85f034c2d249288473c336bdf21b3a600 Mon Sep 17 00:00:00 2001
From: Nebula <40148908+nebula-it@users.noreply.github.com>
Date: Mon, 2 Dec 2024 12:59:34 -0800
Subject: [PATCH] Fix empty clusterIP when no clusterIP is defined in values
 (#426)

```
clusterIP:
```
Should only show up in manifest if `.Values.service.clusterIP` is defined. Otherwise it shows an empty value, which causes argoCD to report app as out of sync because of expects `spec.clusterIP` to have the auto assigned IP but manifest states it should be null value.
---
 deployment/templates/service.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/deployment/templates/service.yaml b/deployment/templates/service.yaml
index ac142fb3..8a398b17 100644
--- a/deployment/templates/service.yaml
+++ b/deployment/templates/service.yaml
@@ -27,7 +27,7 @@ metadata:
   {{- end }}
 spec:
   type: {{ .Values.service.type }}
-  {{- if eq .Values.service.type "ClusterIP" }}
+  {{- if .Values.service.clusterIP }}
   clusterIP: {{ .Values.service.clusterIP | quote }}
   {{- end }}
   ports: