From 31718581b5603f32f25f09d1c71187d4199f78af Mon Sep 17 00:00:00 2001
From: Gifford Nowland <hi@giffordnowland.com>
Date: Fri, 9 May 2025 04:04:50 -0700
Subject: [PATCH] Fix: helm template redis (#7563)

### What problem does this PR solve?

Fixes bug & regression introduced by [PR #7187 - refactor: Update Redis
configuration to use StatefulSet instead of deployment with
pvc](https://github.com/infiniflow/ragflow/pull/7187):

1. Fixes bug #7403 - `redis.persistence.enabled` missing from
`helm/values.yaml` causes helm error:

[ERROR] templates/: template: ragflow/templates/redis.yaml:55:24:
executing "ragflow/templates/redis.yaml" at
<.Values.redis.persistence.enabled>: nil pointer evaluating interface
{}.enabled

2. Fixes regression: reverts hardcoded redis.storage.capacity value back
to using variable `redis.storage.capacity` from `helm/values.yaml`.

### Type of change

- [x] Bug Fix (non-breaking change which fixes an issue)
- [ ] New Feature (non-breaking change which adds functionality)
- [ ] Documentation Update
- [ ] Refactoring
- [ ] Performance Improvement
- [ ] Other (please describe):
---
 helm/templates/redis.yaml | 2 +-
 helm/values.yaml          | 2 ++
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/helm/templates/redis.yaml b/helm/templates/redis.yaml
index 4c59a5412..9ac1e951f 100644
--- a/helm/templates/redis.yaml
+++ b/helm/templates/redis.yaml
@@ -76,7 +76,7 @@ spec:
         {{- end }}
         resources:
           requests:
-            storage: 8Gi
+            storage: {{ .Values.redis.storage.capacity }}
   {{- end }}
 ---
 apiVersion: v1
diff --git a/helm/values.yaml b/helm/values.yaml
index 5a53532ab..06c34db37 100644
--- a/helm/values.yaml
+++ b/helm/values.yaml
@@ -133,6 +133,8 @@ redis:
   storage:
     className:
     capacity: 5Gi
+  persistence:
+    enabled: true
   deployment:
     strategy:
     resources: