CSM CSM 1.16.3 updates (#613)

csmbot · root · francis-nijay · web-flow · commit 757512ea93e6 · 2026-03-17T17:18:54.000-04:00
Co-authored-by: root &lt;root@master-1-1If4zxKsnhG0a.domain&gt;
Co-authored-by: nijayf &lt;francis.nijay@dell.com&gt;
diff --git a/Dockerfile b/Dockerfile
@@ -12,7 +12,7 @@
 #
 ARG GOIMAGE
 ARG BASEIMAGE
-ARG VERSION="2.16.1"
+ARG VERSION="2.16.2"
 
 # Stage to build the driver
 FROM $GOIMAGE as builder
diff --git a/README.md b/README.md
@@ -44,4 +44,3 @@ For a complete list of dependencies, please visit [Prerequisites](https://dell.g
 
 ## Documentation
 For more detailed information on the driver, please refer to [Container Storage Modules documentation](https://dell.github.io/csm-docs/).
-
diff --git a/csireverseproxy/go.mod b/csireverseproxy/go.mod
@@ -4,7 +4,7 @@ go 1.25.0
 
 require (
 	github.com/dell/csmlog v1.0.0
-	github.com/dell/gopowermax/v2 v2.12.1
+	github.com/dell/gopowermax/v2 v2.12.2
 	github.com/fsnotify/fsnotify v1.9.0
 	github.com/gorilla/mux v1.8.1
 	github.com/kubernetes-csi/csi-lib-utils v0.23.0
diff --git a/csireverseproxy/go.sum b/csireverseproxy/go.sum
@@ -1,7 +1,7 @@
 github.com/dell/csmlog v1.0.0 h1:EzW+nMJBD0QTNP88OoaAJUOMVilS9cWkO248BTcJt/4=
 github.com/dell/csmlog v1.0.0/go.mod h1:7rBzSv9xF5t233+J+9vkStjFsmyYO3L/B9tDTy3+9ZU=
-github.com/dell/gopowermax/v2 v2.12.1 h1:+tiLrnmvucvaN5mqW7oi+wSGBEkhjFBwi3X10zLVXcE=
-github.com/dell/gopowermax/v2 v2.12.1/go.mod h1:f/wnjHWZ6H4nJE/w2VTu547yBvZxMQjlz3oc2twb2Fk=
+github.com/dell/gopowermax/v2 v2.12.2 h1:b50XR/a67H8zJHbKVnRqqvtrdTquvawXT4r1IqzwQc0=
+github.com/dell/gopowermax/v2 v2.12.2/go.mod h1:f/wnjHWZ6H4nJE/w2VTu547yBvZxMQjlz3oc2twb2Fk=
 github.com/Masterminds/semver/v3 v3.4.0 h1:Zog+i5UMtVoCU8oKka5P7i9q9HgrJeGzI9SA1Xbatp0=
 github.com/Masterminds/semver/v3 v3.4.0/go.mod h1:4V+yj/TJE1HU9XfppCwVMZq3I84lprf4nC11bSS5beM=
 github.com/davecgh/go-spew v1.1.0/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
diff --git a/csireverseproxy/pkg/proxy/proxy.go b/csireverseproxy/pkg/proxy/proxy.go
@@ -424,6 +424,7 @@ func (revProxy *Proxy) updateConfig(proxyConfig config.ProxyConfig) {
 func (revProxy *Proxy) GetRouter() http.Handler {
 	router := mux.NewRouter()
 	router.Use(revProxy.symIDMiddleware)
+	router.Path(utils.PrefixV1+"/systems/{symid}/volumes").Queries("limit", "{limit}").HandlerFunc(revProxy.ServeReverseProxy)
 	router.Path(utils.PrefixV1 + "/systems/{symid}/volumes").HandlerFunc(revProxy.GetVolumes)
 	router.Path(utils.PrefixV1 + "/systems/{symid}/port-groups").HandlerFunc(revProxy.GetPortGroups)
 	router.Path(utils.PrefixV1 + "/systems/{symid}/storage-groups").HandlerFunc(revProxy.ServeReverseProxy)
diff --git a/dell-csi-helm-installer/README.md b/dell-csi-helm-installer/README.md
@@ -36,7 +36,7 @@ This project provides the following capabilitites, each one is discussed in deta
 
 
 Most of these usages require the creation/specification of a values file. These files specify configuration settings that are passed into the driver and configure it for use. To create one of these files, the following steps should be followed:
-1. Download a template file for the driver to a new location, naming this new file is at the users discretion. The template files are always found at `https://github.com/dell/helm-charts/raw/csi-powermax-2.16.1/charts/csi-powermax/values.yaml`
+1. Download a template file for the driver to a new location, naming this new file is at the users discretion. The template files are always found at `https://github.com/dell/helm-charts/raw/csi-powermax-2.16.2/charts/csi-powermax/values.yaml`
 2. Edit the file such that it contains the proper configuration settings for the specific environment. These files are yaml formatted so maintaining the file structure is important.
 
 For example, to create a values file for the PowerMax driver the following steps can be executed
@@ -45,7 +45,7 @@ For example, to create a values file for the PowerMax driver the following steps
 cd dell-csi-helm-installer
 
 # Download the template file
-wget -O my-powermax-settings.yaml https://github.com/dell/helm-charts/raw/csi-powermax-2.16.1/charts/csi-powermax/values.yaml
+wget -O my-powermax-settings.yaml https://github.com/dell/helm-charts/raw/csi-powermax-2.16.2/charts/csi-powermax/values.yaml
 
 # edit the newly created values file
 vi my-powermax-settings.yaml
diff --git a/dell-csi-helm-installer/csi-install.sh b/dell-csi-helm-installer/csi-install.sh
@@ -20,7 +20,7 @@ PROG="${0}"
 NODE_VERIFY=1
 VERIFY=1
 MODE="install"
-DEFAULT_DRIVER_VERSION="v2.16.1"
+DEFAULT_DRIVER_VERSION="v2.16.2"
 WATCHLIST=""
 
 #
@@ -49,7 +49,7 @@ function usage() {
   exit 0
 }
 
-DRIVERVERSION="csi-powermax-2.16.1"
+DRIVERVERSION="csi-powermax-2.16.2"
 
 while getopts ":h-:" optchar; do
   case "${optchar}" in
diff --git a/dell-csi-helm-installer/csi-offline-bundle.md b/dell-csi-helm-installer/csi-offline-bundle.md
@@ -80,19 +80,19 @@ For example, here is the output of a request to build an offline bundle for the
 
    quay.io/dell/container-storage-modules/csi-isilon:v2.16.0
    quay.io/dell/container-storage-modules/csi-metadata-retriever:v1.13.0
-   quay.io/dell/container-storage-modules/csipowermax-reverseproxy:v2.15.1
-   quay.io/dell/container-storage-modules/csi-powermax:v2.16.1
+   quay.io/dell/container-storage-modules/csipowermax-reverseproxy:v2.15.2
+   quay.io/dell/container-storage-modules/csi-powermax:v2.16.2
    quay.io/dell/container-storage-modules/csi-powerstore:v2.16.0
    quay.io/dell/container-storage-modules/csi-unity:v2.16.0
-   quay.io/dell/container-storage-modules/csi-vxflexos:v2.16.0
+   quay.io/dell/container-storage-modules/csi-vxflexos:v2.16.1
    quay.io/dell/container-storage-modules/csm-authorization-sidecar:v2.4.0
    quay.io/dell/container-storage-modules/csm-metrics-powerflex:v1.14.0
    quay.io/dell/container-storage-modules/csm-metrics-powerscale:v1.11.0
    quay.io/dell/container-storage-modules/csm-topology:v1.12.0
    quay.io/dell/container-storage-modules/dell-csi-replicator:v1.14.0
    quay.io/dell/container-storage-modules/dell-replication-controller:v1.14.0
    quay.io/dell/container-storage-modules/sdc:4.5.2.1
-   quay.io/dell/container-storage-modules/dell-csm-operator:v1.11.0
+   quay.io/dell/container-storage-modules/dell-csm-operator:v1.11.3
    registry.redhat.io/openshift4/ose-kube-rbac-proxy-rhel9:v4.16.0-202409051837.p0.g8ea2c99.assembly.stream.el9
    nginxinc/nginx-unprivileged:1.29
    otel/opentelemetry-collector:0.142.0
diff --git a/dell-csi-helm-installer/csi-offline-bundle.sh b/dell-csi-helm-installer/csi-offline-bundle.sh
@@ -237,7 +237,7 @@ DRIVER="csi-powermax"
 SCRIPTDIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" >/dev/null 2>&1 && pwd)"
 REPODIR="$( dirname "${SCRIPTDIR}" )"
 
-DRIVERVERSION="csi-powermax-2.16.1"
+DRIVERVERSION="csi-powermax-2.16.2"
 
 while getopts "cprv:h" opt; do
   case $opt in
diff --git a/go.mod b/go.mod
@@ -13,7 +13,7 @@ require (
 	github.com/dell/gofsutil v1.21.0
 	github.com/dell/goiscsi v1.14.0
 	github.com/dell/gonvme v1.13.0
-	github.com/dell/gopowermax/v2 v2.12.1
+	github.com/dell/gopowermax/v2 v2.12.2
 	github.com/akutz/goof v0.1.2
 	github.com/container-storage-interface/spec v1.6.0
 	github.com/coreos/go-systemd/v22 v22.6.0
diff --git a/go.sum b/go.sum
@@ -42,8 +42,8 @@ github.com/dell/goiscsi v1.14.0 h1:kNDqOlpJ3cLSJh7Hfyn/Kz/FMCKHzV0s/xx4EqnelFw=
 github.com/dell/goiscsi v1.14.0/go.mod h1:SCSC8dJCqTosU7SspaoLv6ICTKNEz08rt/I8nZ3+ptc=
 github.com/dell/gonvme v1.13.0 h1:j8A1BzYA48gelih3xWd/J6LQ71CbC8Lbdyv0jG8uUNU=
 github.com/dell/gonvme v1.13.0/go.mod h1:L5K7V4JZTf12m3k2wdwKwP+/eA6pr8DvlCsJU1QTGOQ=
-github.com/dell/gopowermax/v2 v2.12.1 h1:+tiLrnmvucvaN5mqW7oi+wSGBEkhjFBwi3X10zLVXcE=
-github.com/dell/gopowermax/v2 v2.12.1/go.mod h1:f/wnjHWZ6H4nJE/w2VTu547yBvZxMQjlz3oc2twb2Fk=
+github.com/dell/gopowermax/v2 v2.12.2 h1:b50XR/a67H8zJHbKVnRqqvtrdTquvawXT4r1IqzwQc0=
+github.com/dell/gopowermax/v2 v2.12.2/go.mod h1:f/wnjHWZ6H4nJE/w2VTu547yBvZxMQjlz3oc2twb2Fk=
 github.com/Azure/go-ansiterm v0.0.0-20210608223527-2377c96fe795/go.mod h1:LmzpDX56iTiv29bbRTIsUNlaFfuhWRQBWjQdVyAevI8=
 github.com/Azure/go-ansiterm v0.0.0-20210617225240-d185dfc1b5a1/go.mod h1:xomTg63KZ2rFqZQzSB4Vz2SUXa1BpHTVz9L5PTmPC4E=
 github.com/Azure/go-autorest v14.2.0+incompatible/go.mod h1:r+4oMnoxhatjLLJ6zxSWATqVooLgysK6ZNox3g/xq24=
diff --git a/pkg/symmetrix/mocks/pmaxclient.go b/pkg/symmetrix/mocks/pmaxclient.go
diff --git a/service/controller.go b/service/controller.go
@@ -3594,7 +3594,8 @@ func (s *service) SelectOrCreateFCPGForHost(ctx context.Context, symID string, h
 		}
 	}
 
-	if version >= 103 {
+	isV4 := s.isV4OrAbove(ctx, symID, pmaxClient)
+	if version >= 103 && isV4 {
 		log.Debug("API version is greater than or equal to 103. Using enhanced API")
 		portGroupList, err := pmaxClient.GetPortGroupListByType(ctx, symID, "fibre")
 		if err != nil {
diff --git a/service/deletion_worker.go b/service/deletion_worker.go
diff --git a/service/deletion_worker_test.go b/service/deletion_worker_test.go
diff --git a/service/node_connectivity_checker.go b/service/node_connectivity_checker.go
diff --git a/service/snap.go b/service/snap.go

Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,7 @@`
`12`	`12`	`#`
`13`	`13`	`ARG GOIMAGE`
`14`	`14`	`ARG BASEIMAGE`
`15`		`-ARG VERSION="2.16.1"`
	`15`	`+ARG VERSION="2.16.2"`
`16`	`16`
`17`	`17`	`# Stage to build the driver`
`18`	`18`	`FROM $GOIMAGE as builder`
Original file line number	Diff line number	Diff line change
`@@ -44,4 +44,3 @@ For a complete list of dependencies, please visit [Prerequisites](https://dell.g`
`44`	`44`
`45`	`45`	`## Documentation`
`46`	`46`	`For more detailed information on the driver, please refer to [Container Storage Modules documentation](https://dell.github.io/csm-docs/).`
`47`		`-`
Original file line number	Diff line number	Diff line change
`@@ -20,7 +20,7 @@ PROG="${0}"`
`20`	`20`	`NODE_VERIFY=1`
`21`	`21`	`VERIFY=1`
`22`	`22`	`MODE="install"`
`23`		`-DEFAULT_DRIVER_VERSION="v2.16.1"`
	`23`	`+DEFAULT_DRIVER_VERSION="v2.16.2"`
`24`	`24`	`WATCHLIST=""`
`25`	`25`
`26`	`26`	`#`
`@@ -49,7 +49,7 @@ function usage() {`
`49`	`49`	`exit 0`
`50`	`50`	`}`
`51`	`51`
`52`		`-DRIVERVERSION="csi-powermax-2.16.1"`
	`52`	`+DRIVERVERSION="csi-powermax-2.16.2"`
`53`	`53`
`54`	`54`	`while getopts ":h-:" optchar; do`
`55`	`55`	`case "${optchar}" in`