From b181ff66ca05b99e97480fdceb23d98fb0c9166a Mon Sep 17 00:00:00 2001 From: Surya Gupta <109594002+suryagupta4@users.noreply.github.com> Date: Mon, 16 Oct 2023 16:54:26 +0530 Subject: [PATCH 01/26] [charts/csi-powerstore] Add PowerStore v2.9.0 support & sidecar version update (#303) * powerstore v2.9.0 support * sidecar version update --- charts/container-storage-modules/Chart.yaml | 2 +- charts/container-storage-modules/values.yaml | 2 +- charts/csi-powerstore/Chart.yaml | 4 ++-- charts/csi-powerstore/templates/_helpers.tpl | 12 ++++++------ charts/csi-powerstore/values.yaml | 2 +- 5 files changed, 11 insertions(+), 11 deletions(-) diff --git a/charts/container-storage-modules/Chart.yaml b/charts/container-storage-modules/Chart.yaml index c2305900..12608ed2 100644 --- a/charts/container-storage-modules/Chart.yaml +++ b/charts/container-storage-modules/Chart.yaml @@ -40,7 +40,7 @@ appVersion: "1.1.0" dependencies: - name: csi-powerstore - version: 2.8.0 + version: 2.9.0 repository: https://dell.github.io/helm-charts condition: csi-powerstore.enabled diff --git a/charts/container-storage-modules/values.yaml b/charts/container-storage-modules/values.yaml index 55b61010..acbe11d1 100644 --- a/charts/container-storage-modules/values.yaml +++ b/charts/container-storage-modules/values.yaml @@ -20,7 +20,7 @@ ######################## csi-powerstore: enabled: false - version: "v2.8.0" + version: "v2.9.0" images: driverRepository: dellemc ## Controller ATTRIBUTES diff --git a/charts/csi-powerstore/Chart.yaml b/charts/csi-powerstore/Chart.yaml index d7815a98..2d4d5a9e 100644 --- a/charts/csi-powerstore/Chart.yaml +++ b/charts/csi-powerstore/Chart.yaml @@ -14,9 +14,9 @@ # # apiVersion: v2 -appVersion: "2.8.0" +appVersion: "2.9.0" name: csi-powerstore -version: 2.8.0 +version: 2.9.0 description: | PowerStore CSI (Container Storage Interface) driver Kubernetes integration. This chart includes everything required to provision via CSI as diff --git a/charts/csi-powerstore/templates/_helpers.tpl b/charts/csi-powerstore/templates/_helpers.tpl index 300585d6..a16f8d5e 100644 --- a/charts/csi-powerstore/templates/_helpers.tpl +++ b/charts/csi-powerstore/templates/_helpers.tpl @@ -4,7 +4,7 @@ Return the appropriate sidecar images based on k8s version {{- define "csi-powerstore.attacherImage" -}} {{- if eq .Capabilities.KubeVersion.Major "1" }} {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "24") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-attacher:v4.3.0" -}} + {{- print "registry.k8s.io/sig-storage/csi-attacher:v4.4.0" -}} {{- end -}} {{- end -}} {{- end -}} @@ -12,7 +12,7 @@ Return the appropriate sidecar images based on k8s version {{- define "csi-powerstore.provisionerImage" -}} {{- if eq .Capabilities.KubeVersion.Major "1" }} {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "24") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-provisioner:v3.5.0" -}} + {{- print "registry.k8s.io/sig-storage/csi-provisioner:v3.6.0" -}} {{- end -}} {{- end -}} {{- end -}} @@ -20,7 +20,7 @@ Return the appropriate sidecar images based on k8s version {{- define "csi-powerstore.snapshotterImage" -}} {{- if eq .Capabilities.KubeVersion.Major "1" }} {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "24") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-snapshotter:v6.2.2" -}} + {{- print "registry.k8s.io/sig-storage/csi-snapshotter:v6.3.0" -}} {{- end -}} {{- end -}} {{- end -}} @@ -28,7 +28,7 @@ Return the appropriate sidecar images based on k8s version {{- define "csi-powerstore.resizerImage" -}} {{- if eq .Capabilities.KubeVersion.Major "1" }} {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "24") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-resizer:v1.8.0" -}} + {{- print "registry.k8s.io/sig-storage/csi-resizer:v1.9.0" -}} {{- end -}} {{- end -}} {{- end -}} @@ -36,7 +36,7 @@ Return the appropriate sidecar images based on k8s version {{- define "csi-powerstore.registrarImage" -}} {{- if eq .Capabilities.KubeVersion.Major "1" }} {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "24") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.8.0" -}} + {{- print "registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.9.0" -}} {{- end -}} {{- end -}} {{- end -}} @@ -44,7 +44,7 @@ Return the appropriate sidecar images based on k8s version {{- define "csi-powerstore.healthmonitorImage" -}} {{- if eq .Capabilities.KubeVersion.Major "1" }} {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "24") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.9.0" -}} + {{- print "registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.10.0" -}} {{- end -}} {{- end -}} {{- end -}} diff --git a/charts/csi-powerstore/values.yaml b/charts/csi-powerstore/values.yaml index e025d014..2159d901 100644 --- a/charts/csi-powerstore/values.yaml +++ b/charts/csi-powerstore/values.yaml @@ -23,7 +23,7 @@ driverName: "csi-powerstore.dellemc.com" # Driver version required to pull the latest driver image -version: "v2.8.0" +version: "v2.9.0" # Specify kubelet config dir path. # Ensure that the config.yaml file is present at this path. From 0c2adcc5e76406428339bffbcf5cf45ef2c7d9e3 Mon Sep 17 00:00:00 2001 From: VamsiSiddu-7 <103578883+VamsiSiddu-7@users.noreply.github.com> Date: Mon, 16 Oct 2023 17:15:53 +0530 Subject: [PATCH 02/26] csi-powerflex v2.9.0 changes. (#308) --- charts/container-storage-modules/Chart.yaml | 2 +- charts/container-storage-modules/values.yaml | 2 +- charts/csi-vxflexos/Chart.yaml | 4 ++-- charts/csi-vxflexos/templates/_helpers.tpl | 12 ++++++------ charts/csi-vxflexos/values.yaml | 2 +- 5 files changed, 11 insertions(+), 11 deletions(-) diff --git a/charts/container-storage-modules/Chart.yaml b/charts/container-storage-modules/Chart.yaml index 12608ed2..3063b8bf 100644 --- a/charts/container-storage-modules/Chart.yaml +++ b/charts/container-storage-modules/Chart.yaml @@ -55,7 +55,7 @@ dependencies: condition: csi-isilon.enabled - name: csi-vxflexos - version: 2.8.0 + version: 2.9.0 repository: https://dell.github.io/helm-charts condition: csi-vxflexos.enabled diff --git a/charts/container-storage-modules/values.yaml b/charts/container-storage-modules/values.yaml index acbe11d1..7e781adc 100644 --- a/charts/container-storage-modules/values.yaml +++ b/charts/container-storage-modules/values.yaml @@ -247,7 +247,7 @@ csi-isilon: ######################## csi-vxflexos: enabled: false - version: v2.8.0 + version: v2.9.0 images: driverRepository: dellemc powerflexSdc: dellemc/sdc:3.6.1 diff --git a/charts/csi-vxflexos/Chart.yaml b/charts/csi-vxflexos/Chart.yaml index adc4da24..9857da49 100644 --- a/charts/csi-vxflexos/Chart.yaml +++ b/charts/csi-vxflexos/Chart.yaml @@ -1,7 +1,7 @@ apiVersion: v2 -appVersion: "2.8.0" +appVersion: "2.9.0" name: csi-vxflexos -version: "2.8.0" +version: "2.9.0" description: | VxFlex OS CSI (Container Storage Interface) driver Kubernetes integration. This chart includes everything required to provision via CSI as diff --git a/charts/csi-vxflexos/templates/_helpers.tpl b/charts/csi-vxflexos/templates/_helpers.tpl index 621f38fc..e21cd0b0 100644 --- a/charts/csi-vxflexos/templates/_helpers.tpl +++ b/charts/csi-vxflexos/templates/_helpers.tpl @@ -4,7 +4,7 @@ Return the appropriate sidecar images based on k8s version {{- define "csi-vxflexos.attacherImage" -}} {{- if eq .Capabilities.KubeVersion.Major "1" }} {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "21") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-attacher:v4.3.0" -}} + {{- print "registry.k8s.io/sig-storage/csi-attacher:v4.4.0" -}} {{- end -}} {{- end -}} {{- end -}} @@ -12,7 +12,7 @@ Return the appropriate sidecar images based on k8s version {{- define "csi-vxflexos.provisionerImage" -}} {{- if eq .Capabilities.KubeVersion.Major "1" }} {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "21") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-provisioner:v3.5.0" -}} + {{- print "registry.k8s.io/sig-storage/csi-provisioner:v3.6.0" -}} {{- end -}} {{- end -}} {{- end -}} @@ -20,7 +20,7 @@ Return the appropriate sidecar images based on k8s version {{- define "csi-vxflexos.snapshotterImage" -}} {{- if eq .Capabilities.KubeVersion.Major "1" }} {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "21") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-snapshotter:v6.2.2" -}} + {{- print "registry.k8s.io/sig-storage/csi-snapshotter:v6.3.0" -}} {{- end -}} {{- end -}} {{- end -}} @@ -28,7 +28,7 @@ Return the appropriate sidecar images based on k8s version {{- define "csi-vxflexos.resizerImage" -}} {{- if eq .Capabilities.KubeVersion.Major "1" }} {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "21") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-resizer:v1.8.0" -}} + {{- print "registry.k8s.io/sig-storage/csi-resizer:v1.9.0" -}} {{- end -}} {{- end -}} {{- end -}} @@ -36,7 +36,7 @@ Return the appropriate sidecar images based on k8s version {{- define "csi-vxflexos.registrarImage" -}} {{- if eq .Capabilities.KubeVersion.Major "1" }} {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "21") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.8.0" -}} + {{- print "registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.9.0" -}} {{- end -}} {{- end -}} {{- end -}} @@ -44,7 +44,7 @@ Return the appropriate sidecar images based on k8s version {{- define "csi-vxflexos.healthmonitorImage" -}} {{- if eq .Capabilities.KubeVersion.Major "1" }} {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "21") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.9.0" -}} + {{- print "registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.10.0" -}} {{- end -}} {{- end -}} {{- end -}} diff --git a/charts/csi-vxflexos/values.yaml b/charts/csi-vxflexos/values.yaml index 6c0052d2..dd890f46 100644 --- a/charts/csi-vxflexos/values.yaml +++ b/charts/csi-vxflexos/values.yaml @@ -3,7 +3,7 @@ # "version" is used to verify the values file matches driver version # Not recommend to change -version: v2.8.0 +version: v2.9.0 images: # "driver" defines the container image, used for the driver container. From 3562928e57fd416860fb5ad1dbfc545813a438cc Mon Sep 17 00:00:00 2001 From: panigs7 <92028646+panigs7@users.noreply.github.com> Date: Thu, 26 Oct 2023 18:36:14 +0530 Subject: [PATCH 03/26] Bump up observability version for CSM 1.9 (#314) * Bump-up observability version for CSM 1.9 * add patch verb to powerscale * Add codeowners --- .github/CODEOWNERS | 8 +++++++- charts/container-storage-modules/Chart.yaml | 2 +- charts/csi-isilon/templates/controller.yaml | 2 +- charts/karavi-observability/Chart.yaml | 4 ++-- charts/karavi-observability/values.yaml | 10 +++++----- 5 files changed, 16 insertions(+), 10 deletions(-) diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS index 62dc61ae..e89989ee 100644 --- a/.github/CODEOWNERS +++ b/.github/CODEOWNERS @@ -13,7 +13,9 @@ # Alik Saring (alikdell) # Aaron Tye (atye) # Chaganti Rajitha (chaganti-rajitha) +# Chiman Jain (chimanjain) # Florian Coulombel (coulof) +# Harish H (HarishH-DELL) # Meghana G M (meggm) # Niranjan N (niranjan-n1) # Shayna Finocchiaro (shaynafinocchiaro) @@ -25,6 +27,10 @@ # Yiming Bao (baoy1) # Yian Zong (YianZong) # Forrest Xia (forrestxia) +# Nitesh Rewatkar (nitesh3108) +# Rajendra Indukuri (rajendraindukuri) +# Shefali Malhotra (shefali-malhotra) +# Spandita Panigrahi (panigs7) # for all files: -* @gallacher @tdawe @alikdell @atye @hoppea2 @chaganti-rajitha @coulof @meggm @niranjan-n1 @shaynafinocchiaro @sharmilarama @tssushma @taohe1012 @P-Cao @shanmydell @baoy1 @YianZong @forrestxia +* @gallacher @tdawe @alikdell @atye @hoppea2 @chaganti-rajitha @coulof @meggm @niranjan-n1 @shaynafinocchiaro @sharmilarama @tssushma @taohe1012 @P-Cao @shanmydell @baoy1 @YianZong @forrestxia @nitesh3108 @rajendraindukuri @shefali-malhotra @panigs7 @chimanjain @HarishH-DELL diff --git a/charts/container-storage-modules/Chart.yaml b/charts/container-storage-modules/Chart.yaml index 3063b8bf..24e8c698 100644 --- a/charts/container-storage-modules/Chart.yaml +++ b/charts/container-storage-modules/Chart.yaml @@ -70,7 +70,7 @@ dependencies: condition: csm-replication.enabled - name: karavi-observability - version: 1.6.0 + version: 1.7.0 repository: https://dell.github.io/helm-charts condition: karavi-observability.enabled diff --git a/charts/csi-isilon/templates/controller.yaml b/charts/csi-isilon/templates/controller.yaml index 26479f3b..4c8dc222 100644 --- a/charts/csi-isilon/templates/controller.yaml +++ b/charts/csi-isilon/templates/controller.yaml @@ -72,7 +72,7 @@ rules: verbs: ["get", "list", "watch", "update"] - apiGroups: ["snapshot.storage.k8s.io"] resources: ["volumesnapshots/status"] - verbs: ["update"] + verbs: ["update", "patch"] - apiGroups: ["snapshot.storage.k8s.io"] resources: ["volumesnapshotcontents/status"] verbs: ["update"] diff --git a/charts/karavi-observability/Chart.yaml b/charts/karavi-observability/Chart.yaml index d15b0f5e..78d715f6 100644 --- a/charts/karavi-observability/Chart.yaml +++ b/charts/karavi-observability/Chart.yaml @@ -1,9 +1,9 @@ apiVersion: v2 -appVersion: "1.6.0" +appVersion: "1.7.0" name: karavi-observability description: CSM for Observability is part of the [Container Storage Modules](https://github.com/dell/csm) open source suite of Kubernetes storage enablers for Dell EMC storage products. CSM for Observability provides Kubernetes administrators with visibility into metrics and topology data related to containerized storage. type: application -version: 1.6.0 +version: 1.7.0 dependencies: - name: cert-manager version: 1.10.0 diff --git a/charts/karavi-observability/values.yaml b/charts/karavi-observability/values.yaml index c47aecbc..236369cf 100644 --- a/charts/karavi-observability/values.yaml +++ b/charts/karavi-observability/values.yaml @@ -1,5 +1,5 @@ karaviTopology: - image: dellemc/csm-topology:v1.6.0 + image: dellemc/csm-topology:v1.7.0 enabled: true # comma separated list of provisioner names (ex: csi-vxflexos.dellemc.com) provisionerNames: csi-vxflexos.dellemc.com,csi-powerstore.dellemc.com,csi-isilon.dellemc.com,csi-powermax.dellemc.com @@ -13,7 +13,7 @@ karaviTopology: probability: 0.0 karaviMetricsPowerflex: - image: dellemc/csm-metrics-powerflex:v1.6.0 + image: dellemc/csm-metrics-powerflex:v1.7.0 enabled: true collectorAddr: otel-collector:55680 # comma separated list of provisioner names (ex: csi-vxflexos.dellemc.com) @@ -53,7 +53,7 @@ karaviMetricsPowerflex: skipCertificateValidation: true karaviMetricsPowerstore: - image: dellemc/csm-metrics-powerstore:v1.6.0 + image: dellemc/csm-metrics-powerstore:v1.7.0 enabled: true collectorAddr: otel-collector:55680 # comma separated list of provisioner names (ex: csi-powerstore.dellemc.com) @@ -79,7 +79,7 @@ karaviMetricsPowerstore: probability: 0.0 karaviMetricsPowerscale: - image: dellemc/csm-metrics-powerscale:v1.3.0 + image: dellemc/csm-metrics-powerscale:v1.4.0 enabled: true collectorAddr: otel-collector:55680 # comma separated list of provisioner names (ex: csi-isilon.dellemc.com) @@ -129,7 +129,7 @@ karaviMetricsPowerscale: skipCertificateValidation: true karaviMetricsPowermax: - image: dellemc/csm-metrics-powermax:v1.1.0 + image: dellemc/csm-metrics-powermax:v1.2.0 enabled: true collectorAddr: otel-collector:55680 # comma separated list of provisioner names (ex: csi-powermax.dellemc.com) From 0e1c79f28b2f313b597219afd764ab374f295607 Mon Sep 17 00:00:00 2001 From: VamsiSiddu-7 <103578883+VamsiSiddu-7@users.noreply.github.com> Date: Sat, 28 Oct 2023 10:31:27 +0530 Subject: [PATCH 04/26] update attacher sidecar for powerflex and powerstore. (#315) --- charts/container-storage-modules/values.yaml | 2 +- charts/csi-powerstore/templates/_helpers.tpl | 2 +- charts/csi-vxflexos/templates/_helpers.tpl | 2 +- charts/csi-vxflexos/values.yaml | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/charts/container-storage-modules/values.yaml b/charts/container-storage-modules/values.yaml index 7e781adc..e9dbdbbe 100644 --- a/charts/container-storage-modules/values.yaml +++ b/charts/container-storage-modules/values.yaml @@ -250,7 +250,7 @@ csi-vxflexos: version: v2.9.0 images: driverRepository: dellemc - powerflexSdc: dellemc/sdc:3.6.1 + powerflexSdc: dellemc/sdc:4.5 certSecretCount: 0 controller: replication: diff --git a/charts/csi-powerstore/templates/_helpers.tpl b/charts/csi-powerstore/templates/_helpers.tpl index a16f8d5e..a628bbb6 100644 --- a/charts/csi-powerstore/templates/_helpers.tpl +++ b/charts/csi-powerstore/templates/_helpers.tpl @@ -4,7 +4,7 @@ Return the appropriate sidecar images based on k8s version {{- define "csi-powerstore.attacherImage" -}} {{- if eq .Capabilities.KubeVersion.Major "1" }} {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "24") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-attacher:v4.4.0" -}} + {{- print "registry.k8s.io/sig-storage/csi-attacher:v4.4.1" -}} {{- end -}} {{- end -}} {{- end -}} diff --git a/charts/csi-vxflexos/templates/_helpers.tpl b/charts/csi-vxflexos/templates/_helpers.tpl index e21cd0b0..66af7753 100644 --- a/charts/csi-vxflexos/templates/_helpers.tpl +++ b/charts/csi-vxflexos/templates/_helpers.tpl @@ -4,7 +4,7 @@ Return the appropriate sidecar images based on k8s version {{- define "csi-vxflexos.attacherImage" -}} {{- if eq .Capabilities.KubeVersion.Major "1" }} {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "21") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-attacher:v4.4.0" -}} + {{- print "registry.k8s.io/sig-storage/csi-attacher:v4.4.1" -}} {{- end -}} {{- end -}} {{- end -}} diff --git a/charts/csi-vxflexos/values.yaml b/charts/csi-vxflexos/values.yaml index dd890f46..f8db6015 100644 --- a/charts/csi-vxflexos/values.yaml +++ b/charts/csi-vxflexos/values.yaml @@ -10,7 +10,7 @@ images: driverRepository: dellemc # "powerflexSdc" defines the SDC image for init container. - powerflexSdc: dellemc/sdc:3.6.1 + powerflexSdc: dellemc/sdc:4.5 # Represents number of certificate secrets, which user is going to create for ssl authentication. (vxflexos-cert-0..vxflexos-cert-n) From f4e8eef084f5f532720eb27cb74f6ae5c57ef938 Mon Sep 17 00:00:00 2001 From: coulof <49235405+coulof@users.noreply.github.com> Date: Tue, 31 Oct 2023 12:12:11 +0100 Subject: [PATCH 05/26] [charts/csi-powerstore] Centralize all images to one place (#310) * One place to configure all images * Use latest versions of CSI official sidecars * Set Default container to the driver and avoid podmon logs by default * Fix annotation and remove driverVersion --- charts/csi-powerstore/templates/_helpers.tpl | 51 ------------------- .../csi-powerstore/templates/controller.yaml | 22 ++++---- charts/csi-powerstore/templates/node.yaml | 8 +-- charts/csi-powerstore/values.yaml | 40 ++++++++------- 4 files changed, 38 insertions(+), 83 deletions(-) diff --git a/charts/csi-powerstore/templates/_helpers.tpl b/charts/csi-powerstore/templates/_helpers.tpl index a628bbb6..cdbe7ada 100644 --- a/charts/csi-powerstore/templates/_helpers.tpl +++ b/charts/csi-powerstore/templates/_helpers.tpl @@ -1,54 +1,3 @@ -{{/* -Return the appropriate sidecar images based on k8s version -*/}} -{{- define "csi-powerstore.attacherImage" -}} - {{- if eq .Capabilities.KubeVersion.Major "1" }} - {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "24") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-attacher:v4.4.1" -}} - {{- end -}} - {{- end -}} -{{- end -}} - -{{- define "csi-powerstore.provisionerImage" -}} - {{- if eq .Capabilities.KubeVersion.Major "1" }} - {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "24") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-provisioner:v3.6.0" -}} - {{- end -}} - {{- end -}} -{{- end -}} - -{{- define "csi-powerstore.snapshotterImage" -}} - {{- if eq .Capabilities.KubeVersion.Major "1" }} - {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "24") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-snapshotter:v6.3.0" -}} - {{- end -}} - {{- end -}} -{{- end -}} - -{{- define "csi-powerstore.resizerImage" -}} - {{- if eq .Capabilities.KubeVersion.Major "1" }} - {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "24") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-resizer:v1.9.0" -}} - {{- end -}} - {{- end -}} -{{- end -}} - -{{- define "csi-powerstore.registrarImage" -}} - {{- if eq .Capabilities.KubeVersion.Major "1" }} - {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "24") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.9.0" -}} - {{- end -}} - {{- end -}} -{{- end -}} - -{{- define "csi-powerstore.healthmonitorImage" -}} - {{- if eq .Capabilities.KubeVersion.Major "1" }} - {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "24") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.10.0" -}} - {{- end -}} - {{- end -}} -{{- end -}} - {{/* Return true if storage capacity tracking is enabled and is supported based on k8s version */}} diff --git a/charts/csi-powerstore/templates/controller.yaml b/charts/csi-powerstore/templates/controller.yaml index 8770c6a2..968ccf61 100644 --- a/charts/csi-powerstore/templates/controller.yaml +++ b/charts/csi-powerstore/templates/controller.yaml @@ -171,6 +171,8 @@ spec: metadata: labels: name: {{ .Release.Name }}-controller + annotations: + kubectl.kubernetes.io/default-container: driver spec: {{ if .Values.controller.nodeSelector }} nodeSelector: @@ -195,7 +197,7 @@ spec: {{- if hasKey .Values "podmon" }} {{- if eq .Values.podmon.enabled true }} - name: podmon - image: {{ required "Must provide the podmon container image." .Values.podmon.image }} + image: {{ required "Must provide the podmon container image." .Values.images.podmon }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: {{- toYaml .Values.podmon.controller.args | nindent 12 }} @@ -223,7 +225,7 @@ spec: {{ if .Values.dev.enableTracing }}{{- include "pstore.tracing" . | nindent 8 }}{{ end }} {{- end }} - name: attacher - image: {{ required "Must provide the CSI attacher container image." ( include "csi-powerstore.attacherImage" . ) }} + image: {{ required "Must provide the CSI attacher container image." .Values.images.attacher }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: - "--csi-address=$(ADDRESS)" @@ -241,7 +243,7 @@ spec: {{- if hasKey .Values.controller "resizer" }} {{- if eq .Values.controller.resizer.enabled true }} - name: resizer - image: {{ required "Must provide the CSI resizer container image." ( include "csi-powerstore.resizerImage" . ) }} + image: {{ required "Must provide the CSI resizer container image." .Values.images.resizer }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: - "--csi-address=$(ADDRESS)" @@ -256,7 +258,7 @@ spec: {{end}} {{end}} - name: provisioner - image: {{ required "Must provide the CSI provisioner container image." ( include "csi-powerstore.provisionerImage" . ) }} + image: {{ required "Must provide the CSI provisioner container image." .Values.images.provisioner }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: - "--csi-address=$(ADDRESS)" @@ -287,7 +289,7 @@ spec: {{- if hasKey .Values.controller "snapshot" }} {{- if eq .Values.controller.snapshot.enabled true }} - name: snapshotter - image: {{ required "Must provide the CSI snapshotter container image." ( include "csi-powerstore.snapshotterImage" . ) }} + image: {{ required "Must provide the CSI snapshotter container image." .Values.images.snapshotter }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: - "--csi-address=$(ADDRESS)" @@ -305,7 +307,7 @@ spec: {{- if hasKey .Values.controller "vgsnapshot" }} {{- if eq .Values.controller.vgsnapshot.enabled true }} - name: vg-snapshotter - image: {{ required "Must provide the vgsnapshotter container image." .Values.controller.vgsnapshot.image }} + image: {{ required "Must provide the vgsnapshotter container image." .Values.images.vgsnapshotter }} imagePullPolicy: {{ .Values.imagePullPolicy }} env: - name: ADDRESS @@ -318,7 +320,7 @@ spec: {{- if hasKey .Values.controller "replication" }} {{- if eq .Values.controller.replication.enabled true}} - name: dell-csi-replicator - image: {{ required "Must provide the Dell CSI Replicator image." .Values.controller.replication.image}} + image: {{ required "Must provide the Dell CSI Replicator image." .Values.images.replication }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: - "--csi-address=$(ADDRESS)" @@ -346,7 +348,7 @@ spec: {{- if hasKey .Values.controller "healthMonitor" }} {{- if eq .Values.controller.healthMonitor.enabled true}} - name: csi-external-health-monitor-controller - image: {{ required "Must provide the CSI external health monitor controller image." ( include "csi-powerstore.healthmonitorImage" . ) }} + image: {{ required "Must provide the CSI external health monitor controller image." .Values.images.healthmonitor }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: - "--v=5" @@ -365,7 +367,7 @@ spec: {{- end }} {{- end }} - name: csi-metadata-retriever - image: {{ required "Must provide the CSI Metadata retriever container image." .Values.controller.metadataretriever }} + image: {{ required "Must provide the CSI Metadata retriever container image." .Values.images.metadataretriever }} imagePullPolicy: {{ .Values.imagePullPolicy }} command: [ "/csi-metadata-retriever" ] env: @@ -380,7 +382,7 @@ spec: - name: socket-dir mountPath: /var/run/csi - name: driver - image: {{ required "Must provide the PowerStore driver image repository." .Values.images.driverRepository }}/{{ .Chart.Name }}:{{ .Values.version }} + image: {{ required "Must provide the PowerStore driver image repository." .Values.images.driver }} imagePullPolicy: {{ .Values.imagePullPolicy }} command: [ "/csi-powerstore" ] env: diff --git a/charts/csi-powerstore/templates/node.yaml b/charts/csi-powerstore/templates/node.yaml index 9771df68..1ee79696 100644 --- a/charts/csi-powerstore/templates/node.yaml +++ b/charts/csi-powerstore/templates/node.yaml @@ -96,6 +96,8 @@ spec: {{- if .Values.podmon.enabled }} driver.dellemc.com: dell-storage {{- end }} + annotations: + kubectl.kubernetes.io/default-container: driver spec: {{ if .Values.node.nodeSelector }} nodeSelector: @@ -118,7 +120,7 @@ spec: capabilities: add: ["SYS_ADMIN"] allowPrivilegeEscalation: true - image: {{ required "Must provide the podmon container image." .Values.podmon.image }} + image: {{ required "Must provide the podmon container image." .Values.images.podmon }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: {{- toYaml .Values.podmon.node.args | nindent 12 }} @@ -171,7 +173,7 @@ spec: capabilities: add: ["SYS_ADMIN"] allowPrivilegeEscalation: true - image: {{ required "Must provide the Powerstore driver image repository." .Values.images.driverRepository }}/{{ .Chart.Name }}:{{ .Values.version }} + image: {{ required "Must provide the Powerstore driver image repository." .Values.images.driver }} imagePullPolicy: {{ .Values.imagePullPolicy }} command: [ "/csi-powerstore" ] env: @@ -264,7 +266,7 @@ spec: - name: powerstore-config-params mountPath: /powerstore-config-params - name: registrar - image: {{ required "Must provide the CSI node registrar container image." ( include "csi-powerstore.registrarImage" . ) }} + image: {{ required "Must provide the CSI node registrar container image." .Values.images.registrar }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: - "--v=5" diff --git a/charts/csi-powerstore/values.yaml b/charts/csi-powerstore/values.yaml index 2159d901..c191e970 100644 --- a/charts/csi-powerstore/values.yaml +++ b/charts/csi-powerstore/values.yaml @@ -21,9 +21,28 @@ # Allowed values: string # Default value: None driverName: "csi-powerstore.dellemc.com" +# "version" is used to verify the values file matches driver version +# Not recommend to change +version: v2.9.0 -# Driver version required to pull the latest driver image -version: "v2.9.0" +# "images" defines every container images used for the driver and its sidecars. +# To use your own images, or a private registry, change the values here. +images: + # "driver" defines the container image, used for the driver container. + driver: dellemc/csi-powerstore:v2.9.0 + # CSI sidecars + attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.0 + provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.0 + snapshotter: registry.k8s.io/sig-storage/csi-snapshotter:v6.3.0 + resizer: registry.k8s.io/sig-storage/csi-resizer:v1.9.0 + registrar: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.9.0 + healthmonitor: registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.10.0 + + # CSM sidecars + replication: dellemc/dell-csi-replicator:v1.6.0 + vgsnapshotter: dellemc/csi-volumegroup-snapshotter:v1.3.0 + podmon: dellemc/podmon:v1.7.0 + metadataretriever: dellemc/csi-metadata-retriever:v1.5.0 # Specify kubelet config dir path. # Ensure that the config.yaml file is present at this path. @@ -103,10 +122,6 @@ controller: # false: disable volume-group-snapshot feature(do not install vg-snapshotter sidecar) # Default value: false enabled: false - # image: Image to use for volume-group-snapshotter. This shouldn't be changed - # Allowed values: string - # Default value: dellemc/csi-volumegroup-snapshotter:v1.3.0 - image: dellemc/csi-volumegroup-snapshotter:v1.3.0 # snapshot: allows to enable/disable snapshot feature # snapshot CRDs needs to be installed before enabling this feature @@ -155,11 +170,6 @@ controller: # Default value: false enabled: false - # image: Image to use for dell-csi-replicator. This shouldn't be changed - # Allowed values: string - # Default value: None - image: dellemc/dell-csi-replicator:v1.6.0 - # replicationContextPrefix: prefix to use for naming of resources created by replication feature # Allowed values: string # Default value: powerstore @@ -170,9 +180,6 @@ controller: # Default value: replication.storage.dell.com replicationPrefix: "replication.storage.dell.com" - # Image for csi-metadata-retriever - metadataretriever: dellemc/csi-metadata-retriever:v1.5.0 - # nodeSelector: Define node selection constraints for controller pods. # For the pod to be eligible to run on a node, the node must have each # of the indicated key-value pairs as labels. @@ -318,7 +325,6 @@ storageCapacity: # Enable this feature only after contact support for additional information podmon: enabled: false - image: dellemc/podmon:v1.7.0 controller: args: - "--csisock=unix:/var/run/csi/csi.sock" @@ -342,7 +348,3 @@ podmon: - "--driver-config-params=/powerstore-config-params/driver-config-params.yaml" - "--driverPodLabelValue=dell-storage" - "--ignoreVolumelessPods=false" - -images: - # "driver" defines the container image, used for the driver container. - driverRepository: dellemc From 91d94d2ea7668037218df967cbf9e7822cfe60d1 Mon Sep 17 00:00:00 2001 From: coulof <49235405+coulof@users.noreply.github.com> Date: Tue, 31 Oct 2023 12:36:55 +0100 Subject: [PATCH 06/26] [charts/csi-powerflex] Centralize all images to one place #263 (#311) * Centralize all images to one place * Use latest CSI sidecars and default container to driver * Add default annotation * Use bleeding edge helm validation action to workaround cosign issue --- .github/workflows/helm-validations.yml | 5 +- charts/csi-vxflexos/templates/_helpers.tpl | 59 ++-------------- charts/csi-vxflexos/templates/controller.yaml | 24 ++++--- charts/csi-vxflexos/templates/node.yaml | 10 +-- charts/csi-vxflexos/values.yaml | 67 ++++++++++--------- 5 files changed, 61 insertions(+), 104 deletions(-) diff --git a/.github/workflows/helm-validations.yml b/.github/workflows/helm-validations.yml index 2993841f..1e91c2d6 100644 --- a/.github/workflows/helm-validations.yml +++ b/.github/workflows/helm-validations.yml @@ -41,7 +41,8 @@ jobs: # Lint and version increment checks on updated charts - name: Run lint/version increment checks id: versionIncrement - uses: helm/chart-testing-action@v2.4.0 + # Bump to proper version after this issue is fixed : https://github.com/helm/chart-testing-action/issues/132 + uses: helm/chart-testing-action@cb49023b9227b1097e5eddd8824f48bdea11b1aa with: command: lint config: lintConfig.yaml @@ -62,7 +63,7 @@ jobs: # Run a check on karavi-observability chart for updated dependency versions - name: Run dependency version checks id: lint - uses: helm/chart-testing-action@v2.4.0 + uses: helm/chart-testing-action@cb49023b9227b1097e5eddd8824f48bdea11b1aa with: command: lint config: dependencyConfig.yaml diff --git a/charts/csi-vxflexos/templates/_helpers.tpl b/charts/csi-vxflexos/templates/_helpers.tpl index 66af7753..a7df6b37 100644 --- a/charts/csi-vxflexos/templates/_helpers.tpl +++ b/charts/csi-vxflexos/templates/_helpers.tpl @@ -1,61 +1,10 @@ -{{/* -Return the appropriate sidecar images based on k8s version -*/}} -{{- define "csi-vxflexos.attacherImage" -}} - {{- if eq .Capabilities.KubeVersion.Major "1" }} - {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "21") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-attacher:v4.4.1" -}} - {{- end -}} - {{- end -}} -{{- end -}} - -{{- define "csi-vxflexos.provisionerImage" -}} - {{- if eq .Capabilities.KubeVersion.Major "1" }} - {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "21") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-provisioner:v3.6.0" -}} - {{- end -}} - {{- end -}} -{{- end -}} - -{{- define "csi-vxflexos.snapshotterImage" -}} - {{- if eq .Capabilities.KubeVersion.Major "1" }} - {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "21") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-snapshotter:v6.3.0" -}} - {{- end -}} - {{- end -}} -{{- end -}} - -{{- define "csi-vxflexos.resizerImage" -}} - {{- if eq .Capabilities.KubeVersion.Major "1" }} - {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "21") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-resizer:v1.9.0" -}} - {{- end -}} - {{- end -}} -{{- end -}} - -{{- define "csi-vxflexos.registrarImage" -}} - {{- if eq .Capabilities.KubeVersion.Major "1" }} - {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "21") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.9.0" -}} - {{- end -}} - {{- end -}} -{{- end -}} - -{{- define "csi-vxflexos.healthmonitorImage" -}} - {{- if eq .Capabilities.KubeVersion.Major "1" }} - {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "21") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.10.0" -}} - {{- end -}} - {{- end -}} -{{- end -}} - {{/* Return true if storage capacity tracking is enabled and is supported based on k8s version */}} {{- define "csi-vxflexos.isStorageCapacitySupported" -}} -{{- if eq .Values.storageCapacity.enabled true -}} - {{- if and (eq .Capabilities.KubeVersion.Major "1") (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "24") -}} - {{- true -}} + {{- if eq .Values.storageCapacity.enabled true -}} + {{- if and (eq .Capabilities.KubeVersion.Major "1") (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "24") -}} + {{- true -}} + {{- end -}} {{- end -}} -{{- end -}} {{- end -}} \ No newline at end of file diff --git a/charts/csi-vxflexos/templates/controller.yaml b/charts/csi-vxflexos/templates/controller.yaml index 6cb80cb2..62f7ec68 100644 --- a/charts/csi-vxflexos/templates/controller.yaml +++ b/charts/csi-vxflexos/templates/controller.yaml @@ -145,7 +145,7 @@ metadata: {{- if hasKey .Values "authorization" }} {{- if eq .Values.authorization.enabled true }} annotations: - com.dell.karavi-authorization-proxy: "true" + com.dell.karavi-authorization-proxy: "true" {{ end }} {{ end }} spec: @@ -167,6 +167,8 @@ spec: vg-snapshotter-enabled: "false" {{- end }} name: {{ .Release.Name }}-controller + annotations: + kubectl.kubernetes.io/default-container: "driver" spec: affinity: nodeSelector: @@ -192,7 +194,7 @@ spec: {{- if hasKey .Values "podmon" }} {{- if eq .Values.podmon.enabled true }} - name: podmon - image: {{ required "Must provide the podmon container image." .Values.podmon.image }} + image: {{ required "Must provide the podmon container image." .Values.images.podmon }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: {{- toYaml .Values.podmon.controller.args | nindent 12 }} @@ -217,7 +219,7 @@ spec: {{- end }} {{- end }} - name: attacher - image: {{ required "Must provide the CSI attacher container image." ( include "csi-vxflexos.attacherImage" . ) }} + image: {{ required "Must provide the CSI attacher container image." .Values.images.attacher }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: - "--csi-address=$(ADDRESS)" @@ -232,7 +234,7 @@ spec: {{- if hasKey .Values.controller "replication" }} {{- if eq .Values.controller.replication.enabled true}} - name: dell-csi-replicator - image: {{ required "Must provide the Dell CSI Replicator image." .Values.controller.replication.image}} + image: {{ required "Must provide the Dell CSI Replicator image." .Values.images.replication }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: - "--csi-address=$(ADDRESS)" @@ -258,7 +260,7 @@ spec: {{- end }} {{- end }} - name: provisioner - image: {{ required "Must provide the CSI provisioner container image." ( include "csi-vxflexos.provisionerImage" . ) }} + image: {{ required "Must provide the CSI provisioner container image." .Values.images.provisioner }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: - "--csi-address=$(ADDRESS)" @@ -290,7 +292,7 @@ spec: {{- if hasKey .Values.controller "healthMonitor" }} {{- if eq .Values.controller.healthMonitor.enabled true}} - name: csi-external-health-monitor-controller - image: {{ required "Must provide the CSI external health monitor image." ( include "csi-vxflexos.healthmonitorImage" . ) }} + image: {{ required "Must provide the CSI external health monitor image." .Values.images.healthmonitor }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: - "--csi-address=$(ADDRESS)" @@ -311,7 +313,7 @@ spec: {{- if hasKey .Values "vgsnapshotter" }} {{- if eq .Values.vgsnapshotter.enabled true }} - name: vg-snapshotter - image: {{ required "Must provide the vgsnapshotter container image." .Values.vgsnapshotter.image }} + image: {{ required "Must provide the vgsnapshotter container image." .Values.images.vgsnapshotter }} imagePullPolicy: {{ .Values.imagePullPolicy }} env: - name: ADDRESS @@ -324,7 +326,7 @@ spec: {{- if hasKey .Values.controller "snapshot" }} {{- if eq .Values.controller.snapshot.enabled true }} - name: snapshotter - image: {{ required "Must provide the CSI snapshotter container image. " ( include "csi-vxflexos.snapshotterImage" . ) }} + image: {{ required "Must provide the CSI snapshotter container image. " .Values.images.snapshotter }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: - "--csi-address=$(ADDRESS)" @@ -342,7 +344,7 @@ spec: {{- if hasKey .Values.controller "resizer" }} {{- if eq .Values.controller.resizer.enabled true }} - name: resizer - image: {{ required "Must provide the CSI resizer container image." ( include "csi-vxflexos.resizerImage" . ) }} + image: {{ required "Must provide the CSI resizer container image." .Values.images.resizer }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: - "--csi-address=$(ADDRESS)" @@ -359,7 +361,7 @@ spec: {{- if hasKey .Values "authorization" }} {{- if eq .Values.authorization.enabled true }} - name: karavi-authorization-proxy - image: {{ required "Must provide the authorization sidecar container image." .Values.authorization.sidecarProxyImage }} + image: {{ required "Must provide the authorization sidecar container image." .Values.images.authorization }} imagePullPolicy: {{ .Values.imagePullPolicy }} env: - name: PROXY_HOST @@ -388,7 +390,7 @@ spec: {{- end }} {{- end }} - name: driver - image: "{{ required "Must provide the driver image repository." .Values.images.driverRepository }}/{{ .Chart.Name }}:{{ .Values.version }}" + image: "{{ required "Must provide the driver image repository." .Values.images.driver }}" imagePullPolicy: {{ .Values.imagePullPolicy }} command: [ "/csi-vxflexos.sh" ] args: diff --git a/charts/csi-vxflexos/templates/node.yaml b/charts/csi-vxflexos/templates/node.yaml index 44806fb0..b914ce63 100644 --- a/charts/csi-vxflexos/templates/node.yaml +++ b/charts/csi-vxflexos/templates/node.yaml @@ -80,6 +80,8 @@ spec: {{- if eq .Values.podmon.enabled true }} driver.dellemc.com: dell-storage {{- end }} + annotations: + kubectl.kubernetes.io/default-container: "driver" spec: {{- if .Values.node.nodeSelector }} nodeSelector: @@ -106,7 +108,7 @@ spec: capabilities: add: ["SYS_ADMIN"] allowPrivilegeEscalation: true - image: {{ required "Must provide the podmon container image." .Values.podmon.image }} + image: {{ required "Must provide the podmon container image." .Values.images.podmon }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: {{- toYaml .Values.podmon.node.args | nindent 12 }} @@ -148,7 +150,7 @@ spec: {{- if hasKey .Values "authorization" }} {{- if eq .Values.authorization.enabled true }} - name: karavi-authorization-proxy - image: {{ required "Must provide the authorization sidecar container image." .Values.authorization.sidecarProxyImage }} + image: {{ required "Must provide the authorization sidecar container image." .Values.images.authorization }} imagePullPolicy: {{ .Values.imagePullPolicy }} env: - name: PROXY_HOST @@ -182,7 +184,7 @@ spec: allowPrivilegeEscalation: true capabilities: add: ["SYS_ADMIN"] - image: "{{ required "Must provide the driver image repository." .Values.images.driverRepository }}/{{ .Chart.Name }}:{{ .Values.version }}" + image: "{{ required "Must provide the driver image repository." .Values.images.driver }}" imagePullPolicy: {{ .Values.imagePullPolicy }} command: [ "/csi-vxflexos.sh" ] args: @@ -238,7 +240,7 @@ spec: readOnly: true {{- end}} - name: registrar - image: {{ required "Must provide the CSI registrar container image." ( include "csi-vxflexos.registrarImage" . ) }} + image: {{ required "Must provide the CSI registrar container image." .Values.images.registrar }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: - "--v=5" diff --git a/charts/csi-vxflexos/values.yaml b/charts/csi-vxflexos/values.yaml index f8db6015..1542feed 100644 --- a/charts/csi-vxflexos/values.yaml +++ b/charts/csi-vxflexos/values.yaml @@ -5,13 +5,26 @@ # Not recommend to change version: v2.9.0 +# "images" defines every container images used for the driver and its sidecars. +# To use your own images, or a private registry, change the values here. + images: # "driver" defines the container image, used for the driver container. - driverRepository: dellemc - + driver: dellemc/csi-vxflexos:v2.9.0 # "powerflexSdc" defines the SDC image for init container. powerflexSdc: dellemc/sdc:4.5 - + # CSI sidecars + attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.0 + provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.0 + snapshotter: registry.k8s.io/sig-storage/csi-snapshotter:v6.3.0 + resizer: registry.k8s.io/sig-storage/csi-resizer:v1.9.0 + registrar: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.9.0 + healthmonitor: registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.10.0 + # CSM sidecars + replication: dellemc/dell-csi-replicator:v1.6.0 + vgsnapshotter: dellemc/csi-volumegroup-snapshotter:v1.3.0 + podmon: dellemc/podmon:v1.7.0 + authorization: dellemc/csm-authorization-sidecar:v1.8.0 # Represents number of certificate secrets, which user is going to create for ssl authentication. (vxflexos-cert-0..vxflexos-cert-n) # If user does not use certificate, set to 0 @@ -107,11 +120,6 @@ controller: # Default value: false enabled: false - # image: Image to use for dell-csi-replicator. This shouldn't be changed - # Allowed values: string - # Default value: None - image: dellemc/dell-csi-replicator:v1.6.0 - # replicationContextPrefix: prefix to use for naming of resources created by replication feature # Allowed values: string # Default value: powerflex @@ -301,31 +309,30 @@ monitor: # These options control the running of the vgsnapshotter container vgsnapshotter: enabled: false - image: dellemc/csi-volumegroup-snapshotter:v1.3.0 # Podmon is an optional feature under development and tech preview. # Enable this feature only after contact support for additional information podmon: enabled: false - image: dellemc/podmon:v1.7.0 - #controller: - # args: - # - "--csisock=unix:/var/run/csi/csi.sock" - # - "--labelvalue=csi-vxflexos" - # - "--mode=controller" - # - "--skipArrayConnectionValidation=false" - # - "--driver-config-params=/vxflexos-config-params/driver-config-params.yaml" - # - "--driverPodLabelValue=dell-storage" - # - "--ignoreVolumelessPods=false" - #node: - # args: - # - "--csisock=unix:/var/lib/kubelet/plugins/vxflexos.emc.dell.com/csi_sock" - # - "--labelvalue=csi-vxflexos" - # - "--mode=node" - # - "--leaderelection=false" - # - "--driver-config-params=/vxflexos-config-params/driver-config-params.yaml" - # - "--driverPodLabelValue=dell-storage" - # - "--ignoreVolumelessPods=false" + + controller: + args: + - "--csisock=unix:/var/run/csi/csi.sock" + - "--labelvalue=csi-vxflexos" + - "--mode=controller" + - "--skipArrayConnectionValidation=false" + - "--driver-config-params=/vxflexos-config-params/driver-config-params.yaml" + - "--driverPodLabelValue=dell-storage" + - "--ignoreVolumelessPods=false" + node: + args: + - "--csisock=unix:/var/lib/kubelet/plugins/vxflexos.emc.dell.com/csi_sock" + - "--labelvalue=csi-vxflexos" + - "--mode=node" + - "--leaderelection=false" + - "--driver-config-params=/vxflexos-config-params/driver-config-params.yaml" + - "--driverPodLabelValue=dell-storage" + - "--ignoreVolumelessPods=false" # CSM module attributes # authorization: enable csm-authorization for RBAC @@ -337,10 +344,6 @@ podmon: authorization: enabled: false - # sidecarProxyImage: the container image used for the csm-authorization-sidecar. - # Default value: dellemc/csm-authorization-sidecar:v1.8.0 - sidecarProxyImage: dellemc/csm-authorization-sidecar:v1.8.0 - # proxyHost: hostname of the csm-authorization server # Default value: None proxyHost: From 0ceea2a9d04c93e29059b799a989251784d6d9f9 Mon Sep 17 00:00:00 2001 From: coulof <49235405+coulof@users.noreply.github.com> Date: Tue, 31 Oct 2023 12:48:44 +0100 Subject: [PATCH 07/26] [charts/csi-powermax] Container images in one place (#309) * One place to configure all images * Fix the csi provisioner to use the full name * Use latest versions of CSI official sidecars --- charts/csi-powermax/templates/_helpers.tpl | 50 ------------------- charts/csi-powermax/templates/controller.yaml | 20 ++++---- charts/csi-powermax/templates/csidriver.yaml | 2 +- charts/csi-powermax/templates/node.yaml | 8 +-- charts/csi-powermax/values.yaml | 35 ++++++------- 5 files changed, 33 insertions(+), 82 deletions(-) diff --git a/charts/csi-powermax/templates/_helpers.tpl b/charts/csi-powermax/templates/_helpers.tpl index 4cb67bdd..80bf5d70 100644 --- a/charts/csi-powermax/templates/_helpers.tpl +++ b/charts/csi-powermax/templates/_helpers.tpl @@ -1,53 +1,3 @@ -{{/* -Return the appropriate sidecar images based on k8s version -*/}} -{{- define "csi-powermax.attacherImage" -}} - {{- if eq .Capabilities.KubeVersion.Major "1" }} - {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "23") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-attacher:v4.3.0" -}} - {{- end -}} - {{- end -}} -{{- end -}} - -{{- define "csi-powermax.provisionerImage" -}} - {{- if eq .Capabilities.KubeVersion.Major "1" }} - {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "23") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-provisioner:v3.5.0" -}} - {{- end -}} - {{- end -}} -{{- end -}} - -{{- define "csi-powermax.snapshotterImage" -}} - {{- if eq .Capabilities.KubeVersion.Major "1" }} - {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "23") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-snapshotter:v6.2.2" -}} - {{- end -}} - {{- end -}} -{{- end -}} - -{{- define "csi-powermax.resizerImage" -}} - {{- if eq .Capabilities.KubeVersion.Major "1" }} - {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "23") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-resizer:v1.8.0" -}} - {{- end -}} - {{- end -}} -{{- end -}} - -{{- define "csi-powermax.registrarImage" -}} - {{- if eq .Capabilities.KubeVersion.Major "1" }} - {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "23") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.8.0" -}} - {{- end -}} - {{- end -}} -{{- end -}} - -{{- define "csi-powermax.healthmonitorImage" -}} - {{- if eq .Capabilities.KubeVersion.Major "1" }} - {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "23") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.9.0" -}} - {{- end -}} - {{- end -}} -{{- end -}} {{- define "csi-powermax.isStorageCapacitySupported" -}} {{- if eq .Values.storageCapacity.enabled true -}} diff --git a/charts/csi-powermax/templates/controller.yaml b/charts/csi-powermax/templates/controller.yaml index 7785fbf0..12d6b2da 100644 --- a/charts/csi-powermax/templates/controller.yaml +++ b/charts/csi-powermax/templates/controller.yaml @@ -168,7 +168,7 @@ spec: topologyKey: kubernetes.io/hostname containers: - name: attacher - image: {{ required "Must provide the CSI attacher container image." ( include "csi-powermax.attacherImage" . ) }} + image: {{ required "Must provide the CSI attacher container image." .Values.images.attacher }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: - "--csi-address=$(ADDRESS)" @@ -183,7 +183,7 @@ spec: - name: socket-dir mountPath: /var/run/csi - name: provisioner - image: {{ required "Must provide the CSI provisioner container image." ( include "csi-powermax.provisionerImage" . ) }} + image: {{ required "Must provide the CSI provisioner container image." .Values.images.provisioner }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: - "--csi-address=$(ADDRESS)" @@ -216,7 +216,7 @@ spec: {{- if hasKey .Values.controller "snapshot" }} {{- if eq .Values.controller.snapshot.enabled true }} - name: snapshotter - image: {{ required "Must provide the CSI snapshotter container image." ( include "csi-powermax.snapshotterImage" . ) }} + image: {{ required "Must provide the CSI snapshotter container image." .Values.images.snapshotter }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: - "--csi-address=$(ADDRESS)" @@ -235,7 +235,7 @@ spec: {{- end }} {{- if eq .Values.replication.enabled true}} - name: dell-csi-replicator - image: {{ required "Must provide the Dell CSI Replicator Resizer image." .Values.replication.image}} + image: {{ required "Must provide the Dell CSI Replicator Resizer image." .Values.images.replication }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: - "--csi-address=$(ADDRESS)" @@ -261,7 +261,7 @@ spec: {{- end }} {{- if eq .Values.migration.enabled true}} - name: dell-csi-migrator - image: {{ required "Must provide the Dell CSI Migrator Resizer image." .Values.migration.image}} + image: {{ required "Must provide the Dell CSI Migrator Resizer image." .Values.images.migration }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: - "--csi-address=$(ADDRESS)" @@ -287,7 +287,7 @@ spec: {{- if hasKey .Values.controller "resizer" }} {{- if eq .Values.controller.resizer.enabled true }} - name: resizer - image: {{ required "Must provide the CSI resizer container image." ( include "csi-powermax.resizerImage" . ) }} + image: {{ required "Must provide the CSI resizer container image." .Values.images.resizer }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: - "--csi-address=$(ADDRESS)" @@ -306,7 +306,7 @@ spec: {{- if eq .Values.authorization.enabled true }} - name: karavi-authorization-proxy imagePullPolicy: {{ .Values.imagePullPolicy }} - image: {{ required "Must provide the authorization sidecar container image." .Values.authorization.sidecarProxyImage }} + image: {{ required "Must provide the authorization sidecar container image." .Values.images.authorization }} env: - name: PROXY_HOST value: "{{ .Values.authorization.proxyHost }}" @@ -337,7 +337,7 @@ spec: {{- if eq .Values.controller.healthMonitor.enabled true }} - name: csi-external-health-monitor-controller imagePullPolicy: {{ .Values.imagePullPolicy }} - image: {{ required "Must provide the CSI external health monitor controller image." ( include "csi-powermax.healthmonitorImage" . ) }} + image: {{ required "Must provide the CSI external health monitor controller image." .Values.images.healthmonitor }} args: - "--v=5" - "--csi-address=$(ADDRESS)" @@ -355,7 +355,7 @@ spec: {{- end }} {{- end }} - name: driver - image: {{ required "Must provide the PowerMax driver image repository." .Values.images.driverRepository }}/{{ .Chart.Name }}:{{ .Values.version }} + image: {{ required "Must provide the PowerMax driver image repository." .Values.images.driver }} imagePullPolicy: {{ .Values.imagePullPolicy }} command: [ "/csi-powermax.sh" ] args: @@ -459,7 +459,7 @@ spec: mountPath: /powermax-config-params {{- if eq .Values.csireverseproxy.deployAsSidecar true }} - name: reverseproxy - image: {{ required "Must provided an image for reverseproxy container." .Values.csireverseproxy.image }} + image: {{ required "Must provided an image for reverseproxy container." .Values.images.csireverseproxy }} imagePullPolicy: {{ .Values.imagePullPolicy }} env: - name: X_CSI_REVPROXY_CONFIG_DIR diff --git a/charts/csi-powermax/templates/csidriver.yaml b/charts/csi-powermax/templates/csidriver.yaml index 2717b327..88b7c7c7 100644 --- a/charts/csi-powermax/templates/csidriver.yaml +++ b/charts/csi-powermax/templates/csidriver.yaml @@ -4,7 +4,7 @@ metadata: {{- if eq .Values.customDriverName.enabled true}} name: {{ printf "%s-%s" .Release.Namespace .Values.customDriverName.value }} {{- else }} - name: csi-powermax + name: csi-powermax.dellemc.com {{- end }} spec: podInfoOnMount: true diff --git a/charts/csi-powermax/templates/node.yaml b/charts/csi-powermax/templates/node.yaml index 2d91c9b1..8b05dd82 100644 --- a/charts/csi-powermax/templates/node.yaml +++ b/charts/csi-powermax/templates/node.yaml @@ -106,7 +106,7 @@ spec: capabilities: add: ["SYS_ADMIN"] allowPrivilegeEscalation: true - image: {{ required "Must provide the PowerMax driver image repository." .Values.images.driverRepository }}/{{ .Chart.Name }}:{{ .Values.version }} + image: {{ required "Must provide the PowerMax driver image repository." .Values.images.driver }} imagePullPolicy: {{ .Values.imagePullPolicy }} command: [ "/csi-powermax.sh" ] env: @@ -245,7 +245,7 @@ spec: {{- end }} {{- end }} - name: registrar - image: {{ required "Must provide the CSI node registrar container image." ( include "csi-powermax.registrarImage" . ) }} + image: {{ required "Must provide the CSI node registrar container image." .Values.images.registrar }} args: - "--v=5" - "--csi-address=$(ADDRESS)" @@ -275,7 +275,7 @@ spec: add: [ "SYS_ADMIN" ] allowPrivilegeEscalation: true imagePullPolicy: {{ .Values.imagePullPolicy }} - image: {{ required "Must provide the node rescanner sidecar container image." .Values.migration.nodeRescanSidecarImage }} + image: {{ required "Must provide the node rescanner sidecar container image." .Values.images.noderescan }} args: - "--csi-address=$(ADDRESS)" - "--retry-interval-start=1s" @@ -313,7 +313,7 @@ spec: {{- if eq .Values.authorization.enabled true }} - name: karavi-authorization-proxy imagePullPolicy: {{ .Values.imagePullPolicy }} - image: {{ required "Must provide the authorization sidecar container image." .Values.authorization.sidecarProxyImage }} + image: {{ required "Must provide the authorization sidecar container image." .Values.images.authorization }} env: - name: PROXY_HOST value: "{{ .Values.authorization.proxyHost }}" diff --git a/charts/csi-powermax/values.yaml b/charts/csi-powermax/values.yaml index 885a8ba9..26a01960 100644 --- a/charts/csi-powermax/values.yaml +++ b/charts/csi-powermax/values.yaml @@ -51,9 +51,26 @@ global: # Don't modify this value as this value will be used by the install script version: "v2.8.0" +# "images" defines every container images used for the driver and its sidecars. +# To use your own images, or a private registry, change the values here. images: # "driver" defines the container image, used for the driver container. - driverRepository: dellemc + driver: dellemc/csi-powermax:v2.8.0 + csireverseproxy: dellemc/csipowermax-reverseproxy:v2.7.0 + # CSI sidecars + attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.0 + provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.0 + snapshotter: registry.k8s.io/sig-storage/csi-snapshotter:v6.3.0 + resizer: registry.k8s.io/sig-storage/csi-resizer:v1.9.0 + registrar: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.9.0 + healthmonitor: registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.10.0 + # CSM sidecars + replication: dellemc/dell-csi-replicator:v1.6.0 + authorization: dellemc/csm-authorization-sidecar:v1.8.0 + migration: dellemc/dell-csi-migrator:v1.2.0 + # Node rescan sidecar does a rescan on nodes for identifying new paths + # Default value: dellemc/dell-csi-node-rescanner:v1.0.1 + noderescan: dellemc/dell-csi-node-rescanner:v1.1.0 ## K8S/DRIVER ATTRIBUTES ######################## # customDriverName: If enabled, sets the driver name to the @@ -260,10 +277,6 @@ enableCHAP: false # csireverseproxy: Refers to the subchart csireverseproxy csireverseproxy: - # image: Define the container images used for the reverse proxy - # Default value: None - # Example: "csipowermax-reverseproxy:v2.7.0" - image: dellemc/csipowermax-reverseproxy:v2.7.0 # "tlsSecret" defines the TLS secret that is created with certificate # and its associated key # Default value: None @@ -380,9 +393,6 @@ openshift: false # Default value: "false" replication: enabled: false - # Change this to use any specific version of the dell-csi-replicator sidecar - # Default value: None - image: dellemc/dell-csi-replicator:v1.6.0 # replicationContextPrefix enables side cars to read # required information from the volume context # Default value: "powermax" @@ -401,12 +411,6 @@ replication: # Default value: "false" migration: enabled: false - # Change this to use any specific version of the dell-csi-migrator sidecar - # Default value: None - image: dellemc/dell-csi-migrator:v1.2.0 - # Node rescan sidecar does a rescan on nodes for identifying new paths - # Default value: None - nodeRescanSidecarImage: dellemc/dell-csi-node-rescanner:v1.1.0 # migrationPrefix: Determine if migration is enabled # Default value: "migration.storage.dell.com" # Examples: "migration.storage.dell.com" @@ -421,9 +425,6 @@ migration: # Default value: "false" authorization: enabled: false - # sidecarProxyImage: the container image used for the csm-authorization-sidecar. - # Default value: dellemc/csm-authorization-sidecar:v1.8.0 - sidecarProxyImage: dellemc/csm-authorization-sidecar:v1.8.0 # proxyHost: hostname of the csm-authorization server # Default value: None proxyHost: From 975c6bfb33cbc1ac76a481818099c73bf0c9dd5b Mon Sep 17 00:00:00 2001 From: boyamurthy <92081029+boyamurthy@users.noreply.github.com> Date: Thu, 2 Nov 2023 12:00:43 +0530 Subject: [PATCH 08/26] Removing linked proxy mode for proxy (#301) --- .../charts/csireverseproxy/conf/config.yaml | 45 +------------------ charts/csi-powermax/values.yaml | 6 --- 2 files changed, 1 insertion(+), 50 deletions(-) diff --git a/charts/csi-powermax/charts/csireverseproxy/conf/config.yaml b/charts/csi-powermax/charts/csireverseproxy/conf/config.yaml index 3f6419f1..51ed5390 100644 --- a/charts/csi-powermax/charts/csireverseproxy/conf/config.yaml +++ b/charts/csi-powermax/charts/csireverseproxy/conf/config.yaml @@ -1,49 +1,7 @@ -mode: {{ .Values.mode }} +mode: "StandAlone" port: {{ .Values.port }} logLevel: {{ .Values.global.logLevel | default "debug" }} logFormat: {{ .Values.global.logFormat | default "TEXT" }} - {{- if eq .Values.mode "Linked" }} -linkConfig: - primary: - {{- $primary := first .Values.global.managementServers }} - url: {{ required "Must provide a primary Unisphere HTTPS endpoint." $primary.endpoint }} - {{- if $primary.certSecret }} - {{- $check := toString $primary.skipCertificateValidation }} - skipCertificateValidation: {{ ternary $primary.skipCertificateValidation true (or (eq $check "true") (eq $check "false")) }} - {{- else }} - skipCertificateValidation: true - {{- end }} - certSecret: {{ $primary.certSecret | default "" }} - {{- if $primary.limits }} - {{- $limits := $primary.limits }} - limits: - maxActiveRead: {{ $limits.maxActiveRead | default 0 }} - maxActiveWrite: {{ $limits.maxActiveWrite | default 0 }} - maxOutStandingRead: {{ $limits.maxOutStandingRead | default 0 }} - maxOutStandingWrite: {{ $limits.maxOutStandingWrite | default 0 }} - {{- end }} - {{- if first (rest .Values.global.managementServers) }} - {{- $backup := first (rest .Values.global.managementServers) }} - backup: - url: {{ required "Must provide a primary Unisphere HTTPS endpoint." $backup.endpoint }} - {{- if $backup.certSecret }} - {{- $check := toString $backup.skipCertificateValidation }} - skipCertificateValidation: {{ ternary $backup.skipCertificateValidation true (or (eq $check "true") (eq $check "false")) }} - {{- else }} - skipCertificateValidation: true - {{- end }} - certSecret: {{ $backup.certSecret | default "" }} - {{- if $backup.limits }} - {{- $limits := $backup.limits }} - limits: - maxActiveRead: {{ $limits.maxActiveRead | default 0 }} - maxActiveWrite: {{ $limits.maxActiveWrite | default 0 }} - maxOutStandingRead: {{ $limits.maxOutStandingRead | default 0 }} - maxOutStandingWrite: {{ $limits.maxOutStandingWrite | default 0 }} - {{- end }} - {{- end }} - {{- end }} -{{- if eq .Values.mode "StandAlone" }} standAloneConfig: {{- $defaultProxyCreds := .Values.global.defaultCredentialsSecret }} storageArrays: @@ -79,4 +37,3 @@ standAloneConfig: maxOutStandingWrite: {{ $value.limits.maxOutStandingWrite | default 0 }} {{- end }} {{- end }} -{{- end }} \ No newline at end of file diff --git a/charts/csi-powermax/values.yaml b/charts/csi-powermax/values.yaml index 26a01960..60d99c35 100644 --- a/charts/csi-powermax/values.yaml +++ b/charts/csi-powermax/values.yaml @@ -16,7 +16,6 @@ global: # The CSI PowerMax ReverseProxy section to fill out the required configuration # Please refer to the doc website about a # detailed explanation of each configuration parameter - # and the various ReverseProxy modes defaultCredentialsSecret: powermax-creds storageArrays: @@ -292,11 +291,6 @@ csireverseproxy: # Default value: None # Examples: "1111", "8080" port: 2222 - # Mode of CSI reverse proxy - this is a standalone API - # it doesn't belong to kubernetes cluster API - # Default value: None - # Example: "StandAlone" - mode: StandAlone # Auto-create TLS certificate for csi-reverseproxy certManager: # Set selfSignedCert to use a self-signed certificate From d2d56557a3137a73e650c0151f64b167170e8d76 Mon Sep 17 00:00:00 2001 From: boyamurthy <92081029+boyamurthy@users.noreply.github.com> Date: Mon, 6 Nov 2023 11:33:45 +0530 Subject: [PATCH 09/26] Updatting latest sidecars for Powermax (#319) --- charts/csi-powermax/values.yaml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/charts/csi-powermax/values.yaml b/charts/csi-powermax/values.yaml index 60d99c35..78e2812e 100644 --- a/charts/csi-powermax/values.yaml +++ b/charts/csi-powermax/values.yaml @@ -57,9 +57,9 @@ images: driver: dellemc/csi-powermax:v2.8.0 csireverseproxy: dellemc/csipowermax-reverseproxy:v2.7.0 # CSI sidecars - attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.0 - provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.0 - snapshotter: registry.k8s.io/sig-storage/csi-snapshotter:v6.3.0 + attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.1 + provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.1 + snapshotter: registry.k8s.io/sig-storage/csi-snapshotter:v6.3.1 resizer: registry.k8s.io/sig-storage/csi-resizer:v1.9.0 registrar: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.9.0 healthmonitor: registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.10.0 From 0bc8c31f9f23f04afd2205598209ab38afefa0ba Mon Sep 17 00:00:00 2001 From: Aaron Tye Date: Tue, 7 Nov 2023 10:59:46 -0500 Subject: [PATCH 10/26] [charts/powerscale] Powerscale driver images in one place (#313) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * PowerFlex v2.8.0 release (#250) * PowerFlex SDC updated (#251) * Centralized images * minor typo fixed (#258) * powerstore v2.8.0 & k8s 1.28 support (#262) * update auth to 1.8 (#271) * update auth to 1.8 Signed-off-by: arnchiequ-dell * Updating podmon to 1.7 Signed-off-by: arnchiequ-dell * Update Observability values file for Authorization sidecar image version --------- Signed-off-by: arnchiequ-dell Co-authored-by: Alik Saring * Adding CSI Powerstore driver’s effective pod security profile for supporting ephemeral volumes as part of Openshift 4.13 support (#261) Co-authored-by: Surya Gupta <109594002+suryagupta4@users.noreply.github.com> * removed externalAccess from powerflex manifests. (#283) * Revert "Adding CSI Powerstore driver’s effective pod security profile for supporting ephemeral volumes as part of Openshift 4.13 support (#261)" (#284) This reverts commit c1dd93b953980d99351488593b343c6903633768. * NFS ACL parameter removed (#289) * fix values --------- Signed-off-by: arnchiequ-dell Co-authored-by: KshitijaKakde <111420075+KshitijaKakde@users.noreply.github.com> Co-authored-by: coulof Co-authored-by: Rajshree Khare <69839943+khareRajshree@users.noreply.github.com> Co-authored-by: Surya Gupta <109594002+suryagupta4@users.noreply.github.com> Co-authored-by: aqu-dell <108017114+arnchiequ-dell@users.noreply.github.com> Co-authored-by: Alik Saring Co-authored-by: alankar-verma <125348121+alankar-verma@users.noreply.github.com> Co-authored-by: VamsiSiddu-7 <103578883+VamsiSiddu-7@users.noreply.github.com> --- charts/csi-isilon/templates/_helpers.tpl | 51 -------------- charts/csi-isilon/templates/controller.yaml | 22 +++--- charts/csi-isilon/templates/node.yaml | 10 +-- charts/csi-isilon/values.yaml | 75 +++++++++++---------- 4 files changed, 57 insertions(+), 101 deletions(-) diff --git a/charts/csi-isilon/templates/_helpers.tpl b/charts/csi-isilon/templates/_helpers.tpl index 0bf04fae..ecfe630d 100644 --- a/charts/csi-isilon/templates/_helpers.tpl +++ b/charts/csi-isilon/templates/_helpers.tpl @@ -1,54 +1,3 @@ -{{/* -Return the appropriate sidecar images based on k8s version -*/}} -{{- define "csi-isilon.attacherImage" -}} - {{- if eq .Capabilities.KubeVersion.Major "1" }} - {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "21") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-attacher:v4.3.0" -}} - {{- end -}} - {{- end -}} -{{- end -}} - -{{- define "csi-isilon.provisionerImage" -}} - {{- if eq .Capabilities.KubeVersion.Major "1" }} - {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "21") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-provisioner:v3.5.0" -}} - {{- end -}} - {{- end -}} -{{- end -}} - -{{- define "csi-isilon.snapshotterImage" -}} - {{- if eq .Capabilities.KubeVersion.Major "1" }} - {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "21") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-snapshotter:v6.2.2" -}} - {{- end -}} - {{- end -}} -{{- end -}} - -{{- define "csi-isilon.resizerImage" -}} - {{- if eq .Capabilities.KubeVersion.Major "1" }} - {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "21") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-resizer:v1.8.0" -}} - {{- end -}} - {{- end -}} -{{- end -}} - -{{- define "csi-isilon.registrarImage" -}} - {{- if eq .Capabilities.KubeVersion.Major "1" }} - {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "21") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.8.0" -}} - {{- end -}} - {{- end -}} -{{- end -}} - -{{- define "csi-isilon.healthmonitorImage" -}} - {{- if eq .Capabilities.KubeVersion.Major "1" }} - {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "21") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.9.0" -}} - {{- end -}} - {{- end -}} -{{- end -}} - {{/* Return true if storage capacity tracking is enabled and is supported based on k8s version */}} diff --git a/charts/csi-isilon/templates/controller.yaml b/charts/csi-isilon/templates/controller.yaml index 4c8dc222..3c279baa 100644 --- a/charts/csi-isilon/templates/controller.yaml +++ b/charts/csi-isilon/templates/controller.yaml @@ -194,7 +194,7 @@ spec: {{- if hasKey .Values "podmon" }} {{- if eq .Values.podmon.enabled true }} - name: podmon - image: {{ required "Must provide the podmon container image." .Values.podmon.image }} + image: {{ required "Must provide the podmon container image." .Values.images.podmon }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: {{- toYaml .Values.podmon.controller.args | nindent 12 }} @@ -221,7 +221,7 @@ spec: {{- if hasKey .Values.controller "replication" }} {{- if eq .Values.controller.replication.enabled true}} - name: dell-csi-replicator - image: {{ required "Must provide the Dell CSI Replicator image." .Values.controller.replication.image}} + image: {{ required "Must provide the Dell CSI Replicator image." .Values.images.replication }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: - "--csi-address={{ $driverSockPath }}" @@ -248,7 +248,7 @@ spec: {{- if hasKey .Values.controller "resizer" }} {{- if eq .Values.controller.resizer.enabled true }} - name: resizer{{ $csiSidecarSuffix }} - image: {{ required "Must provide the CSI resizer container image." ( include "csi-isilon.resizerImage" . ) }} + image: {{ required "Must provide the CSI resizer container image." .Values.images.resizer }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: - "--csi-address={{ $driverSockPath }}" @@ -272,7 +272,7 @@ spec: {{ end }} {{ end }} - name: csi-metadata-retriever {{ $csiSidecarSuffix }} - image: {{ required "Must provide the CSI metadata retriever container image." .Values.controller.metadataretriever.image }} + image: {{ required "Must provide the CSI metadata retriever container image." .Values.images.metadataretriever }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: - "--csi-address={{ $driverSockPath }}" @@ -298,7 +298,7 @@ spec: - name: socket-dir mountPath: /var/run/csi - name: attacher{{ $csiSidecarSuffix }} - image: {{ required "Must provide the CSI attacher container image." ( include "csi-isilon.attacherImage" . ) }} + image: {{ required "Must provide the CSI attacher container image." .Values.images.attacher }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: - "--csi-address={{ $driverSockPath }}" @@ -323,7 +323,7 @@ spec: {{- if hasKey .Values.controller "healthMonitor" }} {{- if eq .Values.controller.healthMonitor.enabled true }} - name: external-health-monitor-controller - image: {{ required "Must provide the CSI external-health-monitor-controller container image." ( include "csi-isilon.healthmonitorImage" . ) }} + image: {{ required "Must provide the CSI external-health-monitor-controller container image." .Values.images.healthmonitor }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: - "--csi-address={{ $driverSockPath }}" @@ -351,7 +351,7 @@ spec: {{end}} {{- end }} - name: provisioner{{ $csiSidecarSuffix }} - image: {{ required "Must provide the CSI provisioner container image." ( include "csi-isilon.provisionerImage" . ) }} + image: {{ required "Must provide the CSI provisioner container image." .Values.images.provisioner }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: - "--csi-address={{ $driverSockPath }}" @@ -393,7 +393,7 @@ spec: {{- if eq .Values.controller.snapshot.enabled true }} - name: snapshotter{{ $csiSidecarSuffix }} #image: quay.io/k8scsi/csi-snapshotter:v1.0.0 - image: {{ required "Must provide the CSI snapshotter container image." ( include "csi-isilon.snapshotterImage" . ) }} + image: {{ required "Must provide the CSI snapshotter container image." .Values.images.snapshotter }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: - "--csi-address={{ $driverSockPath }}" @@ -419,7 +419,7 @@ spec: {{end}} {{- if not $encrypted }} - name: driver - image: {{ required "Must provide the Isilon driver image repository." .Values.images.driverRepository }}/{{ .Chart.Name }}:{{ .Values.version }} + image: {{ required "Must provide the Isilon driver image repository." .Values.images.driver }} imagePullPolicy: {{ .Values.imagePullPolicy }} command: [ "/csi-isilon" ] args: @@ -519,7 +519,7 @@ spec: {{- if eq .Values.authorization.enabled true }} - name: karavi-authorization-proxy imagePullPolicy: {{ .Values.imagePullPolicy }} - image: {{ required "Must provide the authorization sidecar container image." .Values.authorization.sidecarProxyImage }} + image: {{ required "Must provide the authorization sidecar container image." .Values.images.authorization }} env: - name: PROXY_HOST value: "{{ .Values.authorization.proxyHost }}" @@ -549,7 +549,7 @@ spec: {{- end }} {{- if $encrypted }} - name: driver-sec - image: {{ .Values.encryption.image }} + image: {{ .Values.images.encryption }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: - --name={{ .Values.encryption.pluginName }} diff --git a/charts/csi-isilon/templates/node.yaml b/charts/csi-isilon/templates/node.yaml index c08c2f92..d84a505a 100644 --- a/charts/csi-isilon/templates/node.yaml +++ b/charts/csi-isilon/templates/node.yaml @@ -113,7 +113,7 @@ spec: capabilities: add: ["SYS_ADMIN"] allowPrivilegeEscalation: true - image: {{ required "Must provide the podmon container image." .Values.podmon.image }} + image: {{ required "Must provide the podmon container image." .Values.images.podmon }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: {{- toYaml .Values.podmon.node.args | nindent 12 }} @@ -167,7 +167,7 @@ spec: capabilities: add: ["SYS_ADMIN"] allowPrivilegeEscalation: true - image: {{ required "Must provide the Isilon driver image repository." .Values.images.driverRepository }}/{{ .Chart.Name }}:{{ .Values.version }} + image: {{ required "Must provide the Isilon driver image repository." .Values.images.driver }} imagePullPolicy: {{ .Values.imagePullPolicy }} env: - name: CSI_ENDPOINT @@ -251,7 +251,7 @@ spec: mountPath: /csi-isilon-config-params {{- end }} - name: registrar{{ $csiSidecarSuffix }} - image: {{ required "Must provide the CSI node registrar container image." ( include "csi-isilon.registrarImage" . ) }} + image: {{ required "Must provide the CSI node registrar container image." .Values.images.registrar }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: - "--v=5" @@ -273,7 +273,7 @@ spec: {{- if eq .Values.authorization.enabled true }} - name: karavi-authorization-proxy imagePullPolicy: {{ .Values.imagePullPolicy }} - image: {{ required "Must provide the authorization sidecar container image." .Values.authorization.sidecarProxyImage }} + image: {{ required "Must provide the authorization sidecar container image." .Values.images.authorization }} env: - name: PROXY_HOST value: "{{ .Values.authorization.proxyHost }}" @@ -303,7 +303,7 @@ spec: {{- end }} {{- if $encrypted }} - name: driver-sec - image: {{ .Values.encryption.image }} + image: {{ .Values.images.encryption }} imagePullPolicy: {{ .Values.imagePullPolicy }} securityContext: privileged: true diff --git a/charts/csi-isilon/values.yaml b/charts/csi-isilon/values.yaml index 59414b3c..7b250f72 100644 --- a/charts/csi-isilon/values.yaml +++ b/charts/csi-isilon/values.yaml @@ -4,6 +4,24 @@ # Note: Do not change this value version: "v2.8.0" +images: + # "driver" defines the container image, used for the driver container. + driver: dellemc/csi-isilon:v2.8.0 + # CSI sidecars + attacher: registry.k8s.io/sig-storage/csi-attacher:v4.3.0 + provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.5.0 + snapshotter: registry.k8s.io/sig-storage/csi-snapshotter:v6.2.2 + resizer: registry.k8s.io/sig-storage/csi-resizer:v1.8.0 + registrar: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.8.0 + healthmonitor: registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.9.0 + + # CSM sidecars + replication: dellemc/dell-csi-replicator:v1.5.0 + podmon: dellemc/podmon:v1.6.0 + authorization: dellemc/csm-authorization-sidecar:v1.7.0 + metadataretriever: dellemc/csi-metadata-retriever:v1.4.0 + encryption: dellemc/csm-encryption:v0.3.0 + # CSI driver log level # Allowed values: "error", "warn"/"warning", "info", "debug" # Default value: "debug" @@ -110,10 +128,6 @@ controller: # Default value: 5s leaderElectionRetryPeriod: 5s - # Image for csi-metadata-retriever - metadataretriever: - image: dellemc/csi-metadata-retriever:v1.4.0 - # replication: allows to configure replication # Replication CRDs must be installed before installing driver replication: @@ -389,30 +403,29 @@ storageCapacity: # Enable this feature only after contact support for additional information podmon: enabled: false - image: dellemc/podmon:v1.7.0 - #controller: - # args: - # - "--csisock=unix:/var/run/csi/csi.sock" - # - "--labelvalue=csi-isilon" - # - "--arrayConnectivityPollRate=60" - # - "--driverPath=csi-isilon.dellemc.com" - # - "--mode=controller" - # - "--skipArrayConnectionValidation=false" - # - "--driver-config-params=/csi-isilon-config-params/driver-config-params.yaml" - # - "--driverPodLabelValue=dell-storage" - # - "--ignoreVolumelessPods=false" - - #node: - # args: - # - "--csisock=unix:/var/lib/kubelet/plugins/csi-isilon/csi_sock" - # - "--labelvalue=csi-isilon" - # - "--arrayConnectivityPollRate=60" - # - "--driverPath=csi-isilon.dellemc.com" - # - "--mode=node" - # - "--leaderelection=false" - # - "--driver-config-params=/csi-isilon-config-params/driver-config-params.yaml" - # - "--driverPodLabelValue=dell-storage" - # - "--ignoreVolumelessPods=false" + controller: + args: + - "--csisock=unix:/var/run/csi/csi.sock" + - "--labelvalue=csi-isilon" + - "--arrayConnectivityPollRate=60" + - "--driverPath=csi-isilon.dellemc.com" + - "--mode=controller" + - "--skipArrayConnectionValidation=false" + - "--driver-config-params=/csi-isilon-config-params/driver-config-params.yaml" + - "--driverPodLabelValue=dell-storage" + - "--ignoreVolumelessPods=false" + + node: + args: + - "--csisock=unix:/var/lib/kubelet/plugins/csi-isilon/csi_sock" + - "--labelvalue=csi-isilon" + - "--arrayConnectivityPollRate=60" + - "--driverPath=csi-isilon.dellemc.com" + - "--mode=node" + - "--leaderelection=false" + - "--driver-config-params=/csi-isilon-config-params/driver-config-params.yaml" + - "--driverPodLabelValue=dell-storage" + - "--ignoreVolumelessPods=false" encryption: # enabled: Enable/disable volume encryption feature. @@ -421,9 +434,6 @@ encryption: # pluginName: The name of the provisioner to use for encrypted volumes. pluginName: "sec-isilon.dellemc.com" - # image: Encryption driver image name. - image: "dellemc/csm-encryption:v0.3.0" - # apiPort: TCP port number used by the REST API server. apiPort: 3838 @@ -450,6 +460,3 @@ encryption: # See the gocryptfs documentation for more details. extraArgs: [] -images: - # "driver" defines the container image, used for the driver container. - driverRepository: dellemc From 8621a19e62cdbd86600bf5ccb46887060e78c554 Mon Sep 17 00:00:00 2001 From: Aaron Tye Date: Tue, 7 Nov 2023 12:53:44 -0500 Subject: [PATCH 11/26] [charts/csi-unity] Centralize all images to one place (#312) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * PowerFlex v2.8.0 release (#250) * Centralized images * Adding CSI Powerstore driver’s effective pod security profile for supporting ephemeral volumes as part of Openshift 4.13 support (#261) Co-authored-by: Surya Gupta <109594002+suryagupta4@users.noreply.github.com> * removed externalAccess from powerflex manifests. (#283) * Revert "Adding CSI Powerstore driver’s effective pod security profile for supporting ephemeral volumes as part of Openshift 4.13 support (#261)" (#284) This reverts commit c1dd93b953980d99351488593b343c6903633768. * Bump sidecars versions * Bump up observability version for CSM 1.9 (#314) * Bump-up observability version for CSM 1.9 * add patch verb to powerscale * Add codeowners * update attacher sidecar for powerflex and powerstore. (#315) * [charts/csi-powerstore] Centralize all images to one place (#310) * One place to configure all images * Use latest versions of CSI official sidecars * Set Default container to the driver and avoid podmon logs by default * Fix annotation and remove driverVersion * [charts/csi-powerflex] Centralize all images to one place #263 (#311) * Centralize all images to one place * Use latest CSI sidecars and default container to driver * Add default annotation * Use bleeding edge helm validation action to workaround cosign issue * [charts/csi-powermax] Container images in one place (#309) * One place to configure all images * Fix the csi provisioner to use the full name * Use latest versions of CSI official sidecars * remove duplicate podmon image --------- Co-authored-by: KshitijaKakde <111420075+KshitijaKakde@users.noreply.github.com> Co-authored-by: coulof Co-authored-by: alankar-verma <125348121+alankar-verma@users.noreply.github.com> Co-authored-by: Surya Gupta <109594002+suryagupta4@users.noreply.github.com> Co-authored-by: VamsiSiddu-7 <103578883+VamsiSiddu-7@users.noreply.github.com> Co-authored-by: coulof <49235405+coulof@users.noreply.github.com> Co-authored-by: panigs7 <92028646+panigs7@users.noreply.github.com> --- charts/csi-unity/templates/_helpers.tpl | 51 ---------------------- charts/csi-unity/templates/controller.yaml | 14 +++--- charts/csi-unity/templates/node.yaml | 6 +-- charts/csi-unity/values.yaml | 19 +++++--- 4 files changed, 24 insertions(+), 66 deletions(-) diff --git a/charts/csi-unity/templates/_helpers.tpl b/charts/csi-unity/templates/_helpers.tpl index c20b246f..4031377c 100644 --- a/charts/csi-unity/templates/_helpers.tpl +++ b/charts/csi-unity/templates/_helpers.tpl @@ -1,54 +1,3 @@ -{{/* -Return the appropriate sidecar images based on k8s version -*/}} -{{- define "csi-unity.attacherImage" -}} - {{- if eq .Capabilities.KubeVersion.Major "1" }} - {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "24") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-attacher:v4.3.0" -}} - {{- end -}} - {{- end -}} -{{- end -}} - -{{- define "csi-unity.provisionerImage" -}} - {{- if eq .Capabilities.KubeVersion.Major "1" }} - {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "24") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-provisioner:v3.5.0" -}} - {{- end -}} - {{- end -}} -{{- end -}} - -{{- define "csi-unity.snapshotterImage" -}} - {{- if eq .Capabilities.KubeVersion.Major "1" }} - {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "24") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-snapshotter:v6.2.2" -}} - {{- end -}} - {{- end -}} -{{- end -}} - -{{- define "csi-unity.resizerImage" -}} - {{- if eq .Capabilities.KubeVersion.Major "1" }} - {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "24") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-resizer:v1.8.0" -}} - {{- end -}} - {{- end -}} -{{- end -}} - -{{- define "csi-unity.registrarImage" -}} - {{- if eq .Capabilities.KubeVersion.Major "1" }} - {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "24") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.8.0" -}} - {{- end -}} - {{- end -}} -{{- end -}} - -{{- define "csi-unity.healthmonitorImage" -}} - {{- if eq .Capabilities.KubeVersion.Major "1" }} - {{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "24") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}} - {{- print "registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.9.0" -}} - {{- end -}} - {{- end -}} -{{- end -}} - {{/* Return true if storage capacity tracking is enabled and is supported based on k8s version */}} diff --git a/charts/csi-unity/templates/controller.yaml b/charts/csi-unity/templates/controller.yaml index 2b126072..1f3e2220 100644 --- a/charts/csi-unity/templates/controller.yaml +++ b/charts/csi-unity/templates/controller.yaml @@ -152,7 +152,7 @@ spec: {{- if .Values.podmon.enabled }} - name: podmon imagePullPolicy: {{ .Values.imagePullPolicy }} - image: {{ required "Must provide the podmon container image." .Values.podmon.image }} + image: {{ required "Must provide the podmon container image." .Values.images.podmon }} args: {{- toYaml .Values.podmon.controller.args | nindent 12 }} env: @@ -175,7 +175,7 @@ spec: mountPath: /unity-config {{- end }} - name: attacher - image: {{ required "Must provide the CSI attacher container image." ( include "csi-unity.attacherImage" . ) }} + image: {{ required "Must provide the CSI attacher container image." .Values.images.attacher }} args: - "--csi-address=$(ADDRESS)" - "--v=5" @@ -187,7 +187,7 @@ spec: - name: socket-dir mountPath: /var/run/csi - name: provisioner - image: {{ required "Must provide the CSI provisioner container image." ( include "csi-unity.provisionerImage" . ) }} + image: {{ required "Must provide the CSI provisioner container image." .Values.images.provisioner }} args: - "--csi-address=$(ADDRESS)" - "--volume-name-prefix={{ required "Must provide a Volume Name Prefix." .Values.controller.volumeNamePrefix }}" @@ -220,7 +220,7 @@ spec: {{- if hasKey .Values.controller "snapshot" }} {{- if eq .Values.controller.snapshot.enabled true }} - name: snapshotter - image: {{ required "Must provide the CSI snapshotter container image. " ( include "csi-unity.snapshotterImage" . ) }} + image: {{ required "Must provide the CSI snapshotter container image. " .Values.images.snapshotter }} args: - "--csi-address=$(ADDRESS)" - "--snapshot-name-prefix={{ required "Must privided a Snapshot Name Prefix" .Values.controller.snapshot.snapNamePrefix }}" @@ -239,7 +239,7 @@ spec: {{- if hasKey .Values.controller "resizer" }} {{- if eq .Values.controller.resizer.enabled true }} - name: resizer - image: {{ required "Must provide the CSI resizer container image." ( include "csi-unity.resizerImage" . ) }} + image: {{ required "Must provide the CSI resizer container image." .Values.images.resizer }} args: - "--csi-address=$(ADDRESS)" - "--v=5" @@ -255,7 +255,7 @@ spec: {{- if hasKey .Values.controller "healthMonitor" }} {{- if eq .Values.controller.healthMonitor.enabled true }} - name: csi-external-health-monitor-controller - image: {{ required "Must provide the CSI external health monitor image." ( include "csi-unity.healthmonitorImage" . ) }} + image: {{ required "Must provide the CSI external health monitor image." .Values.images.healthmonitor }} imagePullPolicy: {{ .Values.imagePullPolicy }} args: - "--v=5" @@ -274,7 +274,7 @@ spec: {{- end }} {{- end }} - name: driver - image: "{{ required "Must provide the driver image repository." .Values.images.driverRepository }}/{{ .Chart.Name }}:{{ .Values.version }}" + image: "{{ required "Must provide the driver image repository." .Values.images.driver }}" args: - "--driver-name=csi-unity.dellemc.com" - "--driver-config=/unity-config/driver-config-params.yaml" diff --git a/charts/csi-unity/templates/node.yaml b/charts/csi-unity/templates/node.yaml index d013812e..9358b0cd 100644 --- a/charts/csi-unity/templates/node.yaml +++ b/charts/csi-unity/templates/node.yaml @@ -97,7 +97,7 @@ spec: add: ["SYS_ADMIN"] allowPrivilegeEscalation: true imagePullPolicy: {{ .Values.imagePullPolicy }} - image: {{ required "Must provide the podmon container image." .Values.podmon.image }} + image: {{ required "Must provide the podmon container image." .Values.images.podmon }} args: {{- toYaml .Values.podmon.node.args | nindent 12 }} env: @@ -145,7 +145,7 @@ spec: capabilities: add: ["SYS_ADMIN"] allowPrivilegeEscalation: true - image: "{{ required "Must provide the driver image repository." .Values.images.driverRepository }}/{{ .Chart.Name }}:{{ .Values.version }}" + image: "{{ required "Must provide the driver image repository." .Values.images.driver }}" args: - "--driver-name=csi-unity.dellemc.com" - "--driver-config=/unity-config/driver-config-params.yaml" @@ -204,7 +204,7 @@ spec: - name: unity-secret mountPath: /unity-secret - name: registrar - image: {{ required "Must provide the CSI registrar container image." ( include "csi-unity.registrarImage" . ) }} + image: {{ required "Must provide the CSI registrar container image." .Values.images.registrar }} args: - "--v=5" - "--csi-address=$(ADDRESS)" diff --git a/charts/csi-unity/values.yaml b/charts/csi-unity/values.yaml index 56825fae..d86fa7ed 100644 --- a/charts/csi-unity/values.yaml +++ b/charts/csi-unity/values.yaml @@ -6,6 +6,20 @@ # Examples : "v2.8.0" , "nightly" version: "v2.8.0" +images: + # "driver" defines the container image, used for the driver container. + driver: dellemc/csi-unity:v2.8.0 + # CSI sidecars + attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.0 + provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.0 + snapshotter: registry.k8s.io/sig-storage/csi-snapshotter:v6.3.0 + resizer: registry.k8s.io/sig-storage/csi-resizer:v1.9.0 + registrar: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.9.0 + healthmonitor: registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.10.0 + + # CSM sidecars + podmon: dellemc/podmon:v1.7.0 + # LogLevel is used to set the logging level of the driver. # Allowed values: "error", "warn"/"warning", "info", "debug" # Default value: "info" @@ -194,7 +208,6 @@ podmon: # allowed values - string # default value : None # Example : "podman:latest", "pod:latest" - image: dellemc/podmon:v1.7.0 controller: args: - "--csisock=unix:/var/run/csi/csi.sock" @@ -255,7 +268,3 @@ storageCapacity: # Allowed values: 1m,2m,3m,...,10m,...,60m etc # Default value: 5m pollInterval: 5m - -images: - # "driver" defines the container image, used for the driver container. - driverRepository: dellemc From 657fe6087e03e67718148580d053c8a34cb40c0c Mon Sep 17 00:00:00 2001 From: Aaron Tye Date: Tue, 7 Nov 2023 13:17:25 -0500 Subject: [PATCH 12/26] [charts/csi-powerscale]: Remove duplicate images (#320) * remove duplicate images * auth 1.8 --- charts/csi-isilon/values.yaml | 12 ++---------- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/charts/csi-isilon/values.yaml b/charts/csi-isilon/values.yaml index 7b250f72..e6d3be6c 100644 --- a/charts/csi-isilon/values.yaml +++ b/charts/csi-isilon/values.yaml @@ -16,9 +16,9 @@ images: healthmonitor: registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.9.0 # CSM sidecars - replication: dellemc/dell-csi-replicator:v1.5.0 + replication: dellemc/dell-csi-replicator:v1.6.0 podmon: dellemc/podmon:v1.6.0 - authorization: dellemc/csm-authorization-sidecar:v1.7.0 + authorization: dellemc/csm-authorization-sidecar:v1.8.0 metadataretriever: dellemc/csi-metadata-retriever:v1.4.0 encryption: dellemc/csm-encryption:v0.3.0 @@ -138,11 +138,6 @@ controller: # Default value: false enabled: false - # image: Image to use for dell-csi-replicator. This shouldn't be changed - # Allowed values: string - # Default value: None - image: dellemc/dell-csi-replicator:v1.6.0 - # replicationContextPrefix: prefix to use for naming of resources created by replication feature # Allowed values: string # Default value: powerscale @@ -373,9 +368,6 @@ autoProbe: true authorization: enabled: false - # sidecarProxyImage: the container image used for the csm-authorization-sidecar. - # Default value: dellemc/csm-authorization-sidecar:v1.8.0 - sidecarProxyImage: dellemc/csm-authorization-sidecar:v1.8.0 # proxyHost: hostname of the csm-authorization server # Default value: None proxyHost: From d46c71cef7292f8ea52d57db5a401c84ae6b2c4f Mon Sep 17 00:00:00 2001 From: coulof <49235405+coulof@users.noreply.github.com> Date: Thu, 9 Nov 2023 14:00:10 +0100 Subject: [PATCH 13/26] New images location (#316) --- charts/container-storage-modules/values.yaml | 97 +++++++++++++++----- 1 file changed, 72 insertions(+), 25 deletions(-) diff --git a/charts/container-storage-modules/values.yaml b/charts/container-storage-modules/values.yaml index e9dbdbbe..bc5a9741 100644 --- a/charts/container-storage-modules/values.yaml +++ b/charts/container-storage-modules/values.yaml @@ -22,7 +22,21 @@ csi-powerstore: enabled: false version: "v2.9.0" images: - driverRepository: dellemc + # "driver" defines the container image, used for the driver container. + driver: dellemc/csi-powerstore:v2.9.0 + # CSI sidecars + attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.0 + provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.0 + snapshotter: registry.k8s.io/sig-storage/csi-snapshotter:v6.3.0 + resizer: registry.k8s.io/sig-storage/csi-resizer:v1.9.0 + registrar: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.9.0 + healthmonitor: registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.10.0 + + # CSM sidecars + replication: dellemc/dell-csi-replicator:v1.6.0 + vgsnapshotter: dellemc/csi-volumegroup-snapshotter:v1.3.0 + podmon: dellemc/podmon:v1.7.0 + metadataretriever: dellemc/csi-metadata-retriever:v1.5.0 ## Controller ATTRIBUTES controller: controllerCount: 2 @@ -31,11 +45,8 @@ csi-powerstore: nodeSelector: replication: enabled: false - image: dellemc/dell-csi-replicator:v1.6.0 vgsnapshot: enabled: false - image: dellemc/csi-volumegroup-snapshotter:v1.3.0 - metadataretriever: dellemc/csi-metadata-retriever:v1.5.0 snapshot: enabled: true resizer: @@ -76,7 +87,6 @@ csi-powerstore: # Enable this feature only after contact support for additional information podmon: enabled: false - image: dellemc/podmon:v1.7.0 controller: args: - "--csisock=unix:/var/run/csi/csi.sock" @@ -121,7 +131,21 @@ csi-powermax: # - endpoint: https://backup-2.unisphe.re:8443 version: "v2.8.0" images: - driverRepository: "dellemc" + # "driver" defines the container image, used for the driver container. + driver: dellemc/csi-powermax:v2.8.0 + csireverseproxy: dellemc/csipowermax-reverseproxy:v2.7.0 + # CSI sidecars + attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.0 + provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.0 + snapshotter: registry.k8s.io/sig-storage/csi-snapshotter:v6.3.0 + resizer: registry.k8s.io/sig-storage/csi-resizer:v1.9.0 + registrar: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.9.0 + healthmonitor: registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.10.0 + # CSM sidecars + replication: dellemc/dell-csi-replicator:v1.6.0 + authorization: dellemc/csm-authorization-sidecar:v1.8.0 + migration: dellemc/dell-csi-migrator:v1.2.0 + noderescan: dellemc/dell-csi-node-rescanner:v1.1.0 clusterPrefix: ABC portGroups: PortGroup1, PortGroup2, PortGroup3 controller: @@ -138,18 +162,13 @@ csi-powermax: enabled: false nodeSelector: csireverseproxy: - image: dellemc/csipowermax-reverseproxy:v2.7.0 deployAsSidecar: true replication: enabled: false - image: dellemc/dell-csi-replicator:v1.6.0 migration: enabled: false - image: dellemc/dell-csi-migrator:v1.2.0 - nodeRescanSidecarImage: dellemc/dell-csi-node-rescanner:v1.1.0 authorization: enabled: false - sidecarProxyImage: dellemc/csm-authorization-sidecar:v1.8.0 proxyHost: vSphere: enabled: false @@ -168,7 +187,21 @@ csi-isilon: enabled: false version: "v2.8.0" images: - driverRepository: dellemc + # "driver" defines the container image, used for the driver container. + driver: dellemc/csi-isilon:v2.8.0 + # CSI sidecars + attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.0 + provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.0 + snapshotter: registry.k8s.io/sig-storage/csi-snapshotter:v6.3.0 + resizer: registry.k8s.io/sig-storage/csi-resizer:v1.9.0 + registrar: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.9.0 + healthmonitor: registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.10.0 + # CSM sidecars + replication: dellemc/dell-csi-replicator:v1.6.0 + podmon: dellemc/podmon:v1.7.0 + authorization: dellemc/csm-authorization-sidecar:v1.8.0 + metadataretriever: dellemc/csi-metadata-retriever:v1.4.0 + encryption: dellemc/csm-encryption:v0.3.0 ## Controller ATTRIBUTES controller: controllerCount: 2 @@ -177,7 +210,6 @@ csi-isilon: nodeSelector: replication: enabled: false - image: dellemc/dell-csi-replicator:v1.6.0 snapshot: enabled: true resizer: @@ -212,7 +244,6 @@ csi-isilon: # Enable this feature only after contact support for additional information podmon: enabled: false - image: dellemc/podmon:v1.7.0 controller: args: - "--csisock=unix:/var/run/csi/csi.sock" @@ -224,7 +255,6 @@ csi-isilon: - "--driver-config-params=/csi-isilon-config-params/driver-config-params.yaml" - "--driverPodLabelValue=dell-storage" - "--ignoreVolumelessPods=false" - node: args: - "--csisock=unix:/var/lib/kubelet/plugins/csi-isilon/csi_sock" @@ -238,7 +268,6 @@ csi-isilon: - "--ignoreVolumelessPods=false" authorization: enabled: false - sidecarProxyImage: dellemc/csm-authorization-sidecar:v1.8.0 proxyHost: encryption: enabled: false @@ -249,13 +278,26 @@ csi-vxflexos: enabled: false version: v2.9.0 images: - driverRepository: dellemc + # "driver" defines the container image, used for the driver container. + driver: dellemc/csi-vxflexos:v2.9.0 + # "powerflexSdc" defines the SDC image for init container. powerflexSdc: dellemc/sdc:4.5 + # CSI sidecars + attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.0 + provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.0 + snapshotter: registry.k8s.io/sig-storage/csi-snapshotter:v6.3.0 + resizer: registry.k8s.io/sig-storage/csi-resizer:v1.9.0 + registrar: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.9.0 + healthmonitor: registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.10.0 + # CSM sidecars + replication: dellemc/dell-csi-replicator:v1.6.0 + vgsnapshotter: dellemc/csi-volumegroup-snapshotter:v1.3.0 + podmon: dellemc/podmon:v1.7.0 + authorization: dellemc/csm-authorization-sidecar:v1.8.0 certSecretCount: 0 controller: replication: enabled: false - image: dellemc/dell-csi-replicator:v1.6.0 healthMonitor: enabled: false controllerCount: 2 @@ -300,13 +342,11 @@ csi-vxflexos: enabled: false vgsnapshotter: enabled: false - image: dellemc/csi-volumegroup-snapshotter:v1.3.0 # maxVxflexosVolumesPerNode - Maximum number of volumes that controller can publish to the node. maxVxflexosVolumesPerNode: 0 podmon: enabled: false - image: dellemc/podmon:v1.7.0 controller: args: - "--csisock=unix:/var/run/csi/csi.sock" @@ -335,7 +375,18 @@ csi-vxflexos: csi-unity: enabled: false version: "v2.8.0" - + images: + # "driver" defines the container image, used for the driver container. + driver: dellemc/csi-unity:v2.8.0 + # CSI sidecars + attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.0 + provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.0 + snapshotter: registry.k8s.io/sig-storage/csi-snapshotter:v6.3.0 + resizer: registry.k8s.io/sig-storage/csi-resizer:v1.9.0 + registrar: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.9.0 + healthmonitor: registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.10.0 + # CSM sidecars + podmon: dellemc/podmon:v1.7.0 # certSecretCount: Represents number of certificate secrets, which user is going to create for # ssl authentication. (unity-cert-0..unity-cert-n) # Allowed values: n, where n > 0 @@ -422,7 +473,6 @@ csi-unity: # effect: "NoSchedule" podmon: enabled: false - image: dellemc/podmon:v1.7.0 controller: args: - "--csisock=unix:/var/run/csi/csi.sock" @@ -458,9 +508,6 @@ csi-unity: storageCapacity: enabled: true - images: - driverRepository: dellemc - ## K8S/Module ATTRIBUTES ########################################## From 22c63987882a2f3124b2f68126ad3014c4b34f10 Mon Sep 17 00:00:00 2001 From: shaynafinocchiaro Date: Tue, 14 Nov 2023 09:44:32 -0500 Subject: [PATCH 14/26] [charts/csi-unity]: Update Unity version for CSM 1.9 (#322) * add unity v2.9.0 support * remove image description --- charts/container-storage-modules/Chart.yaml | 2 +- charts/container-storage-modules/values.yaml | 4 ++-- charts/csi-unity/Chart.yaml | 4 ++-- charts/csi-unity/values.yaml | 10 +++------- 4 files changed, 8 insertions(+), 12 deletions(-) diff --git a/charts/container-storage-modules/Chart.yaml b/charts/container-storage-modules/Chart.yaml index 24e8c698..a290765a 100644 --- a/charts/container-storage-modules/Chart.yaml +++ b/charts/container-storage-modules/Chart.yaml @@ -60,7 +60,7 @@ dependencies: condition: csi-vxflexos.enabled - name: csi-unity - version: 2.8.0 + version: 2.9.0 repository: https://dell.github.io/helm-charts condition: csi-unity.enabled diff --git a/charts/container-storage-modules/values.yaml b/charts/container-storage-modules/values.yaml index bc5a9741..403609f2 100644 --- a/charts/container-storage-modules/values.yaml +++ b/charts/container-storage-modules/values.yaml @@ -374,10 +374,10 @@ csi-vxflexos: ######################## csi-unity: enabled: false - version: "v2.8.0" + version: "v2.9.0" images: # "driver" defines the container image, used for the driver container. - driver: dellemc/csi-unity:v2.8.0 + driver: dellemc/csi-unity:v2.9.0 # CSI sidecars attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.0 provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.0 diff --git a/charts/csi-unity/Chart.yaml b/charts/csi-unity/Chart.yaml index e0071cc2..5d9259fd 100644 --- a/charts/csi-unity/Chart.yaml +++ b/charts/csi-unity/Chart.yaml @@ -1,7 +1,7 @@ apiVersion: v2 -appVersion: 2.8.0 +appVersion: 2.9.0 name: csi-unity -version: 2.8.0 +version: 2.9.0 description: | Unity XT CSI (Container Storage Interface) driver Kubernetes integration. This chart includes everything required to provision via CSI as diff --git a/charts/csi-unity/values.yaml b/charts/csi-unity/values.yaml index d86fa7ed..ebcf4468 100644 --- a/charts/csi-unity/values.yaml +++ b/charts/csi-unity/values.yaml @@ -3,12 +3,12 @@ # version: version of this values file # Note: Do not change this value -# Examples : "v2.8.0" , "nightly" -version: "v2.8.0" +# Examples : "v2.9.0" , "nightly" +version: "v2.9.0" images: # "driver" defines the container image, used for the driver container. - driver: dellemc/csi-unity:v2.8.0 + driver: dellemc/csi-unity:v2.9.0 # CSI sidecars attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.0 provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.0 @@ -204,10 +204,6 @@ podmon: # defaule value : None # Examples : true , false enabled: false - # image - image name - # allowed values - string - # default value : None - # Example : "podman:latest", "pod:latest" controller: args: - "--csisock=unix:/var/run/csi/csi.sock" From b0ecff3017a2ed0de5f8f4ee0a1edc3fde17ad6d Mon Sep 17 00:00:00 2001 From: Luna Xu <10015938+xuluna@users.noreply.github.com> Date: Fri, 17 Nov 2023 14:07:02 -0600 Subject: [PATCH 15/26] bump up version (#323) --- charts/csi-unity/values.yaml | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/charts/csi-unity/values.yaml b/charts/csi-unity/values.yaml index ebcf4468..b96b774b 100644 --- a/charts/csi-unity/values.yaml +++ b/charts/csi-unity/values.yaml @@ -10,11 +10,11 @@ images: # "driver" defines the container image, used for the driver container. driver: dellemc/csi-unity:v2.9.0 # CSI sidecars - attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.0 - provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.0 - snapshotter: registry.k8s.io/sig-storage/csi-snapshotter:v6.3.0 - resizer: registry.k8s.io/sig-storage/csi-resizer:v1.9.0 - registrar: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.9.0 + attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.2 + provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.2 + snapshotter: registry.k8s.io/sig-storage/csi-snapshotter:v6.3.2 + resizer: registry.k8s.io/sig-storage/csi-resizer:v1.9.2 + registrar: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.9.1 healthmonitor: registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.10.0 # CSM sidecars From c7642fba99e9f5dff707dd4b20a286cf5d056fd7 Mon Sep 17 00:00:00 2001 From: Surya Gupta <109594002+suryagupta4@users.noreply.github.com> Date: Mon, 20 Nov 2023 12:47:12 +0530 Subject: [PATCH 16/26] adding noderoot mount path (#324) --- charts/csi-vxflexos/templates/node.yaml | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/charts/csi-vxflexos/templates/node.yaml b/charts/csi-vxflexos/templates/node.yaml index b914ce63..80c898dc 100644 --- a/charts/csi-vxflexos/templates/node.yaml +++ b/charts/csi-vxflexos/templates/node.yaml @@ -228,6 +228,8 @@ spec: - name: pods-path mountPath: {{ .Values.kubeletConfigDir }}/pods mountPropagation: "Bidirectional" + - name: noderoot + mountPath: /noderoot - name: dev mountPath: /dev - name: vxflexos-config @@ -344,6 +346,10 @@ spec: hostPath: path: {{ .Values.kubeletConfigDir }}/pods type: Directory + - name: noderoot + hostPath: + path: / + type: Directory - name: dev hostPath: path: /dev From 543cbdc30e502f88d3747192c312437035317e59 Mon Sep 17 00:00:00 2001 From: KshitijaKakde <111420075+KshitijaKakde@users.noreply.github.com> Date: Mon, 20 Nov 2023 14:19:22 +0530 Subject: [PATCH 17/26] changed sdc version (#325) --- charts/container-storage-modules/values.yaml | 2 +- charts/csi-vxflexos/values.yaml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/charts/container-storage-modules/values.yaml b/charts/container-storage-modules/values.yaml index 403609f2..a8d0fecb 100644 --- a/charts/container-storage-modules/values.yaml +++ b/charts/container-storage-modules/values.yaml @@ -281,7 +281,7 @@ csi-vxflexos: # "driver" defines the container image, used for the driver container. driver: dellemc/csi-vxflexos:v2.9.0 # "powerflexSdc" defines the SDC image for init container. - powerflexSdc: dellemc/sdc:4.5 + powerflexSdc: dellemc/sdc:3.6.1 # CSI sidecars attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.0 provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.0 diff --git a/charts/csi-vxflexos/values.yaml b/charts/csi-vxflexos/values.yaml index 1542feed..96755764 100644 --- a/charts/csi-vxflexos/values.yaml +++ b/charts/csi-vxflexos/values.yaml @@ -12,7 +12,7 @@ images: # "driver" defines the container image, used for the driver container. driver: dellemc/csi-vxflexos:v2.9.0 # "powerflexSdc" defines the SDC image for init container. - powerflexSdc: dellemc/sdc:4.5 + powerflexSdc: dellemc/sdc:3.6.1 # CSI sidecars attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.0 provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.0 From a90d17ec5449339acf66141c55f59fc3805f7818 Mon Sep 17 00:00:00 2001 From: Adarsh Kumar Yadav <109620911+adarsh-dell@users.noreply.github.com> Date: Mon, 20 Nov 2023 14:52:26 +0530 Subject: [PATCH 18/26] sidecars version update for powerstore and powerflex (#326) --- charts/container-storage-modules/values.yaml | 22 ++++++++++---------- charts/csi-powerstore/values.yaml | 12 +++++------ charts/csi-vxflexos/values.yaml | 10 ++++----- 3 files changed, 22 insertions(+), 22 deletions(-) diff --git a/charts/container-storage-modules/values.yaml b/charts/container-storage-modules/values.yaml index a8d0fecb..598bdf1c 100644 --- a/charts/container-storage-modules/values.yaml +++ b/charts/container-storage-modules/values.yaml @@ -25,18 +25,18 @@ csi-powerstore: # "driver" defines the container image, used for the driver container. driver: dellemc/csi-powerstore:v2.9.0 # CSI sidecars - attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.0 - provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.0 - snapshotter: registry.k8s.io/sig-storage/csi-snapshotter:v6.3.0 - resizer: registry.k8s.io/sig-storage/csi-resizer:v1.9.0 - registrar: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.9.0 + attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.2 + provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.2 + snapshotter: registry.k8s.io/sig-storage/csi-snapshotter:v6.3.2 + resizer: registry.k8s.io/sig-storage/csi-resizer:v1.9.2 + registrar: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.9.1 healthmonitor: registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.10.0 # CSM sidecars replication: dellemc/dell-csi-replicator:v1.6.0 vgsnapshotter: dellemc/csi-volumegroup-snapshotter:v1.3.0 podmon: dellemc/podmon:v1.7.0 - metadataretriever: dellemc/csi-metadata-retriever:v1.5.0 + metadataretriever: dellemc/csi-metadata-retriever:v1.6.0 ## Controller ATTRIBUTES controller: controllerCount: 2 @@ -283,11 +283,11 @@ csi-vxflexos: # "powerflexSdc" defines the SDC image for init container. powerflexSdc: dellemc/sdc:3.6.1 # CSI sidecars - attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.0 - provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.0 - snapshotter: registry.k8s.io/sig-storage/csi-snapshotter:v6.3.0 - resizer: registry.k8s.io/sig-storage/csi-resizer:v1.9.0 - registrar: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.9.0 + attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.2 + provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.2 + snapshotter: registry.k8s.io/sig-storage/csi-snapshotter:v6.3.2 + resizer: registry.k8s.io/sig-storage/csi-resizer:v1.9.2 + registrar: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.9.1 healthmonitor: registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.10.0 # CSM sidecars replication: dellemc/dell-csi-replicator:v1.6.0 diff --git a/charts/csi-powerstore/values.yaml b/charts/csi-powerstore/values.yaml index c191e970..10dd0a53 100644 --- a/charts/csi-powerstore/values.yaml +++ b/charts/csi-powerstore/values.yaml @@ -31,18 +31,18 @@ images: # "driver" defines the container image, used for the driver container. driver: dellemc/csi-powerstore:v2.9.0 # CSI sidecars - attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.0 - provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.0 - snapshotter: registry.k8s.io/sig-storage/csi-snapshotter:v6.3.0 - resizer: registry.k8s.io/sig-storage/csi-resizer:v1.9.0 - registrar: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.9.0 + attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.2 + provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.2 + snapshotter: registry.k8s.io/sig-storage/csi-snapshotter:v6.3.2 + resizer: registry.k8s.io/sig-storage/csi-resizer:v1.9.2 + registrar: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.9.1 healthmonitor: registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.10.0 # CSM sidecars replication: dellemc/dell-csi-replicator:v1.6.0 vgsnapshotter: dellemc/csi-volumegroup-snapshotter:v1.3.0 podmon: dellemc/podmon:v1.7.0 - metadataretriever: dellemc/csi-metadata-retriever:v1.5.0 + metadataretriever: dellemc/csi-metadata-retriever:v1.6.0 # Specify kubelet config dir path. # Ensure that the config.yaml file is present at this path. diff --git a/charts/csi-vxflexos/values.yaml b/charts/csi-vxflexos/values.yaml index 96755764..f9b8ae76 100644 --- a/charts/csi-vxflexos/values.yaml +++ b/charts/csi-vxflexos/values.yaml @@ -14,11 +14,11 @@ images: # "powerflexSdc" defines the SDC image for init container. powerflexSdc: dellemc/sdc:3.6.1 # CSI sidecars - attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.0 - provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.0 - snapshotter: registry.k8s.io/sig-storage/csi-snapshotter:v6.3.0 - resizer: registry.k8s.io/sig-storage/csi-resizer:v1.9.0 - registrar: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.9.0 + attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.2 + provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.2 + snapshotter: registry.k8s.io/sig-storage/csi-snapshotter:v6.3.2 + resizer: registry.k8s.io/sig-storage/csi-resizer:v1.9.2 + registrar: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.9.1 healthmonitor: registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.10.0 # CSM sidecars replication: dellemc/dell-csi-replicator:v1.6.0 From 0087eb30f88431309ee80fb200dbe0271b53a690 Mon Sep 17 00:00:00 2001 From: Peres Kereotubo <118770866+PeresKereotubo@users.noreply.github.com> Date: Mon, 20 Nov 2023 12:38:33 -0600 Subject: [PATCH 19/26] updating sidecar image version (#328) --- charts/container-storage-modules/values.yaml | 4 ++-- charts/csi-isilon/values.yaml | 4 ++-- charts/csi-powermax/values.yaml | 2 +- charts/csi-powerstore/values.yaml | 2 +- charts/csi-unity/values.yaml | 2 +- charts/csi-vxflexos/values.yaml | 4 ++-- charts/csm-authorization/Chart.yaml | 4 ++-- charts/csm-authorization/values.yaml | 8 ++++---- charts/karavi-observability/values.yaml | 12 ++++++------ 9 files changed, 21 insertions(+), 21 deletions(-) diff --git a/charts/container-storage-modules/values.yaml b/charts/container-storage-modules/values.yaml index 598bdf1c..68fbd9cf 100644 --- a/charts/container-storage-modules/values.yaml +++ b/charts/container-storage-modules/values.yaml @@ -143,7 +143,7 @@ csi-powermax: healthmonitor: registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.10.0 # CSM sidecars replication: dellemc/dell-csi-replicator:v1.6.0 - authorization: dellemc/csm-authorization-sidecar:v1.8.0 + authorization: dellemc/csm-authorization-sidecar:v1.9.0 migration: dellemc/dell-csi-migrator:v1.2.0 noderescan: dellemc/dell-csi-node-rescanner:v1.1.0 clusterPrefix: ABC @@ -198,7 +198,7 @@ csi-isilon: healthmonitor: registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.10.0 # CSM sidecars replication: dellemc/dell-csi-replicator:v1.6.0 - podmon: dellemc/podmon:v1.7.0 + podmon: dellemc/podmon:v1.8.0 authorization: dellemc/csm-authorization-sidecar:v1.8.0 metadataretriever: dellemc/csi-metadata-retriever:v1.4.0 encryption: dellemc/csm-encryption:v0.3.0 diff --git a/charts/csi-isilon/values.yaml b/charts/csi-isilon/values.yaml index e6d3be6c..94270b5d 100644 --- a/charts/csi-isilon/values.yaml +++ b/charts/csi-isilon/values.yaml @@ -17,8 +17,8 @@ images: # CSM sidecars replication: dellemc/dell-csi-replicator:v1.6.0 - podmon: dellemc/podmon:v1.6.0 - authorization: dellemc/csm-authorization-sidecar:v1.8.0 + podmon: dellemc/podmon:v1.8.0 + authorization: dellemc/csm-authorization-sidecar:v1.9.0 metadataretriever: dellemc/csi-metadata-retriever:v1.4.0 encryption: dellemc/csm-encryption:v0.3.0 diff --git a/charts/csi-powermax/values.yaml b/charts/csi-powermax/values.yaml index 78e2812e..0d1db47c 100644 --- a/charts/csi-powermax/values.yaml +++ b/charts/csi-powermax/values.yaml @@ -65,7 +65,7 @@ images: healthmonitor: registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.10.0 # CSM sidecars replication: dellemc/dell-csi-replicator:v1.6.0 - authorization: dellemc/csm-authorization-sidecar:v1.8.0 + authorization: dellemc/csm-authorization-sidecar:v1.9.0 migration: dellemc/dell-csi-migrator:v1.2.0 # Node rescan sidecar does a rescan on nodes for identifying new paths # Default value: dellemc/dell-csi-node-rescanner:v1.0.1 diff --git a/charts/csi-powerstore/values.yaml b/charts/csi-powerstore/values.yaml index 10dd0a53..5f589a12 100644 --- a/charts/csi-powerstore/values.yaml +++ b/charts/csi-powerstore/values.yaml @@ -41,7 +41,7 @@ images: # CSM sidecars replication: dellemc/dell-csi-replicator:v1.6.0 vgsnapshotter: dellemc/csi-volumegroup-snapshotter:v1.3.0 - podmon: dellemc/podmon:v1.7.0 + podmon: dellemc/podmon:v1.8.0 metadataretriever: dellemc/csi-metadata-retriever:v1.6.0 # Specify kubelet config dir path. diff --git a/charts/csi-unity/values.yaml b/charts/csi-unity/values.yaml index b96b774b..c311a19f 100644 --- a/charts/csi-unity/values.yaml +++ b/charts/csi-unity/values.yaml @@ -18,7 +18,7 @@ images: healthmonitor: registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.10.0 # CSM sidecars - podmon: dellemc/podmon:v1.7.0 + podmon: dellemc/podmon:v1.8.0 # LogLevel is used to set the logging level of the driver. # Allowed values: "error", "warn"/"warning", "info", "debug" diff --git a/charts/csi-vxflexos/values.yaml b/charts/csi-vxflexos/values.yaml index f9b8ae76..98ed13c5 100644 --- a/charts/csi-vxflexos/values.yaml +++ b/charts/csi-vxflexos/values.yaml @@ -23,8 +23,8 @@ images: # CSM sidecars replication: dellemc/dell-csi-replicator:v1.6.0 vgsnapshotter: dellemc/csi-volumegroup-snapshotter:v1.3.0 - podmon: dellemc/podmon:v1.7.0 - authorization: dellemc/csm-authorization-sidecar:v1.8.0 + podmon: dellemc/podmon:v1.8.0 + authorization: dellemc/csm-authorization-sidecar:v1.9.0 # Represents number of certificate secrets, which user is going to create for ssl authentication. (vxflexos-cert-0..vxflexos-cert-n) # If user does not use certificate, set to 0 diff --git a/charts/csm-authorization/Chart.yaml b/charts/csm-authorization/Chart.yaml index 21dbf905..ac692b3d 100644 --- a/charts/csm-authorization/Chart.yaml +++ b/charts/csm-authorization/Chart.yaml @@ -1,7 +1,7 @@ apiVersion: v2 name: csm-authorization -version: 1.8.0 -appVersion: 1.8.0 +version: 1.9.0 +appVersion: 1.9.0 type: application description: | CSM for Authorization is part of the [Container Storage Modules](https://github.com/dell/csm) open source suite of Kubernetes diff --git a/charts/csm-authorization/values.yaml b/charts/csm-authorization/values.yaml index c07ff9b8..20967512 100644 --- a/charts/csm-authorization/values.yaml +++ b/charts/csm-authorization/values.yaml @@ -11,10 +11,10 @@ cert-manager: authorization: # images to use in installation images: - proxyService: dellemc/csm-authorization-proxy:v1.8.0 - tenantService: dellemc/csm-authorization-tenant:v1.8.0 - roleService: dellemc/csm-authorization-role:v1.8.0 - storageService: dellemc/csm-authorization-storage:v1.8.0 + proxyService: dellemc/csm-authorization-proxy:v1.9.0 + tenantService: dellemc/csm-authorization-tenant:v1.9.0 + roleService: dellemc/csm-authorization-role:v1.9.0 + storageService: dellemc/csm-authorization-storage:v1.9.0 opa: openpolicyagent/opa opaKubeMgmt: openpolicyagent/kube-mgmt:0.11 diff --git a/charts/karavi-observability/values.yaml b/charts/karavi-observability/values.yaml index 236369cf..51f97b95 100644 --- a/charts/karavi-observability/values.yaml +++ b/charts/karavi-observability/values.yaml @@ -40,8 +40,8 @@ karaviMetricsPowerflex: authorization: enabled: false # sidecarProxyImage: the container image used for the csm-authorization-sidecar. - # Default value: dellemc/csm-authorization-sidecar:v1.8.0 - sidecarProxyImage: dellemc/csm-authorization-sidecar:v1.8.0 + # Default value: dellemc/csm-authorization-sidecar:v1.9.0 + sidecarProxyImage: dellemc/csm-authorization-sidecar:v1.9.0 # proxyHost: hostname of the csm-authorization server # Default value: None proxyHost: @@ -116,8 +116,8 @@ karaviMetricsPowerscale: authorization: enabled: false # sidecarProxyImage: the container image used for the csm-authorization-sidecar. - # Default value: dellemc/csm-authorization-sidecar:v1.8.0 - sidecarProxyImage: dellemc/csm-authorization-sidecar:v1.8.0 + # Default value: dellemc/csm-authorization-sidecar:v1.9.0 + sidecarProxyImage: dellemc/csm-authorization-sidecar:v1.9.0 # proxyHost: hostname of the csm-authorization server # Default value: None proxyHost: @@ -153,8 +153,8 @@ karaviMetricsPowermax: authorization: enabled: false # sidecarProxyImage: the container image used for the csm-authorization-sidecar. - # Default value: dellemc/csm-authorization-sidecar:v1.8.0 - sidecarProxyImage: dellemc/csm-authorization-sidecar:v1.8.0 + # Default value: dellemc/csm-authorization-sidecar:v1.9.0 + sidecarProxyImage: dellemc/csm-authorization-sidecar:v1.9.0 # proxyHost: hostname of the csm-authorization server # Default value: None proxyHost: From e7a8a3db9522ab835536e839dd476171d2a8ea3e Mon Sep 17 00:00:00 2001 From: Adarsh Kumar Yadav <109620911+adarsh-dell@users.noreply.github.com> Date: Tue, 21 Nov 2023 15:52:22 +0530 Subject: [PATCH 20/26] Revert "changed sdc version (#325)" (#329) This reverts commit 543cbdc30e502f88d3747192c312437035317e59. --- charts/container-storage-modules/values.yaml | 2 +- charts/csi-vxflexos/values.yaml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/charts/container-storage-modules/values.yaml b/charts/container-storage-modules/values.yaml index 68fbd9cf..f54c5ba1 100644 --- a/charts/container-storage-modules/values.yaml +++ b/charts/container-storage-modules/values.yaml @@ -281,7 +281,7 @@ csi-vxflexos: # "driver" defines the container image, used for the driver container. driver: dellemc/csi-vxflexos:v2.9.0 # "powerflexSdc" defines the SDC image for init container. - powerflexSdc: dellemc/sdc:3.6.1 + powerflexSdc: dellemc/sdc:4.5 # CSI sidecars attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.2 provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.2 diff --git a/charts/csi-vxflexos/values.yaml b/charts/csi-vxflexos/values.yaml index 98ed13c5..48c55f84 100644 --- a/charts/csi-vxflexos/values.yaml +++ b/charts/csi-vxflexos/values.yaml @@ -12,7 +12,7 @@ images: # "driver" defines the container image, used for the driver container. driver: dellemc/csi-vxflexos:v2.9.0 # "powerflexSdc" defines the SDC image for init container. - powerflexSdc: dellemc/sdc:3.6.1 + powerflexSdc: dellemc/sdc:4.5 # CSI sidecars attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.2 provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.2 From 427ee2278b367de74ec3e9a610a3ad5e48078853 Mon Sep 17 00:00:00 2001 From: Akshay Saini <109056238+AkshaySainiDell@users.noreply.github.com> Date: Wed, 22 Nov 2023 16:46:46 +0530 Subject: [PATCH 21/26] Update VGS version to 1.4.0 (#331) --- charts/container-storage-modules/values.yaml | 4 ++-- charts/csi-powerstore/values.yaml | 2 +- charts/csi-vxflexos/values.yaml | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/charts/container-storage-modules/values.yaml b/charts/container-storage-modules/values.yaml index f54c5ba1..6105ad66 100644 --- a/charts/container-storage-modules/values.yaml +++ b/charts/container-storage-modules/values.yaml @@ -34,7 +34,7 @@ csi-powerstore: # CSM sidecars replication: dellemc/dell-csi-replicator:v1.6.0 - vgsnapshotter: dellemc/csi-volumegroup-snapshotter:v1.3.0 + vgsnapshotter: dellemc/csi-volumegroup-snapshotter:v1.4.0 podmon: dellemc/podmon:v1.7.0 metadataretriever: dellemc/csi-metadata-retriever:v1.6.0 ## Controller ATTRIBUTES @@ -291,7 +291,7 @@ csi-vxflexos: healthmonitor: registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.10.0 # CSM sidecars replication: dellemc/dell-csi-replicator:v1.6.0 - vgsnapshotter: dellemc/csi-volumegroup-snapshotter:v1.3.0 + vgsnapshotter: dellemc/csi-volumegroup-snapshotter:v1.4.0 podmon: dellemc/podmon:v1.7.0 authorization: dellemc/csm-authorization-sidecar:v1.8.0 certSecretCount: 0 diff --git a/charts/csi-powerstore/values.yaml b/charts/csi-powerstore/values.yaml index 5f589a12..a9785e58 100644 --- a/charts/csi-powerstore/values.yaml +++ b/charts/csi-powerstore/values.yaml @@ -40,7 +40,7 @@ images: # CSM sidecars replication: dellemc/dell-csi-replicator:v1.6.0 - vgsnapshotter: dellemc/csi-volumegroup-snapshotter:v1.3.0 + vgsnapshotter: dellemc/csi-volumegroup-snapshotter:v1.4.0 podmon: dellemc/podmon:v1.8.0 metadataretriever: dellemc/csi-metadata-retriever:v1.6.0 diff --git a/charts/csi-vxflexos/values.yaml b/charts/csi-vxflexos/values.yaml index 48c55f84..b8e8ebcf 100644 --- a/charts/csi-vxflexos/values.yaml +++ b/charts/csi-vxflexos/values.yaml @@ -22,7 +22,7 @@ images: healthmonitor: registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.10.0 # CSM sidecars replication: dellemc/dell-csi-replicator:v1.6.0 - vgsnapshotter: dellemc/csi-volumegroup-snapshotter:v1.3.0 + vgsnapshotter: dellemc/csi-volumegroup-snapshotter:v1.4.0 podmon: dellemc/podmon:v1.8.0 authorization: dellemc/csm-authorization-sidecar:v1.9.0 From 8efde3fc01f28a6dec0c350d26e182de097a9cc5 Mon Sep 17 00:00:00 2001 From: HarishH-DELL <109663924+HarishH-DELL@users.noreply.github.com> Date: Tue, 28 Nov 2023 17:25:26 +0530 Subject: [PATCH 22/26] Powerscale and sidecar version changes (#332) * Powerscale and sidecar version changes * csi-node-driver-registrar version changed to 2.9.1 --- charts/container-storage-modules/Chart.yaml | 2 +- charts/container-storage-modules/values.yaml | 14 +++++++------- charts/csi-isilon/Chart.yaml | 4 ++-- charts/csi-isilon/values.yaml | 16 ++++++++-------- 4 files changed, 18 insertions(+), 18 deletions(-) diff --git a/charts/container-storage-modules/Chart.yaml b/charts/container-storage-modules/Chart.yaml index a290765a..8462eae4 100644 --- a/charts/container-storage-modules/Chart.yaml +++ b/charts/container-storage-modules/Chart.yaml @@ -50,7 +50,7 @@ dependencies: condition: csi-powermax.enabled - name: csi-isilon - version: 2.8.0 + version: 2.9.0 repository: https://dell.github.io/helm-charts condition: csi-isilon.enabled diff --git a/charts/container-storage-modules/values.yaml b/charts/container-storage-modules/values.yaml index 6105ad66..a8d2c912 100644 --- a/charts/container-storage-modules/values.yaml +++ b/charts/container-storage-modules/values.yaml @@ -185,16 +185,16 @@ csi-powermax: ######################## csi-isilon: enabled: false - version: "v2.8.0" + version: "v2.9.0" images: # "driver" defines the container image, used for the driver container. - driver: dellemc/csi-isilon:v2.8.0 + driver: dellemc/csi-isilon:v2.9.0 # CSI sidecars - attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.0 - provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.0 - snapshotter: registry.k8s.io/sig-storage/csi-snapshotter:v6.3.0 - resizer: registry.k8s.io/sig-storage/csi-resizer:v1.9.0 - registrar: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.9.0 + attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.2 + provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.2 + snapshotter: registry.k8s.io/sig-storage/csi-snapshotter:v6.3.2 + resizer: registry.k8s.io/sig-storage/csi-resizer:v1.9.2 + registrar: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.9.1 healthmonitor: registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.10.0 # CSM sidecars replication: dellemc/dell-csi-replicator:v1.6.0 diff --git a/charts/csi-isilon/Chart.yaml b/charts/csi-isilon/Chart.yaml index 870eaa52..3e63f3dc 100644 --- a/charts/csi-isilon/Chart.yaml +++ b/charts/csi-isilon/Chart.yaml @@ -1,7 +1,7 @@ apiVersion: v2 name: csi-isilon -version: 2.8.0 -appVersion: "2.8.0" +version: 2.9.0 +appVersion: "2.9.0" kubeVersion: ">= 1.21.0 < 1.29.0" #If you are using a complex K8s version like "v1.22.3-mirantis-1", use this kubeVersion check instead #WARNING: this version of the check will allow the use of alpha and beta versions, which is NOT SUPPORTED diff --git a/charts/csi-isilon/values.yaml b/charts/csi-isilon/values.yaml index 94270b5d..c9ba4a9e 100644 --- a/charts/csi-isilon/values.yaml +++ b/charts/csi-isilon/values.yaml @@ -2,18 +2,18 @@ ######################## # version: version of this values file # Note: Do not change this value -version: "v2.8.0" +version: "v2.9.0" images: # "driver" defines the container image, used for the driver container. - driver: dellemc/csi-isilon:v2.8.0 + driver: dellemc/csi-isilon:v2.9.0 # CSI sidecars - attacher: registry.k8s.io/sig-storage/csi-attacher:v4.3.0 - provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.5.0 - snapshotter: registry.k8s.io/sig-storage/csi-snapshotter:v6.2.2 - resizer: registry.k8s.io/sig-storage/csi-resizer:v1.8.0 - registrar: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.8.0 - healthmonitor: registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.9.0 + attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.2 + provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.2 + snapshotter: registry.k8s.io/sig-storage/csi-snapshotter:v6.3.2 + resizer: registry.k8s.io/sig-storage/csi-resizer:v1.9.2 + registrar: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.9.1 + healthmonitor: registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.10.0 # CSM sidecars replication: dellemc/dell-csi-replicator:v1.6.0 From 2358c08d9a3a040a5cde7a77e9100efa4da6f92c Mon Sep 17 00:00:00 2001 From: boyamurthy <92081029+boyamurthy@users.noreply.github.com> Date: Wed, 29 Nov 2023 13:50:30 +0530 Subject: [PATCH 23/26] PowerMax 2.9.0 support (#321) --- charts/container-storage-modules/Chart.yaml | 2 +- charts/container-storage-modules/values.yaml | 6 +++--- charts/csi-powermax/Chart.yaml | 6 +++--- charts/csi-powermax/charts/csireverseproxy/Chart.yaml | 4 ++-- charts/csi-powermax/charts/csireverseproxy/values.yaml | 2 +- charts/csi-powermax/values.yaml | 6 +++--- 6 files changed, 13 insertions(+), 13 deletions(-) diff --git a/charts/container-storage-modules/Chart.yaml b/charts/container-storage-modules/Chart.yaml index 8462eae4..24b2b06c 100644 --- a/charts/container-storage-modules/Chart.yaml +++ b/charts/container-storage-modules/Chart.yaml @@ -45,7 +45,7 @@ dependencies: condition: csi-powerstore.enabled - name: csi-powermax - version: 2.8.0 + version: 2.9.0 repository: https://dell.github.io/helm-charts condition: csi-powermax.enabled diff --git a/charts/container-storage-modules/values.yaml b/charts/container-storage-modules/values.yaml index a8d2c912..f13add48 100644 --- a/charts/container-storage-modules/values.yaml +++ b/charts/container-storage-modules/values.yaml @@ -129,11 +129,11 @@ 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.8.0" + version: "v2.9.0" images: # "driver" defines the container image, used for the driver container. - driver: dellemc/csi-powermax:v2.8.0 - csireverseproxy: dellemc/csipowermax-reverseproxy:v2.7.0 + driver: dellemc/csi-powermax:v2.9.0 + csireverseproxy: dellemc/csipowermax-reverseproxy:v2.8.0 # CSI sidecars attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.0 provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.0 diff --git a/charts/csi-powermax/Chart.yaml b/charts/csi-powermax/Chart.yaml index b4244160..43930591 100644 --- a/charts/csi-powermax/Chart.yaml +++ b/charts/csi-powermax/Chart.yaml @@ -1,7 +1,7 @@ apiVersion: v2 -appVersion: "2.8.0" +appVersion: "2.9.0" name: csi-powermax -version: 2.8.0 +version: 2.9.0 description: | PowerMax CSI (Container Storage Interface) driver Kubernetes integration. This chart includes everything required to provision via CSI as @@ -16,7 +16,7 @@ keywords: - storage dependencies: - name: csireverseproxy - version: 2.7.0 + version: 2.8.0 condition: required home: https://github.com/dell/csi-powermax icon: https://avatars1.githubusercontent.com/u/20958494?s=200&v=4 diff --git a/charts/csi-powermax/charts/csireverseproxy/Chart.yaml b/charts/csi-powermax/charts/csireverseproxy/Chart.yaml index 1825b03b..25ad85eb 100644 --- a/charts/csi-powermax/charts/csireverseproxy/Chart.yaml +++ b/charts/csi-powermax/charts/csireverseproxy/Chart.yaml @@ -4,6 +4,6 @@ description: A Helm chart for CSI PowerMax ReverseProxy type: application -version: 2.7.0 +version: 2.8.0 -appVersion: 2.7.0 +appVersion: 2.8.0 diff --git a/charts/csi-powermax/charts/csireverseproxy/values.yaml b/charts/csi-powermax/charts/csireverseproxy/values.yaml index fcd87e30..bdfc36fa 100644 --- a/charts/csi-powermax/charts/csireverseproxy/values.yaml +++ b/charts/csi-powermax/charts/csireverseproxy/values.yaml @@ -1,4 +1,4 @@ -image: dellemc/csipowermax-reverseproxy:v2.7.0 +image: dellemc/csipowermax-reverseproxy:v2.8.0 port: 2222 # TLS secret which is used for setting up the proxy HTTPS server diff --git a/charts/csi-powermax/values.yaml b/charts/csi-powermax/values.yaml index 0d1db47c..189adeb3 100644 --- a/charts/csi-powermax/values.yaml +++ b/charts/csi-powermax/values.yaml @@ -48,14 +48,14 @@ global: # Current version of the driver # Don't modify this value as this value will be used by the install script -version: "v2.8.0" +version: "v2.9.0" # "images" defines every container images used for the driver and its sidecars. # To use your own images, or a private registry, change the values here. images: # "driver" defines the container image, used for the driver container. - driver: dellemc/csi-powermax:v2.8.0 - csireverseproxy: dellemc/csipowermax-reverseproxy:v2.7.0 + driver: dellemc/csi-powermax:v2.9.0 + csireverseproxy: dellemc/csipowermax-reverseproxy:v2.8.0 # CSI sidecars attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.1 provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.1 From 6b71a683b776078b252c86cb56af6cc7634767c8 Mon Sep 17 00:00:00 2001 From: boyamurthy <92081029+boyamurthy@users.noreply.github.com> Date: Wed, 29 Nov 2023 17:37:55 +0530 Subject: [PATCH 24/26] Updating latest sidecars for PowerMax (#334) --- charts/csi-powermax/values.yaml | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/charts/csi-powermax/values.yaml b/charts/csi-powermax/values.yaml index 189adeb3..265cc950 100644 --- a/charts/csi-powermax/values.yaml +++ b/charts/csi-powermax/values.yaml @@ -57,14 +57,14 @@ images: driver: dellemc/csi-powermax:v2.9.0 csireverseproxy: dellemc/csipowermax-reverseproxy:v2.8.0 # CSI sidecars - attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.1 - provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.1 - snapshotter: registry.k8s.io/sig-storage/csi-snapshotter:v6.3.1 - resizer: registry.k8s.io/sig-storage/csi-resizer:v1.9.0 - registrar: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.9.0 + attacher: registry.k8s.io/sig-storage/csi-attacher:v4.4.2 + provisioner: registry.k8s.io/sig-storage/csi-provisioner:v3.6.2 + snapshotter: registry.k8s.io/sig-storage/csi-snapshotter:v6.3.2 + resizer: registry.k8s.io/sig-storage/csi-resizer:v1.9.2 + registrar: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.9.1 healthmonitor: registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.10.0 # CSM sidecars - replication: dellemc/dell-csi-replicator:v1.6.0 + replication: dellemc/dell-csi-replicator:v1.7.0 authorization: dellemc/csm-authorization-sidecar:v1.9.0 migration: dellemc/dell-csi-migrator:v1.2.0 # Node rescan sidecar does a rescan on nodes for identifying new paths From af162cae6ca4dcb12d7007928d8ab75b8e9e41e0 Mon Sep 17 00:00:00 2001 From: Surya Gupta <109594002+suryagupta4@users.noreply.github.com> Date: Fri, 1 Dec 2023 11:35:01 +0530 Subject: [PATCH 25/26] [charts/csi-powerflex] Added external access (#333) * external access powerflex * adding external access in csm chart --- charts/container-storage-modules/values.yaml | 1 + charts/csi-vxflexos/templates/controller.yaml | 4 +++- charts/csi-vxflexos/values.yaml | 7 ++++++- 3 files changed, 10 insertions(+), 2 deletions(-) diff --git a/charts/container-storage-modules/values.yaml b/charts/container-storage-modules/values.yaml index f13add48..c99d142d 100644 --- a/charts/container-storage-modules/values.yaml +++ b/charts/container-storage-modules/values.yaml @@ -338,6 +338,7 @@ csi-vxflexos: storageCapacity: enabled: true enableQuota: false + externalAccess: monitor: enabled: false vgsnapshotter: diff --git a/charts/csi-vxflexos/templates/controller.yaml b/charts/csi-vxflexos/templates/controller.yaml index 62f7ec68..d308cc00 100644 --- a/charts/csi-vxflexos/templates/controller.yaml +++ b/charts/csi-vxflexos/templates/controller.yaml @@ -428,6 +428,8 @@ spec: value: "{{ .Values.enableQuota }}" {{- end }} {{- end }} + - name: X_CSI_POWERFLEX_EXTERNAL_ACCESS + value: {{ .Values.externalAccess }} volumeMounts: - name: socket-dir mountPath: /var/run/csi @@ -470,4 +472,4 @@ spec: - key: cert-{{ $e }} path: cert-{{ $e }} {{- end }} -{{- end }} \ No newline at end of file +{{- end }} diff --git a/charts/csi-vxflexos/values.yaml b/charts/csi-vxflexos/values.yaml index b8e8ebcf..a5cae4fa 100644 --- a/charts/csi-vxflexos/values.yaml +++ b/charts/csi-vxflexos/values.yaml @@ -59,6 +59,11 @@ defaultFsType: ext4 # Default value: None imagePullPolicy: IfNotPresent +# externalAccess: allows to specify additional entries for hostAccess of NFS volumes. Both single IP address and subnet are valid entries. +# Allowed Values: x.x.x.x/xx or x.x.x.x +# Default Value: None +externalAccess: + # enableQuota: a boolean that, when enabled, will set quota limit for a newly provisioned NFS volume. # Allowed values: # true: set quota for volume @@ -353,4 +358,4 @@ authorization: # "true" - TLS certificate verification will be skipped # "false" - TLS certificate will be verified # Default value: "true" - skipCertificateValidation: true \ No newline at end of file + skipCertificateValidation: true From e66068d0c2fd976420c87ee61108ff3b974914ef Mon Sep 17 00:00:00 2001 From: Malgorzata Dutka Date: Tue, 5 Dec 2023 10:57:50 -0500 Subject: [PATCH 26/26] update COSI version --- charts/cosi/Chart.yaml | 4 ++-- charts/cosi/values.yaml | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/charts/cosi/Chart.yaml b/charts/cosi/Chart.yaml index 157a687d..07b90d88 100644 --- a/charts/cosi/Chart.yaml +++ b/charts/cosi/Chart.yaml @@ -27,10 +27,10 @@ type: application # This is the chart version. This version number should be incremented each time you make changes # to the chart and its templates, including the app version. # Versions are expected to follow Semantic Versioning (https://semver.org/) -version: 0.1.0 +version: 0.1.2 # This is the version number of the application being deployed. This version number should be # incremented each time you make changes to the application. Versions are not expected to # follow Semantic Versioning. They should reflect the version the application is using. # It is recommended to use it with quotes. -appVersion: 1.0.0-alpha +appVersion: 0.1.1-alpha diff --git a/charts/cosi/values.yaml b/charts/cosi/values.yaml index 50d1e4cb..e1578e28 100644 --- a/charts/cosi/values.yaml +++ b/charts/cosi/values.yaml @@ -31,7 +31,7 @@ provisioner: # repository is the COSI driver provisioner container image repository. repository: "docker.io/dell/cosi" # tag is the COSI driver provisioner container image tag. - tag: "v0.1.0" + tag: "v0.1.1" # pullPolicy is the COSI driver provisioner container image pull policy. pullPolicy: "IfNotPresent"