Skip to content

Commit b700f91

Browse files
HarishH-DELLHarishH-DELL
authored
Release v1.9.0 for Modules and Drivers (#344)
1 parent c48cfd6 commit b700f91

File tree

39 files changed

+281
-546
lines changed

39 files changed

+281
-546
lines changed

‎.github/CODEOWNERS

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,9 @@
1313
# Alik Saring (alikdell)
1414
# Aaron Tye (atye)
1515
# Chaganti Rajitha (chaganti-rajitha)
16+
# Chiman Jain (chimanjain)
1617
# Florian Coulombel (coulof)
18+
# Harish H (HarishH-DELL)
1719
# Meghana G M (meggm)
1820
# Niranjan N (niranjan-n1)
1921
# Shayna Finocchiaro (shaynafinocchiaro)
@@ -25,6 +27,10 @@
2527
# Yiming Bao (baoy1)
2628
# Yian Zong (YianZong)
2729
# Forrest Xia (forrestxia)
30+
# Nitesh Rewatkar (nitesh3108)
31+
# Rajendra Indukuri (rajendraindukuri)
32+
# Shefali Malhotra (shefali-malhotra)
33+
# Spandita Panigrahi (panigs7)
2834

2935
# for all files:
30-
* @gallacher @tdawe @alikdell @atye @hoppea2 @chaganti-rajitha @coulof @meggm @niranjan-n1 @shaynafinocchiaro @sharmilarama @tssushma @taohe1012 @P-Cao @shanmydell @baoy1 @YianZong @forrestxia
36+
* @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

‎.github/workflows/helm-validations.yml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,8 @@ jobs:
4141
# Lint and version increment checks on updated charts
4242
- name: Run lint/version increment checks
4343
id: versionIncrement
44-
uses: helm/chart-testing-action@v2.4.0
44+
# Bump to proper version after this issue is fixed : https://github.com/helm/chart-testing-action/issues/132
45+
uses: helm/chart-testing-action@cb49023b9227b1097e5eddd8824f48bdea11b1aa
4546
with:
4647
command: lint
4748
config: lintConfig.yaml
@@ -62,7 +63,7 @@ jobs:
6263
# Run a check on karavi-observability chart for updated dependency versions
6364
- name: Run dependency version checks
6465
id: lint
65-
uses: helm/chart-testing-action@v2.4.0
66+
uses: helm/chart-testing-action@cb49023b9227b1097e5eddd8824f48bdea11b1aa
6667
with:
6768
command: lint
6869
config: dependencyConfig.yaml

‎charts/container-storage-modules/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,4 +77,4 @@ dependencies:
7777
- name: cert-manager
7878
version: 1.10.0
7979
repository: https://charts.jetstack.io
80-
condition: cert-manager.enabled
80+
condition: cert-manager.enabled

‎charts/container-storage-modules/values.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -487,4 +487,4 @@ karavi-observability:
487487
## K8S/Cert-manager ATTRIBUTES
488488
########################
489489
cert-manager:
490-
enabled: false
490+
enabled: false

‎charts/csi-isilon/Chart.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
apiVersion: v2
22
name: csi-isilon
3-
version: 2.8.0
4-
appVersion: "2.8.0"
3+
version: 2.9.0
4+
appVersion: "2.9.0"
55
kubeVersion: ">= 1.21.0 < 1.29.0"
66
#If you are using a complex K8s version like "v1.22.3-mirantis-1", use this kubeVersion check instead
77
#WARNING: this version of the check will allow the use of alpha and beta versions, which is NOT SUPPORTED

‎charts/csi-isilon/templates/_helpers.tpl

Lines changed: 0 additions & 51 deletions
Original file line numberDiff line numberDiff line change
@@ -1,54 +1,3 @@
1-
{{/*
2-
Return the appropriate sidecar images based on k8s version
3-
*/}}
4-
{{- define "csi-isilon.attacherImage" -}}
5-
{{- if eq .Capabilities.KubeVersion.Major "1" }}
6-
{{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "21") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}}
7-
{{- print "registry.k8s.io/sig-storage/csi-attacher:v4.3.0" -}}
8-
{{- end -}}
9-
{{- end -}}
10-
{{- end -}}
11-
12-
{{- define "csi-isilon.provisionerImage" -}}
13-
{{- if eq .Capabilities.KubeVersion.Major "1" }}
14-
{{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "21") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}}
15-
{{- print "registry.k8s.io/sig-storage/csi-provisioner:v3.5.0" -}}
16-
{{- end -}}
17-
{{- end -}}
18-
{{- end -}}
19-
20-
{{- define "csi-isilon.snapshotterImage" -}}
21-
{{- if eq .Capabilities.KubeVersion.Major "1" }}
22-
{{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "21") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}}
23-
{{- print "registry.k8s.io/sig-storage/csi-snapshotter:v6.2.2" -}}
24-
{{- end -}}
25-
{{- end -}}
26-
{{- end -}}
27-
28-
{{- define "csi-isilon.resizerImage" -}}
29-
{{- if eq .Capabilities.KubeVersion.Major "1" }}
30-
{{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "21") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}}
31-
{{- print "registry.k8s.io/sig-storage/csi-resizer:v1.8.0" -}}
32-
{{- end -}}
33-
{{- end -}}
34-
{{- end -}}
35-
36-
{{- define "csi-isilon.registrarImage" -}}
37-
{{- if eq .Capabilities.KubeVersion.Major "1" }}
38-
{{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "21") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}}
39-
{{- print "registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.8.0" -}}
40-
{{- end -}}
41-
{{- end -}}
42-
{{- end -}}
43-
44-
{{- define "csi-isilon.healthmonitorImage" -}}
45-
{{- if eq .Capabilities.KubeVersion.Major "1" }}
46-
{{- if and (ge (trimSuffix "+" .Capabilities.KubeVersion.Minor) "21") (le (trimSuffix "+" .Capabilities.KubeVersion.Minor) "28") -}}
47-
{{- print "registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.9.0" -}}
48-
{{- end -}}
49-
{{- end -}}
50-
{{- end -}}
51-
521
{{/*
532
Return true if storage capacity tracking is enabled and is supported based on k8s version
543
*/}}

‎charts/csi-isilon/templates/controller.yaml

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ rules:
7272
verbs: ["get", "list", "watch", "update"]
7373
- apiGroups: ["snapshot.storage.k8s.io"]
7474
resources: ["volumesnapshots/status"]
75-
verbs: ["update"]
75+
verbs: ["update", "patch"]
7676
- apiGroups: ["snapshot.storage.k8s.io"]
7777
resources: ["volumesnapshotcontents/status"]
7878
verbs: ["update"]
@@ -194,7 +194,7 @@ spec:
194194
{{- if hasKey .Values "podmon" }}
195195
{{- if eq .Values.podmon.enabled true }}
196196
- name: podmon
197-
image: {{ required "Must provide the podmon container image." .Values.podmon.image }}
197+
image: {{ required "Must provide the podmon container image." .Values.images.podmon }}
198198
imagePullPolicy: {{ .Values.imagePullPolicy }}
199199
args:
200200
{{- toYaml .Values.podmon.controller.args | nindent 12 }}
@@ -221,7 +221,7 @@ spec:
221221
{{- if hasKey .Values.controller "replication" }}
222222
{{- if eq .Values.controller.replication.enabled true}}
223223
- name: dell-csi-replicator
224-
image: {{ required "Must provide the Dell CSI Replicator image." .Values.controller.replication.image}}
224+
image: {{ required "Must provide the Dell CSI Replicator image." .Values.images.replication }}
225225
imagePullPolicy: {{ .Values.imagePullPolicy }}
226226
args:
227227
- "--csi-address={{ $driverSockPath }}"
@@ -248,7 +248,7 @@ spec:
248248
{{- if hasKey .Values.controller "resizer" }}
249249
{{- if eq .Values.controller.resizer.enabled true }}
250250
- name: resizer{{ $csiSidecarSuffix }}
251-
image: {{ required "Must provide the CSI resizer container image." ( include "csi-isilon.resizerImage" . ) }}
251+
image: {{ required "Must provide the CSI resizer container image." .Values.images.resizer }}
252252
imagePullPolicy: {{ .Values.imagePullPolicy }}
253253
args:
254254
- "--csi-address={{ $driverSockPath }}"
@@ -272,7 +272,7 @@ spec:
272272
{{ end }}
273273
{{ end }}
274274
- name: csi-metadata-retriever {{ $csiSidecarSuffix }}
275-
image: {{ required "Must provide the CSI metadata retriever container image." .Values.controller.metadataretriever.image }}
275+
image: {{ required "Must provide the CSI metadata retriever container image." .Values.images.metadataretriever }}
276276
imagePullPolicy: {{ .Values.imagePullPolicy }}
277277
args:
278278
- "--csi-address={{ $driverSockPath }}"
@@ -298,7 +298,7 @@ spec:
298298
- name: socket-dir
299299
mountPath: /var/run/csi
300300
- name: attacher{{ $csiSidecarSuffix }}
301-
image: {{ required "Must provide the CSI attacher container image." ( include "csi-isilon.attacherImage" . ) }}
301+
image: {{ required "Must provide the CSI attacher container image." .Values.images.attacher }}
302302
imagePullPolicy: {{ .Values.imagePullPolicy }}
303303
args:
304304
- "--csi-address={{ $driverSockPath }}"
@@ -323,7 +323,7 @@ spec:
323323
{{- if hasKey .Values.controller "healthMonitor" }}
324324
{{- if eq .Values.controller.healthMonitor.enabled true }}
325325
- name: external-health-monitor-controller
326-
image: {{ required "Must provide the CSI external-health-monitor-controller container image." ( include "csi-isilon.healthmonitorImage" . ) }}
326+
image: {{ required "Must provide the CSI external-health-monitor-controller container image." .Values.images.healthmonitor }}
327327
imagePullPolicy: {{ .Values.imagePullPolicy }}
328328
args:
329329
- "--csi-address={{ $driverSockPath }}"
@@ -351,7 +351,7 @@ spec:
351351
{{end}}
352352
{{- end }}
353353
- name: provisioner{{ $csiSidecarSuffix }}
354-
image: {{ required "Must provide the CSI provisioner container image." ( include "csi-isilon.provisionerImage" . ) }}
354+
image: {{ required "Must provide the CSI provisioner container image." .Values.images.provisioner }}
355355
imagePullPolicy: {{ .Values.imagePullPolicy }}
356356
args:
357357
- "--csi-address={{ $driverSockPath }}"
@@ -393,7 +393,7 @@ spec:
393393
{{- if eq .Values.controller.snapshot.enabled true }}
394394
- name: snapshotter{{ $csiSidecarSuffix }}
395395
#image: quay.io/k8scsi/csi-snapshotter:v1.0.0
396-
image: {{ required "Must provide the CSI snapshotter container image." ( include "csi-isilon.snapshotterImage" . ) }}
396+
image: {{ required "Must provide the CSI snapshotter container image." .Values.images.snapshotter }}
397397
imagePullPolicy: {{ .Values.imagePullPolicy }}
398398
args:
399399
- "--csi-address={{ $driverSockPath }}"
@@ -419,7 +419,7 @@ spec:
419419
{{end}}
420420
{{- if not $encrypted }}
421421
- name: driver
422-
image: {{ required "Must provide the Isilon driver image repository." .Values.images.driverRepository }}/{{ .Chart.Name }}:{{ .Values.version }}
422+
image: {{ required "Must provide the Isilon driver image repository." .Values.images.driver }}
423423
imagePullPolicy: {{ .Values.imagePullPolicy }}
424424
command: [ "/csi-isilon" ]
425425
args:
@@ -519,7 +519,7 @@ spec:
519519
{{- if eq .Values.authorization.enabled true }}
520520
- name: karavi-authorization-proxy
521521
imagePullPolicy: {{ .Values.imagePullPolicy }}
522-
image: {{ required "Must provide the authorization sidecar container image." .Values.authorization.sidecarProxyImage }}
522+
image: {{ required "Must provide the authorization sidecar container image." .Values.images.authorization }}
523523
env:
524524
- name: PROXY_HOST
525525
value: "{{ .Values.authorization.proxyHost }}"
@@ -549,7 +549,7 @@ spec:
549549
{{- end }}
550550
{{- if $encrypted }}
551551
- name: driver-sec
552-
image: {{ .Values.encryption.image }}
552+
image: {{ .Values.images.encryption }}
553553
imagePullPolicy: {{ .Values.imagePullPolicy }}
554554
args:
555555
- --name={{ .Values.encryption.pluginName }}

‎charts/csi-isilon/templates/node.yaml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ spec:
113113
capabilities:
114114
add: ["SYS_ADMIN"]
115115
allowPrivilegeEscalation: true
116-
image: {{ required "Must provide the podmon container image." .Values.podmon.image }}
116+
image: {{ required "Must provide the podmon container image." .Values.images.podmon }}
117117
imagePullPolicy: {{ .Values.imagePullPolicy }}
118118
args:
119119
{{- toYaml .Values.podmon.node.args | nindent 12 }}
@@ -167,7 +167,7 @@ spec:
167167
capabilities:
168168
add: ["SYS_ADMIN"]
169169
allowPrivilegeEscalation: true
170-
image: {{ required "Must provide the Isilon driver image repository." .Values.images.driverRepository }}/{{ .Chart.Name }}:{{ .Values.version }}
170+
image: {{ required "Must provide the Isilon driver image repository." .Values.images.driver }}
171171
imagePullPolicy: {{ .Values.imagePullPolicy }}
172172
env:
173173
- name: CSI_ENDPOINT
@@ -251,7 +251,7 @@ spec:
251251
mountPath: /csi-isilon-config-params
252252
{{- end }}
253253
- name: registrar{{ $csiSidecarSuffix }}
254-
image: {{ required "Must provide the CSI node registrar container image." ( include "csi-isilon.registrarImage" . ) }}
254+
image: {{ required "Must provide the CSI node registrar container image." .Values.images.registrar }}
255255
imagePullPolicy: {{ .Values.imagePullPolicy }}
256256
args:
257257
- "--v=5"
@@ -273,7 +273,7 @@ spec:
273273
{{- if eq .Values.authorization.enabled true }}
274274
- name: karavi-authorization-proxy
275275
imagePullPolicy: {{ .Values.imagePullPolicy }}
276-
image: {{ required "Must provide the authorization sidecar container image." .Values.authorization.sidecarProxyImage }}
276+
image: {{ required "Must provide the authorization sidecar container image." .Values.images.authorization }}
277277
env:
278278
- name: PROXY_HOST
279279
value: "{{ .Values.authorization.proxyHost }}"
@@ -303,7 +303,7 @@ spec:
303303
{{- end }}
304304
{{- if $encrypted }}
305305
- name: driver-sec
306-
image: {{ .Values.encryption.image }}
306+
image: {{ .Values.images.encryption }}
307307
imagePullPolicy: {{ .Values.imagePullPolicy }}
308308
securityContext:
309309
privileged: true

0 commit comments

Comments
 (0)