Add support for existing PVCs and read-only configurations

Introduce `existingClaim` and `readOnly` fields for PVCs to allow using pre-existing claims and enforcing read-only mounts. Update Deployment and PVC templates to handle these new configurations, providing greater flexibility in volume management.

Author: devclef

charts/shoebox/Chart.yaml

@@ -2,7 +2,7 @@ apiVersion: v2
 name: shoebox
 description: A Helm chart for the Shoebox application - a digital shoebox for your videos
 type: application
-version: 0.0.11
+version: 0.0.12
 appVersion: "main"
 keywords:
   - video

charts/shoebox/templates/deployment.yaml

@@ -66,39 +66,56 @@ spec:
             {{- if and .Values.persistence.data.enabled (not .Values.postgresql.enabled) }}
             - name: data
               mountPath: /app/data
+              {{- if .Values.persistence.data.readOnly }}
+              readOnly: true
+              {{- end }}
             {{- end }}
             {{- if .Values.persistence.thumbnails.enabled }}
             - name: thumbnails
               mountPath: /app/thumbnails
+              {{- if .Values.persistence.thumbnails.readOnly }}
+              readOnly: true
+              {{- end }}
             {{- end }}
             {{- if .Values.persistence.exports.enabled }}
             - name: exports
               mountPath: /app/exports
+              {{- if .Values.persistence.exports.readOnly }}
+              readOnly: true
+              {{- end }}
             {{- end }}
             {{- if .Values.persistence.media.enabled }}
             {{- if kindIs "string" .Values.config.mediaSourcePaths }}
             - name: media
               mountPath: /mnt/videos
+              {{- if .Values.persistence.media.readOnly }}
               readOnly: true
+              {{- end }}
             {{- else }}
             {{- range .Values.config.mediaSourcePaths.sources }}
             - name: media-{{ .name | lower }}
               mountPath: {{ .path }}
+              {{- if $.Values.persistence.media.readOnly }}
               readOnly: true
+              {{- end }}
             {{- end }}
             {{- end }}
             {{- end }}
             {{- if .Values.persistence.media.enabled }}
             {{- if kindIs "string" .Values.config.mediaSourcePaths }}
             - name: original-media
               mountPath: /home/user/videos
+              {{- if .Values.persistence.media.readOnly }}
               readOnly: true
+              {{- end }}
             {{- else }}
             {{- range .Values.config.mediaSourcePaths.sources }}
             {{- if .originalPath }}
             - name: original-media-{{ .name | lower }}
               mountPath: {{ .originalPath }}
+              {{- if $.Values.persistence.media.readOnly }}
               readOnly: true
+              {{- end }}
             {{- end }}
             {{- end }}
             {{- end }}
@@ -119,12 +136,20 @@ spec:
         {{- if and .Values.persistence.data.enabled (not .Values.postgresql.enabled) }}
         - name: data
           persistentVolumeClaim:
+            {{- if .Values.persistence.data.existingClaim }}
+            claimName: {{ .Values.persistence.data.existingClaim }}
+            {{- else }}
             claimName: {{ include "shoebox.fullname" . }}-data
+            {{- end }}
         {{- end }}
         {{- if .Values.persistence.thumbnails.enabled }}
         - name: thumbnails
           persistentVolumeClaim:
+            {{- if .Values.persistence.thumbnails.existingClaim }}
+            claimName: {{ .Values.persistence.thumbnails.existingClaim }}
+            {{- else }}
             claimName: {{ include "shoebox.fullname" . }}-thumbnails
+            {{- end }}
         {{- end }}
         {{- if .Values.persistence.exports.enabled }}
         - name: exports

charts/shoebox/templates/pvc.yaml

@@ -1,4 +1,4 @@
-{{- if and .Values.persistence.data.enabled (not .Values.postgresql.enabled) }}
+{{- if and .Values.persistence.data.enabled (not .Values.postgresql.enabled) (not .Values.persistence.data.existingClaim) }}
 apiVersion: v1
 kind: PersistentVolumeClaim
 metadata:
@@ -16,7 +16,7 @@ spec:
   {{- end }}
 ---
 {{- end }}
-{{- if .Values.persistence.thumbnails.enabled }}
+{{- if and .Values.persistence.thumbnails.enabled (not .Values.persistence.thumbnails.existingClaim) }}
 apiVersion: v1
 kind: PersistentVolumeClaim
 metadata:

charts/shoebox/values.yaml

@@ -119,26 +119,33 @@ persistence:
     size: 1Gi
     storageClass: ""
     accessMode: ReadWriteOnce
+    existingClaim: ""
+    readOnly: false
 
   # Thumbnails directory
   thumbnails:
     enabled: true
     size: 5Gi
     storageClass: ""
     accessMode: ReadWriteOnce
+    existingClaim: ""
+    readOnly: false
 
   # Exports directory
   exports:
     enabled: true
     size: 10Gi
     storageClass: ""
     accessMode: ReadWriteOnce
+    existingClaim: ""
+    readOnly: false
 
   # Media source directory (read-only)
   media:
     enabled: true
     # This should be a PV that points to your media storage
     existingClaim: ""
+    readOnly: true
     # If no existing claim, create a PVC with these settings
     size: 100Gi
     storageClass: ""