Merge pull request #130 from Leaseweb/prioclass

feat: Set priorityClassName in deploy manifest, split setting in chart

Author: hrak

charts/cloudstack-csi/templates/csi-controller-deploy.yaml

@@ -208,8 +208,8 @@ spec:
       hostAliases:
         {{- toYaml . | nindent 8 }}
       {{- end }}
-    {{- if .Values.priorityClassName }}
-      priorityClassName: {{ .Values.priorityClassName }}
+    {{- if .Values.controller.priorityClassName }}
+      priorityClassName: {{ .Values.controller.priorityClassName }}
     {{- end }}
     {{- with .Values.imagePullSecrets }}
       imagePullSecrets:

charts/cloudstack-csi/templates/csi-node-ds.yaml

@@ -174,8 +174,8 @@ spec:
       hostAliases:
         {{- toYaml . | nindent 8 }}
       {{- end }}
-    {{- if .Values.priorityClassName }}
-      priorityClassName: {{ .Values.priorityClassName }}
+    {{- if .Values.node.priorityClassName }}
+      priorityClassName: {{ .Values.node.priorityClassName }}
     {{- end }}
     {{- with .Values.imagePullSecrets }}
       imagePullSecrets:

charts/cloudstack-csi/values.yaml

@@ -66,6 +66,7 @@ controller:
       # maxSurge is the maximum number of pods that can be
       # created over the desired number of pods.
       maxSurge: 1
+  priorityClassName: ""
   podSecurityContext:
     runAsNonRoot: true
     runAsUser: 65532
@@ -118,7 +119,7 @@ node:
       hostPath:
         path: /etc/cacert
   dnsPolicy: ClusterFirstWithHostNet
-    
+  priorityClassName: ""
   podSecurityContext: {}
 
   securityContext: {}
@@ -208,8 +209,6 @@ secret:
       secret-key: ""
   hostMount: false
 
-priorityClassName: ""
-
 imagePullSecrets: []
 # - name: my-imagepull-secret
 

deploy/k8s/controller-deployment.yaml

@@ -6,7 +6,7 @@ metadata:
   name: cloudstack-csi-controller
   namespace: kube-system
 spec:
-  replicas: 3
+  replicas: 2
   strategy:
     type: RollingUpdate
     rollingUpdate:
@@ -21,6 +21,7 @@ spec:
         app.kubernetes.io/name: cloudstack-csi-controller
         app.kubernetes.io/part-of: cloudstack-csi-driver
     spec:
+      priorityClassName: system-cluster-critical
       serviceAccountName: cloudstack-csi-controller
       nodeSelector:
         kubernetes.io/os: linux

deploy/k8s/node-daemonset.yaml

@@ -15,6 +15,7 @@ spec:
         app.kubernetes.io/name: cloudstack-csi-node
         app.kubernetes.io/part-of: cloudstack-csi-driver
     spec:
+      priorityClassName: system-node-critical
       nodeSelector:
         kubernetes.io/os: linux
       tolerations: