Address review comments

canonical · Feb 1, 2024 · b53a23e · b53a23e
1 parent ea22d16
commit b53a23e
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/deploy/charts/rawfile-csi/templates/01-controller-plugin.yaml b/deploy/charts/rawfile-csi/templates/01-controller-plugin.yaml
@@ -42,12 +42,9 @@ spec:
           image: "{{ .Values.controller.image.repository }}:{{ .Values.controller.image.tag }}"
           imagePullPolicy: {{ .Values.controller.image.pullPolicy }}
           args:
-          {{- if regexMatch "\\/canonical\\/" .Values.controller.image.repository }}
-            - --args
-            - rawfile
-          {{- end }}
-            - csi-driver
-            - --disable-metrics
+            {{ range .Values.controller.csi-driver-args }}
+            - {{ . }}
+            {{ end }}
           env:
             - name: PROVISIONER_NAME
               value: "{{ .Values.provisionerName }}"

diff --git a/deploy/charts/rawfile-csi/templates/01-node-plugin.yaml b/deploy/charts/rawfile-csi/templates/01-node-plugin.yaml
@@ -51,7 +51,7 @@ spec:
             type: DirectoryOrCreate
         - name: data-dir
           hostPath:
-            path: "{{ .Values.node.storage.path }}"
+            path: {{ .Values.node.storage.path | quote }}
             type: DirectoryOrCreate
       containers:
         - name: csi-driver

diff --git a/deploy/charts/rawfile-csi/values.yaml b/deploy/charts/rawfile-csi/values.yaml
@@ -15,6 +15,9 @@ defaults: &defaults
 
 controller:
   <<: *defaults
+  csi-driver-args:
+    - csi-driver
+    - --disable-metrics
 
 node:
   <<: *defaults