Merge pull request 'fix: use hasKey to safely check for logIngest.persistence existence' (#39) from Nicolas_local_ops into master

Reviewed-on: https://gitea.freeleaps.mathmast.com/freeleaps/freeleaps-ops/pulls/39

freeleaps/helm-pkg/centralStorage/templates/central-storage/deployment.yaml

@@ -114,7 +114,7 @@ spec:
 {{- if .Values.logIngest.enabled }}
       volumes:
         - name: app-logs
-          {{- if (default false .Values.logIngest.persistence.enabled) }}
+          {{- if and (hasKey .Values.logIngest "persistence") (.Values.logIngest.persistence.enabled) }}
           persistentVolumeClaim:
             claimName: {{ .Release.Name }}-logs-pvc
           {{- else }}

freeleaps/helm-pkg/centralStorage/templates/central-storage/pvc.yaml

@@ -1,4 +1,4 @@
-{{- if and .Values.logIngest.enabled (default false .Values.logIngest.persistence.enabled) }}
+{{- if and .Values.logIngest.enabled (hasKey .Values.logIngest "persistence") (.Values.logIngest.persistence.enabled) }}
 apiVersion: v1
 kind: PersistentVolumeClaim
 metadata:
@@ -12,7 +12,7 @@ spec:
     - ReadWriteOnce
   resources:
     requests:
-      storage: {{ default "1Gi" .Values.logIngest.persistence.size }}
+      storage: {{ .Values.logIngest.persistence.size | default "1Gi" }}
   {{- if .Values.logIngest.persistence.storageClass }}
   storageClassName: {{ .Values.logIngest.persistence.storageClass }}
   {{- end }}