Merge pull request #642 from snyk/feat/configurable-pvc-storage-class-name

feat: allow configuring PVC StorageClass name

Author: ivanstanev

snyk-monitor/templates/pvc.yaml

@@ -1,4 +1,7 @@
-{{ if .Values.pvc.enabled }}
+# We create a PVC only if the default PVC name has not changed.
+# If the name has changed, it means our users want to use their own pre-provisioned PVC.
+# In such cases it would be an error to create a PVC as it would result in a conflict - the PVC already exists.
+{{- if .Values.pvc.enabled }}{{- if eq .Values.pvc.name "snyk-monitor-pvc" }}
 kind: PersistentVolumeClaim
 apiVersion: v1
 metadata:
@@ -10,4 +13,5 @@ spec:
   resources:
     requests:
       storage: {{ .Values.temporaryStorageSize }}
-{{ end }}
+  storageClassName: {{ .Values.pvc.storageClassName }}
+{{- end }}{{- end }}

snyk-monitor/values.yaml

@@ -39,7 +39,8 @@ temporaryStorageSize: 50Gi #  Applies to PVC too
 # Change to true to use a PVC instead of emptyDir for local storage
 pvc:
   enabled: false
-  name: 'snyk-monitor-pvc'
+  name: snyk-monitor-pvc
+  storageClassName: null
 
 # Node.js in-container process memory enhancements
 envs:

snyk-operator/deploy/olm-catalog/snyk-operator/0.0.0/snyk-operator.v0.0.0.clusterserviceversion.yaml

@@ -23,7 +23,8 @@ metadata:
             "temporaryStorageSize": "50Gi",
             "pvc": {
               "enabled": false,
-              "name": "snyk-monitor-pvc"
+              "name": "snyk-monitor-pvc",
+              "storageClassName": null
             },
             "initContainerImage": {
               "repository": "busybox",
@@ -126,6 +127,12 @@ spec:
             path: pvc.name
             x-descriptors:
               - 'urn:alm:descriptor:text'
+          - description: >-
+              The name of the StorageClass to use for the PVC, when enabled.
+            displayName: PVC StorageClass name
+            path: pvc.storageClassName
+            x-descriptors:
+              - 'urn:alm:descriptor:text'
           - description: >-
               The repo to use for initContainer, if overriding.
             displayName: InitContainer image repo

snyk-operator/deploy/olm-catalog/snyk-operator/0.0.0/snykmonitors.charts.helm.k8s.io.crd.yaml

@@ -52,6 +52,8 @@ spec:
                   type: boolean
                 name:
                   type: string
+                storageClassName:
+                  type: string
               type: object
             initContainerImage:
               properties: