Merge pull request #886 from andyzhangx/fix-runOnMaster2

fix: controller.runOnMaster chart config

Author: andyzhangx

charts/latest/csi-driver-smb-v0.0.0.tgz

@@ -24,19 +24,26 @@ spec:
 {{ toYaml .Values.podAnnotations | indent 8 }}
 {{- end }}
     spec:
+    # runOnControlPlane=true or runOnMaster=true only takes effect if affinity is not set
     {{- if contains (tpl "{{ .Values.controller.affinity }}" .) "nodeSelectorTerms" }}
       {{- with .Values.controller.affinity }}
       affinity:
       {{ toYaml . | indent 8 }}
       {{- end }}
-    {{- else if .Values.controller.runOnControlPlane}}
+    {{- else if or .Values.controller.runOnControlPlane .Values.controller.runOnMaster}}
       affinity:
         nodeAffinity:
           requiredDuringSchedulingIgnoredDuringExecution:
             nodeSelectorTerms:
               - matchExpressions:
+                {{- if .Values.controller.runOnControlPlane}}
                 - key: node-role.kubernetes.io/control-plane
                   operator: Exists
+                {{- end}}
+                {{- if .Values.controller.runOnMaster}}
+                - key: node-role.kubernetes.io/master
+                  operator: Exists
+                {{- end}}
     {{- end }}
       hostNetwork: true
       dnsPolicy: {{ .Values.controller.dnsPolicy }}

charts/latest/csi-driver-smb/templates/csi-smb-controller.yaml

@@ -24,19 +24,26 @@ spec:
 {{ toYaml .Values.podAnnotations | indent 8 }}
 {{- end }}
     spec:
+    # runOnControlPlane=true or runOnMaster=true only takes effect if affinity is not set
     {{- if contains (tpl "{{ .Values.controller.affinity }}" .) "nodeSelectorTerms" }}
       {{- with .Values.controller.affinity }}
       affinity:
       {{ toYaml . | indent 8 }}
       {{- end }}
-    {{- else if .Values.controller.runOnControlPlane}}
+    {{- else if or .Values.controller.runOnControlPlane .Values.controller.runOnMaster}}
       affinity:
         nodeAffinity:
           requiredDuringSchedulingIgnoredDuringExecution:
             nodeSelectorTerms:
               - matchExpressions:
+                {{- if .Values.controller.runOnControlPlane}}
                 - key: node-role.kubernetes.io/control-plane
                   operator: Exists
+                {{- end}}
+                {{- if .Values.controller.runOnMaster}}
+                - key: node-role.kubernetes.io/master
+                  operator: Exists
+                {{- end}}
     {{- end }}
       hostNetwork: true
       dnsPolicy: {{ .Values.controller.dnsPolicy }}