fix: update helm chart to make the initContainer optional and allow s…

…pecifying PVC access mode (#150) Signed-off-by: Harikrishnan Balagopal <harikrishmenon@gmail.com>
konveyor · Aug 10, 2023 · b85f422 · b85f422
1 parent d2f62f9
commit b85f422
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 4 deletions.
diff --git a/helm-charts/move2kube/templates/move2kube-api-stateful-set.yaml b/helm-charts/move2kube/templates/move2kube-api-stateful-set.yaml
@@ -16,7 +16,7 @@ spec:
       labels:
         move2kube.konveyor.io/service: "{{ .Release.Name }}-move2kubeapi"
     spec:
-      {{- if .Values.persistentVolumeClaim.enable }}
+      {{- if and (.Values.deployment.api.initContainer) (.Values.persistentVolumeClaim.enable) }}
       initContainers:
         - name: change-perms
           image: quay.io/konveyor/busybox

diff --git a/helm-charts/move2kube/templates/persistent-volume-claim.yaml b/helm-charts/move2kube/templates/persistent-volume-claim.yaml
@@ -8,9 +8,13 @@ metadata:
 spec:
   storageClassName: "{{ .Values.persistentVolumeClaim.storageClassName }}"
   accessModes:
+    {{- if or (gt (int .Values.deployment.api.replicas) 1) (.Values.deployment.database.enable) (.Values.persistentVolumeClaim.readWriteMany) }}
     - ReadWriteMany
+    {{- else }}
+    - ReadWriteOnce
+    {{- end }}
   volumeMode: Filesystem
   resources:
     requests:
       storage: "{{ .Values.persistentVolumeClaim.storageSize }}"
-{{- end }}
+{{- end }}
diff --git a/helm-charts/move2kube/values.schema.json b/helm-charts/move2kube/values.schema.json
@@ -74,7 +74,10 @@
                 },
                 "cpu": {
                     "type": "string"
-                }
+                },
+		"initContainer": {
+	            "type": "boolean"
+		}
             },
             "required": [
                 "cpu",
@@ -156,7 +159,10 @@
                 },
                 "storageSize": {
                     "type": "string"
-                }
+                },
+		"readWriteMany": {
+		    "type": "boolean"
+		}
             },
             "required": [
                 "createNew",

diff --git a/helm-charts/move2kube/values.yaml b/helm-charts/move2kube/values.yaml
@@ -5,6 +5,7 @@ deployment:
     imageTag: "" # if empty we will use AppVersion from the Chart.yaml as the tag
     memory: "1Gi"
     cpu: "300m"
+    initContainer: false
   authServer:
     enable: false
     replicas: 1
@@ -27,6 +28,7 @@ persistentVolumeClaim:
   name: "" # if createNew is false this MUST contain the name of an existing persistent volume claim
   storageClassName: ""
   storageSize: 1Gi
+  readWriteMany: true
 secret:
   api:
     enable: false