Skip to content

Commit

Permalink
Add both storage and snapshot volume names to values.yaml (#181)
Browse files Browse the repository at this point in the history
Co-authored-by: Tony Skorik <anton.skorik@aliexpress.ru>
  • Loading branch information
TonySkorik and Tony Skorik authored May 6, 2024
1 parent b892dd0 commit 55a591b
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 7 deletions.
14 changes: 7 additions & 7 deletions charts/qdrant/templates/statefulset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -52,9 +52,9 @@ spec:
- {{ .Values.snapshotRestoration.mountPath }}
{{- end }}
volumeMounts:
- name: qdrant-storage
- name: {{ .Values.persistence.storageVolumeName | default "qdrant-storage" }}
mountPath: /qdrant/storage
- name: qdrant-snapshots
- name: {{ .Values.snapshotPersistence.snapshotsVolumeName | default "qdrant-snapshots" }}
mountPath: /qdrant/snapshots
{{- if and .Values.snapshotRestoration.enabled .Values.snapshotRestoration.pvcName }}
- name: qdrant-snapshot-restoration
Expand Down Expand Up @@ -162,7 +162,7 @@ spec:
{{- toYaml . | nindent 12 }}
{{- end }}
volumeMounts:
- name: qdrant-storage
- name: {{ .Values.persistence.storageVolumeName | default "qdrant-storage" }}
mountPath: /qdrant/storage
- name: qdrant-config
mountPath: /qdrant/config/initialize.sh
Expand All @@ -179,7 +179,7 @@ spec:
- name: qdrant-snapshot-restoration
mountPath: {{ .Values.snapshotRestoration.mountPath }}
{{- end }}
- name: qdrant-snapshots
- name: {{ .Values.snapshotPersistence.snapshotsVolumeName | default "qdrant-snapshots" }}
mountPath: /qdrant/snapshots
- name: qdrant-init
mountPath: /qdrant/init
Expand Down Expand Up @@ -218,7 +218,7 @@ spec:
claimName: {{ .Values.snapshotRestoration.pvcName }}
{{- end }}
{{- if not .Values.snapshotPersistence.enabled }}
- name: qdrant-snapshots
- name: {{ .Values.snapshotPersistence.snapshotsVolumeName | default "qdrant-snapshots" }}
emptyDir: {}
{{- end }}
- name: qdrant-init
Expand All @@ -234,7 +234,7 @@ spec:
{{- end}}
volumeClaimTemplates:
- metadata:
name: qdrant-storage
name: {{ .Values.persistence.storageVolumeName | default "qdrant-storage" }}
labels:
app: {{ template "qdrant.name" . }}
{{- with .Values.persistence.annotations }}
Expand All @@ -252,7 +252,7 @@ spec:
storage: {{ .Values.persistence.size | quote }}
{{- if .Values.snapshotPersistence.enabled }}
- metadata:
name: qdrant-snapshots
name: {{ .Values.snapshotPersistence.snapshotsVolumeName | default "qdrant-snapshots" }}
labels:
app: {{ template "qdrant.name" . }}
{{- with .Values.snapshotPersistence.annotations }}
Expand Down
2 changes: 2 additions & 0 deletions charts/qdrant/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -128,6 +128,7 @@ persistence:
accessModes: ["ReadWriteOnce"]
size: 10Gi
annotations: {}
# storageVolumeName: qdrant-storage
# storageClassName: local-path

# If you use snapshots or the snapshot shard transfer mechanism, we recommend
Expand All @@ -138,6 +139,7 @@ snapshotPersistence:
accessModes: ["ReadWriteOnce"]
size: 10Gi
annotations: {}
# snapshotsVolumeName: qdrant-snapshots
# You can change the storageClassName to ensure snapshots are saved to cold storage.
# storageClassName: local-path

Expand Down

0 comments on commit 55a591b

Please sign in to comment.