Merge pull request #147 from marcopaggioro/feature/add-volume-subpaths

Feature/add volume subPaths for PVCs
docker-mailserver · Feb 6, 2025 · a475362 · a475362
2 parents 89f0ffa + 5681a14
commit a475362
Show file tree

Hide file tree

Showing 4 changed files with 20 additions and 1 deletion.
diff --git a/charts/docker-mailserver/Chart.yaml b/charts/docker-mailserver/Chart.yaml
@@ -2,7 +2,7 @@ apiVersion: v2
 appVersion: "14.0.0"
 description: A fullstack but simple mailserver (smtp, imap, antispam, antivirus, ssl...) using Docker.
 name: docker-mailserver
-version: 4.1.2
+version: 4.1.3
 sources:
 - https://github.com/docker-mailserver/docker-mailserver-helm
 maintainers:

diff --git a/charts/docker-mailserver/templates/deployment.yaml b/charts/docker-mailserver/templates/deployment.yaml
@@ -171,6 +171,9 @@ spec:
           {{- if $persistence.enabled }}
             - name: {{ $name }}
               mountPath: {{ $persistence.mountPath }}
+            {{- if $persistence.subPath }}
+              subPath: {{ $persistence.subPath }}
+            {{- end }}
           {{- end }}
         {{- end }}
 

diff --git a/charts/docker-mailserver/tests/deployment_test.yaml b/charts/docker-mailserver/tests/deployment_test.yaml
@@ -21,6 +21,16 @@ tests:
           path: spec.template.spec.containers[0].image
           value: bananas:in-pajamas
 
+  - it: should use subPath for volumes if defined
+    set:
+      persistence:
+        mail-data:
+          subPath: another-folder
+    asserts:
+      - equal:
+          path: spec.template.spec.containers[0].volumeMounts[?(@.name=="mail-data")].subPath
+          value: another-folder
+
   - it: manifest should match snapshot
     asserts:
       - matchSnapshot: { }
diff --git a/charts/docker-mailserver/values.yaml b/charts/docker-mailserver/values.yaml
@@ -282,11 +282,13 @@ deployment:
   #    volumeMounts:
   #      - name: extra-storage
   #        mountPath: /mnt/extra-storage
+  #        subPath: another-folder
 
   ## Optionally specify a list of extra mounts to add (normally used with extraVolumes)
   extraVolumeMounts: []
   #  - name: extra-storage
   #    mountPath: /mnt/extra-storage
+  #    subPath: another-folder
 
   ## Optionally specify a list of extra volumes to add
   extraVolumes: []
@@ -327,6 +329,7 @@ persistence:
     enabled: true
     existingClaim: ""
     mountPath: /tmp/docker-mailserver
+    subPath: ""
     size: "1Mi"
     annotations: {}
     accessModes:
@@ -338,6 +341,7 @@ persistence:
   mail-data:
     enabled: true
     existingClaim: ""
+    subPath: ""
     size: 10Gi
     mountPath: /var/mail
     annotations: {}
@@ -352,6 +356,7 @@ persistence:
     enabled: true
     existingClaim: ""
     mountPath: /var/mail-state
+    subPath: ""
     size: "1Gi"
     annotations: {}
     accessModes:
@@ -364,6 +369,7 @@ persistence:
     enabled: true
     existingClaim: ""
     mountPath: /var/log/mail
+    subPath: ""
     size: "1Gi"
     annotations: {}
     accessModes: