refactor: Update Redis configuration to use StatefulSet instead of deployment with pvc (#7187)

### What problem does this PR solve?

This PR changes Redis to be a statefulset. In some situation when we
Redis pod gets rescheduled to another Node, it gets stuck in pending
state due to the PVC attached to another Kubernetes node.

### Type of change

- [ ] Bug Fix (non-breaking change which fixes an issue)
- [ ] New Feature (non-breaking change which adds functionality)
- [ ] Documentation Update
- [X] Refactoring
- [ ] Performance Improvement
- [ ] Other (please describe):
This commit is contained in:
Wanderson Pinto dos Santos 2025-04-22 01:53:30 -03:00 committed by GitHub
parent f2c9ffc056
commit c8194f5fd0
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

View File

@ -1,38 +1,35 @@
--- ---
apiVersion: v1 apiVersion: v1
kind: PersistentVolumeClaim kind: Service
metadata: metadata:
name: {{ include "ragflow.fullname" . }}-redis name: {{ include "ragflow.fullname" . }}-redis
labels: labels:
{{- include "ragflow.labels" . | nindent 4 }} {{- include "ragflow.labels" . | nindent 4 }}
app.kubernetes.io/component: redis app.kubernetes.io/component: redis
spec: spec:
{{- with .Values.redis.storage.className }} ports:
storageClassName: {{ . }} - port: 6379
{{- end }} name: redis
accessModes: protocol: TCP
- ReadWriteOnce clusterIP: None # Headless service for StatefulSet
resources: selector:
requests: {{- include "ragflow.selectorLabels" . | nindent 4 }}
storage: 8Gi app.kubernetes.io/component: redis
--- ---
apiVersion: apps/v1 apiVersion: apps/v1
kind: Deployment kind: StatefulSet
metadata: metadata:
name: {{ include "ragflow.fullname" . }}-redis name: {{ include "ragflow.fullname" . }}-redis
labels: labels:
{{- include "ragflow.labels" . | nindent 4 }} {{- include "ragflow.labels" . | nindent 4 }}
app.kubernetes.io/component: redis app.kubernetes.io/component: redis
spec: spec:
serviceName: {{ include "ragflow.fullname" . }}-redis
replicas: 1 replicas: 1
selector: selector:
matchLabels: matchLabels:
{{- include "ragflow.selectorLabels" . | nindent 6 }} {{- include "ragflow.selectorLabels" . | nindent 6 }}
app.kubernetes.io/component: redis app.kubernetes.io/component: redis
{{- with .Values.redis.deployment.strategy }}
strategy:
{{- . | toYaml | nindent 4 }}
{{- end }}
template: template:
metadata: metadata:
labels: labels:
@ -41,6 +38,7 @@ spec:
annotations: annotations:
checksum/config-env: {{ include (print $.Template.BasePath "/env.yaml") . | sha256sum }} checksum/config-env: {{ include (print $.Template.BasePath "/env.yaml") . | sha256sum }}
spec: spec:
terminationGracePeriodSeconds: 60
containers: containers:
- name: redis - name: redis
image: {{ .Values.redis.image.repository }}:{{ .Values.redis.image.tag }} image: {{ .Values.redis.image.repository }}:{{ .Values.redis.image.tag }}
@ -54,31 +52,59 @@ spec:
ports: ports:
- containerPort: 6379 - containerPort: 6379
name: redis name: redis
{{- if .Values.redis.persistence.enabled }}
volumeMounts:
- name: redis-data
mountPath: /data
{{- end }}
{{- with .Values.redis.deployment.resources }} {{- with .Values.redis.deployment.resources }}
resources: resources:
{{- . | toYaml | nindent 10 }} {{- . | toYaml | nindent 12 }}
{{- end }} {{- end }}
volumeMounts: {{- if .Values.redis.persistence.enabled }}
- mountPath: /data volumeClaimTemplates:
- metadata:
name: redis-data name: redis-data
volumes: labels:
- name: redis-data {{- include "ragflow.labels" . | nindent 10 }}
persistentVolumeClaim: app.kubernetes.io/component: redis
claimName: {{ include "ragflow.fullname" . }}-redis spec:
accessModes:
- ReadWriteOnce
{{- with .Values.redis.storage.className }}
storageClassName: {{ . }}
{{- end }}
resources:
requests:
storage: 8Gi
{{- end }}
--- ---
apiVersion: v1 apiVersion: v1
kind: Service kind: Service
metadata: metadata:
name: {{ include "ragflow.fullname" . }}-redis name: {{ include "ragflow.fullname" . }}-redis-svc
labels: labels:
{{- include "ragflow.labels" . | nindent 4 }} {{- include "ragflow.labels" . | nindent 4 }}
app.kubernetes.io/component: redis app.kubernetes.io/component: redis
spec: spec:
ports:
- port: 6379
targetPort: redis
protocol: TCP
selector: selector:
{{- include "ragflow.selectorLabels" . | nindent 4 }} {{- include "ragflow.selectorLabels" . | nindent 4 }}
app.kubernetes.io/component: redis app.kubernetes.io/component: redis
ports: ---
- protocol: TCP apiVersion: policy/v1
port: 6379 kind: PodDisruptionBudget
targetPort: redis metadata:
type: {{ .Values.redis.service.type }} name: {{ include "ragflow.fullname" . }}-redis-pdb
labels:
{{- include "ragflow.labels" . | nindent 4 }}
app.kubernetes.io/component: redis
spec:
minAvailable: 1
selector:
matchLabels:
{{- include "ragflow.selectorLabels" . | nindent 6 }}
app.kubernetes.io/component: redis