feat: unset default storage class in chart values

The correct way to specify a default storage class in Kubernetes is
via the `storageclass.kubernetes.io/is-default-class` annotation [1]
rather than assuming a storage class named 'standard' exists.

[1] https://kubernetes.io/docs/concepts/storage/storage-classes/#default-storageclass

Signed-off-by: sd109 <[email protected]>

Author: sd109

CHANGELOG.md

@@ -13,4 +13,6 @@ All notable changes to this project will be documented in this file.
 
 ### Changed
 
+- Changed default storageClassName to empty.
+
 ### Removed

helm/slurm/README.md

@@ -81,7 +81,7 @@ Helm Chart for Slurm HPC Workload Manager
 | controller.persistence.labels | object | `{}` |  Create a `PersistentVolumeClaim` with these labels. |
 | controller.persistence.selector | object | `{}` |  Selector to match an existing `PersistentVolume`. |
 | controller.persistence.size | string | `"4Gi"` |  Create a `PersistentVolumeClaim` with this storage size. |
-| controller.persistence.storageClass | string | `"standard"` |  Create a `PersistentVolumeClaim` with this storage class. |
+| controller.persistence.storageClass | string | `nil` |  Create a `PersistentVolumeClaim` with this storage class. |
 | controller.priorityClassName | string | `""` |  Set the priority class to use. Ref: https://kubernetes.io/docs/concepts/scheduling-eviction/pod-priority-preemption/#priorityclass |
 | controller.resources | object | `{}` |  Set container resource requests and limits for Kubernetes Pod scheduling. Ref: https://kubernetes.io/docs/concepts/configuration/manage-resources-containers/#resource-requests-and-limits-of-pod-and-container |
 | controller.service | object | `{}` |  The controller service configuration. Ref: https://kubernetes.io/docs/concepts/services-networking/service/ |
@@ -129,7 +129,7 @@ Helm Chart for Slurm HPC Workload Manager
 | mariadb.primary.persistence.labels | object | `{}` |  |
 | mariadb.primary.persistence.selector | object | `{}` |  |
 | mariadb.primary.persistence.size | string | `"8Gi"` |  |
-| mariadb.primary.persistence.storageClass | string | `"standard"` |  |
+| mariadb.primary.persistence.storageClass | string | `nil` |  |
 | mariadb.primary.priorityClassName | string | `""` |  |
 | mariadb.primary.tolerations | list | `[]` |  |
 | mariadb.resources | object | `{}` |  |

helm/slurm/values.yaml

@@ -284,7 +284,7 @@ controller:
     #
     # -- (string)
     # Create a `PersistentVolumeClaim` with this storage class.
-    storageClass: standard
+    storageClass:
     #
     # -- (list)
     # Create a `PersistentVolumeClaim` with these access modes.
@@ -608,7 +608,7 @@ compute:
         #     name: scratch
         #   spec:
         #     mountPath: /mnt/scratch
-        #     storageClassName: standard
+        #     storageClassName:
         #     accessModes:
         #       - ReadWriteOnce
         #     resources:
@@ -849,7 +849,7 @@ mariadb:
     persistence:
       enabled: true
       existingClaim: ""
-      storageClass: standard
+      storageClass:
       labels: {}
       annotations: {}
       accessModes: