From 85fa39f8e215f9b6c02ea4d18cf0f9a2044b91c1 Mon Sep 17 00:00:00 2001
From: Nicolas <nicolas@mathmast.com>
Date: Thu, 31 Jul 2025 10:18:47 +0800
Subject: [PATCH] fix: add default values for logIngest.persistence to prevent
 nil pointer errors

---
 .../centralStorage/templates/central-storage/deployment.yaml  | 2 +-
 .../centralStorage/templates/central-storage/pvc.yaml         | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/freeleaps/helm-pkg/centralStorage/templates/central-storage/deployment.yaml b/freeleaps/helm-pkg/centralStorage/templates/central-storage/deployment.yaml
index cc83273a..dd768fd1 100644
--- a/freeleaps/helm-pkg/centralStorage/templates/central-storage/deployment.yaml
+++ b/freeleaps/helm-pkg/centralStorage/templates/central-storage/deployment.yaml
@@ -114,7 +114,7 @@ spec:
 {{- if .Values.logIngest.enabled }}
       volumes:
         - name: app-logs
-          {{- if .Values.logIngest.persistence.enabled }}
+          {{- if (default false .Values.logIngest.persistence.enabled) }}
           persistentVolumeClaim:
             claimName: {{ .Release.Name }}-logs-pvc
           {{- else }}
diff --git a/freeleaps/helm-pkg/centralStorage/templates/central-storage/pvc.yaml b/freeleaps/helm-pkg/centralStorage/templates/central-storage/pvc.yaml
index 3ac7bee2..52f60c58 100644
--- a/freeleaps/helm-pkg/centralStorage/templates/central-storage/pvc.yaml
+++ b/freeleaps/helm-pkg/centralStorage/templates/central-storage/pvc.yaml
@@ -1,4 +1,4 @@
-{{- if and .Values.logIngest.enabled .Values.logIngest.persistence.enabled }}
+{{- if and .Values.logIngest.enabled (default false .Values.logIngest.persistence.enabled) }}
 apiVersion: v1
 kind: PersistentVolumeClaim
 metadata:
@@ -12,7 +12,7 @@ spec:
     - ReadWriteOnce
   resources:
     requests:
-      storage: {{ .Values.logIngest.persistence.size | default "1Gi" }}
+      storage: {{ default "1Gi" .Values.logIngest.persistence.size }}
   {{- if .Values.logIngest.persistence.storageClass }}
   storageClassName: {{ .Values.logIngest.persistence.storageClass }}
   {{- end }}