From 831df39d0a6cdf83727949ed7eb5bcef8ab6c18a Mon Sep 17 00:00:00 2001
From: alexdga <alexandre.barth35@gmail.com>
Date: Wed, 17 Jun 2026 16:00:47 +0200
Subject: [PATCH] Add namespace in default VaultAuth spec only if value is not
 nil in values.yaml

When deploying the chart with argocd in serversideapply mode, a nil value for namespace in default VaultAuth spec is rejected
---
 chart/templates/default-vault-auth-method.yaml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/chart/templates/default-vault-auth-method.yaml b/chart/templates/default-vault-auth-method.yaml
index 2c223297..92fef45c 100644
--- a/chart/templates/default-vault-auth-method.yaml
+++ b/chart/templates/default-vault-auth-method.yaml
@@ -15,7 +15,9 @@ metadata:
     app.kubernetes.io/component: controller-manager
   {{- include "vso.chart.labels" . | nindent 4 }}
 spec:
+  {{- if .Values.defaultAuthMethod.namespace }}
   namespace: {{ .Values.defaultAuthMethod.namespace }}
+  {{- end }}
   {{- if .Values.defaultAuthMethod.allowedNamespaces }}
   allowedNamespaces:
   {{- toYaml .Values.defaultAuthMethod.allowedNamespaces | nindent 4 }}