Merge pull request #82 from sathieu/maxVolumesPerNode

Allow to set MAX_VOLUMES_PER_NODE
vsphere-tmm · May 28, 2024 · c4fda6a · c4fda6a
2 parents 29c6fc0 + 7ce828d
commit c4fda6a
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 3 deletions.
diff --git a/charts/vsphere-csi/Chart.yaml b/charts/vsphere-csi/Chart.yaml
@@ -35,4 +35,4 @@ maintainers:
 name: vsphere-csi
 sources:
   - https://github.com/kubernetes-sigs/vsphere-csi-driver
-version: 3.3.3
+version: 3.4.0
diff --git a/charts/vsphere-csi/README.md b/charts/vsphere-csi/README.md
@@ -541,6 +541,7 @@ The command removes all the Kubernetes components associated with the chart and
 | `node.updateStrategy.type`                                    | node statefulset strategy type                                                                            | `RollingUpdate`                                                                                                                                                                                                                                                                |
 | `node.priorityClassName`                                      | node pods' priorityClassName                                                                              | `system-node-critical`                                                                                                                                                                                                                                                         |
 | `node.lifecycleHooks`                                         | for the node container(s) to automate configuration before or after startup                               | `{}`                                                                                                                                                                                                                                                                           |
+| `node.maxVolumesPerNode`                                      | Maximum number of volumes that controller can publish to the node. If zero Kubernetes decide              | `59`                                                                                                                                                                                                                                                                           |
 | `node.extraEnvVars`                                           | Array with extra environment variables to add to node nodes                                               | `[]`                                                                                                                                                                                                                                                                           |
 | `node.extraEnvVarsCM`                                         | Name of existing ConfigMap containing extra env vars for node nodes                                       | `nil`                                                                                                                                                                                                                                                                          |
 | `node.extraEnvVarsSecret`                                     | Name of existing Secret containing extra env vars for node nodes                                          | `nil`                                                                                                                                                                                                                                                                          |
@@ -660,6 +661,7 @@ The command removes all the Kubernetes components associated with the chart and
 | `winnode.updateStrategy.type`                                 | node statefulset strategy type                                                                            | `RollingUpdate`                                                                                                                                                                                                                                                                |
 | `winnode.priorityClassName`                                   | node pods' priorityClassName                                                                              | `system-node-critical`                                                                                                                                                                                                                                                         |
 | `winnode.lifecycleHooks`                                      | for the node container(s) to automate configuration before or after startup                               | `{}`                                                                                                                                                                                                                                                                           |
+| `winnode.maxVolumesPerNode`                                   | Maximum number of volumes that controller can publish to the node. If zero Kubernetes decide              | `59`                                                                                                                                                                                                                                                                           |
 | `winnode.extraEnvVars`                                        | Array with extra environment variables to add to node nodes                                               | `[]`                                                                                                                                                                                                                                                                           |
 | `winnode.extraEnvVarsCM`                                      | Name of existing ConfigMap containing extra env vars for node nodes                                       | `nil`                                                                                                                                                                                                                                                                          |
 | `winnode.extraEnvVarsSecret`                                  | Name of existing Secret containing extra env vars for node nodes                                          | `nil`                                                                                                                                                                                                                                                                          |

diff --git a/charts/vsphere-csi/templates/node/daemonset-linux.yaml b/charts/vsphere-csi/templates/node/daemonset-linux.yaml
@@ -150,7 +150,7 @@ spec:
             - name: LOGGER_LEVEL
               value: "PRODUCTION" # Options: DEVELOPMENT, PRODUCTION
             - name: MAX_VOLUMES_PER_NODE
-              value: "59"     # Maximum number of volumes that controller can publish to the node. If value is not set or zero Kubernetes decide how many volumes can be published by the controller to the node.
+              value: {{ .Values.node.maxVolumesPerNode | quote }}
             - name: GODEBUG
               value: x509sha1=1
             - name: CSI_NAMESPACE

diff --git a/charts/vsphere-csi/templates/node/daemonset-win.yaml b/charts/vsphere-csi/templates/node/daemonset-win.yaml
@@ -152,7 +152,7 @@ spec:
             - name: LOGGER_LEVEL
               value: "PRODUCTION" # Options: DEVELOPMENT, PRODUCTION
             - name: MAX_VOLUMES_PER_NODE
-              value: "59"     # Maximum number of volumes that controller can publish to the node. If value is not set or zero Kubernetes decide how many volumes can be published by the controller to the node.
+              value: {{ .Values.winnode.maxVolumesPerNode | quote }}
             - name: CSI_NAMESPACE
               valueFrom:
                 fieldRef:

diff --git a/charts/vsphere-csi/values.yaml b/charts/vsphere-csi/values.yaml
@@ -1887,6 +1887,8 @@ node:
   ## @param node.lifecycleHooks for the node container(s) to automate configuration before or after startup
   ##
   lifecycleHooks: {}
+  ## @param node.maxVolumesPerNode Maximum number of volumes that controller can publish to the node. If zero Kubernetes decide
+  maxVolumesPerNode: 59
   ## @param node.extraEnvVars Array with extra environment variables to add to node nodes
   ## e.g:
   ## extraEnvVars:
@@ -2376,6 +2378,8 @@ winnode:
   ## @param winnode.lifecycleHooks for the node container(s) to automate configuration before or after startup
   ##
   lifecycleHooks: {}
+  ## @param winnode.maxVolumesPerNode Maximum number of volumes that controller can publish to the node. If zero Kubernetes decide
+  maxVolumesPerNode: 59
   ## @param winnode.extraEnvVars Array with extra environment variables to add to node nodes
   ## e.g:
   ## extraEnvVars: