Added the Volumelimits for Powermax (#240)

Co-authored-by: Yamunadevi N Shanmugam <82038610+shanmydell@users.noreply.github.com>
dell · Sep 14, 2023 · e768b91 · e768b91
1 parent 3bf65dc
commit e768b91
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 7 deletions.
diff --git a/charts/container-storage-modules/values.yaml b/charts/container-storage-modules/values.yaml
@@ -117,7 +117,7 @@ csi-powermax:
       - endpoint: https://backup-1.unisphe.re:8443
   #    - endpoint: https://primary-2.unisphe.re:8443
   #    - endpoint: https://backup-2.unisphe.re:8443
-  version: "v2.7.0"
+  version: "v2.8.0"
   images:
     driverRepository: "dellemc"
   clusterPrefix: ABC

diff --git a/charts/csi-powermax/templates/node.yaml b/charts/csi-powermax/templates/node.yaml
@@ -1,4 +1,3 @@
-{{- if or (eq .Values.migration.enabled true) (eq .Values.openshift true) }}
 apiVersion: v1
 kind: ServiceAccount
 metadata:
@@ -14,6 +13,9 @@ metadata:
   name: {{ .Release.Name }}-node
   {{- end }}
 rules:
+  - apiGroups: [""]
+    resources: ["nodes"]
+    verbs: ["get", "list", "watch", "update", "patch"]
   {{- if eq .Values.openshift true }}
   - apiGroups: ["security.openshift.io"]
     resources: ["securitycontextconstraints"]
@@ -60,7 +62,6 @@ roleRef:
   {{- end }}
   apiGroup: rbac.authorization.k8s.io
 ---
-{{ end }}
 kind: DaemonSet
 apiVersion: apps/v1
 metadata:
@@ -81,9 +82,7 @@ spec:
       labels:
         app: {{ .Release.Name }}-node
     spec:
-      {{- if or (eq .Values.migration.enabled true) (eq .Values.openshift true) }}
       serviceAccountName: {{ .Release.Name }}-node
-      {{ end }}
       {{ if .Values.node.nodeSelector }}
       nodeSelector:
       {{- toYaml .Values.node.nodeSelector | nindent 8 }}
@@ -121,6 +120,8 @@ spec:
               value: unix://{{ .Values.kubeletConfigDir }}/plugins/powermax.emc.dell.com/csi_sock
             - name: X_CSI_MODE
               value: node
+            - name: X_CSI_MAX_VOLUMES_PER_NODE
+              value: "{{ .Values.maxPowerMaxVolumesPerNode }}"
             - name: X_CSI_PRIVATE_MOUNT_DIR
               value: "{{ .Values.kubeletConfigDir }}/plugins/powermax.emc.dell.com/disks"
             - name: X_CSI_MANAGED_ARRAYS
@@ -395,4 +396,4 @@ spec:
           secret:
             secretName: proxy-server-root-certificate
       {{ end }}
-      {{ end }}
+      {{ end }}
diff --git a/charts/csi-powermax/values.yaml b/charts/csi-powermax/values.yaml
@@ -49,7 +49,7 @@ global:
 
 # Current version of the driver
 # Don't modify this value as this value will be used by the install script
-version: "v2.7.0"
+version: "v2.8.0"
 
 images:
   # "driver" defines the container image, used for the driver container.
@@ -101,6 +101,13 @@ kubeletConfigDir: /var/lib/kubelet
 # Default value: ReadWriteOnceWithFSType
 fsGroupPolicy: ReadWriteOnceWithFSType
 
+# maxPowermaxVolumesPerNode: Specify default value for maximum number of volumes that controller can publish to the node.
+# If value is zero CO SHALL decide how many volumes of this type can be published by the controller to the node.
+# This limit is applicable to all the nodes in the cluster for which node label 'max-Powermax-volumes-per-node' is not set.
+# Allowed values: n, where n >= 0
+# Default value: 0
+maxPowerMaxVolumesPerNode: 0
+
 # controller: configure controller specific parameters
 controller:
   # controllerCount: Define the number of PowerMax controller nodes