Skip to content

Commit

Permalink
Adding dependabot compatibility for csi-unity (#508)
Browse files Browse the repository at this point in the history
* Adding dependabot compatibility for csi-unity

* removing trailing spaces
  • Loading branch information
sakshi-garg1 authored and shaynafinocchiaro committed Oct 30, 2024
1 parent 508cf40 commit 17a22ee
Show file tree
Hide file tree
Showing 3 changed files with 26 additions and 18 deletions.
14 changes: 7 additions & 7 deletions charts/csi-unity/templates/controller.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -154,7 +154,7 @@ spec:
{{- if .Values.podmon.enabled }}
- name: podmon
imagePullPolicy: {{ .Values.imagePullPolicy }}
image: {{ required "Must provide the podmon container image." .Values.images.podmon }}
image: {{ required "Must provide the podmon container image." .Values.images.podmon.image }}
args:
{{- toYaml .Values.podmon.controller.args | nindent 12 }}
env:
Expand All @@ -177,7 +177,7 @@ spec:
mountPath: /unity-config
{{- end }}
- name: attacher
image: {{ required "Must provide the CSI attacher container image." .Values.images.attacher }}
image: {{ required "Must provide the CSI attacher container image." .Values.images.attacher.image }}
args:
- "--csi-address=$(ADDRESS)"
- "--v=5"
Expand All @@ -189,7 +189,7 @@ spec:
- name: socket-dir
mountPath: /var/run/csi
- name: provisioner
image: {{ required "Must provide the CSI provisioner container image." .Values.images.provisioner }}
image: {{ required "Must provide the CSI provisioner container image." .Values.images.provisioner.image }}
args:
- "--csi-address=$(ADDRESS)"
- "--volume-name-prefix={{ required "Must provide a Volume Name Prefix." .Values.controller.volumeNamePrefix }}"
Expand Down Expand Up @@ -222,7 +222,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. " .Values.images.snapshotter }}
image: {{ required "Must provide the CSI snapshotter container image. " .Values.images.snapshotter.image }}
args:
- "--csi-address=$(ADDRESS)"
- "--snapshot-name-prefix={{ required "Must privided a Snapshot Name Prefix" .Values.controller.snapshot.snapNamePrefix }}"
Expand All @@ -241,7 +241,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." .Values.images.resizer }}
image: {{ required "Must provide the CSI resizer container image." .Values.images.resizer.image }}
args:
- "--csi-address=$(ADDRESS)"
- "--v=5"
Expand All @@ -257,7 +257,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." .Values.images.healthmonitor }}
image: {{ required "Must provide the CSI external health monitor image." .Values.images.healthmonitor.image }}
imagePullPolicy: {{ .Values.imagePullPolicy }}
args:
- "--v=5"
Expand All @@ -276,7 +276,7 @@ spec:
{{- end }}
{{- end }}
- name: driver
image: "{{ required "Must provide the driver image repository." .Values.images.driver }}"
image: "{{ required "Must provide the driver image repository." .Values.images.driver.image }}"
args:
- "--driver-name=csi-unity.dellemc.com"
- "--driver-config=/unity-config/driver-config-params.yaml"
Expand Down
6 changes: 3 additions & 3 deletions charts/csi-unity/templates/node.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,7 @@ spec:
add: ["SYS_ADMIN"]
allowPrivilegeEscalation: true
imagePullPolicy: {{ .Values.imagePullPolicy }}
image: {{ required "Must provide the podmon container image." .Values.images.podmon }}
image: {{ required "Must provide the podmon container image." .Values.images.podmon.image }}
args:
{{- toYaml .Values.podmon.node.args | nindent 12 }}
env:
Expand Down Expand Up @@ -147,7 +147,7 @@ spec:
capabilities:
add: ["SYS_ADMIN"]
allowPrivilegeEscalation: true
image: "{{ required "Must provide the driver image repository." .Values.images.driver }}"
image: "{{ required "Must provide the driver image repository." .Values.images.driver.image }}"
args:
- "--driver-name=csi-unity.dellemc.com"
- "--driver-config=/unity-config/driver-config-params.yaml"
Expand Down Expand Up @@ -208,7 +208,7 @@ spec:
- name: unity-secret
mountPath: /unity-secret
- name: registrar
image: {{ required "Must provide the CSI registrar container image." .Values.images.registrar }}
image: {{ required "Must provide the CSI registrar container image." .Values.images.registrar.image }}
args:
- "--v=5"
- "--csi-address=$(ADDRESS)"
Expand Down
24 changes: 16 additions & 8 deletions charts/csi-unity/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,17 +8,25 @@ version: "v2.12.0"

images:
# "driver" defines the container image, used for the driver container.
driver: dellemc/csi-unity:v2.12.0
driver:
image: dellemc/csi-unity:v2.12.0
# CSI sidecars
attacher: registry.k8s.io/sig-storage/csi-attacher:v4.6.1
provisioner: registry.k8s.io/sig-storage/csi-provisioner:v5.0.1
snapshotter: registry.k8s.io/sig-storage/csi-snapshotter:v8.0.1
resizer: registry.k8s.io/sig-storage/csi-resizer:v1.11.1
registrar: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.10.1
healthmonitor: registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.12.1
attacher:
image: registry.k8s.io/sig-storage/csi-attacher:v4.6.1
provisioner:
image: registry.k8s.io/sig-storage/csi-provisioner:v5.0.1
snapshotter:
image: registry.k8s.io/sig-storage/csi-snapshotter:v8.0.1
resizer:
image: registry.k8s.io/sig-storage/csi-resizer:v1.11.1
registrar:
image: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.10.1
healthmonitor:
image: registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.12.1

# CSM sidecars
podmon: dellemc/podmon:v1.10.0
podmon:
image: dellemc/podmon:v1.10.0

# LogLevel is used to set the logging level of the driver.
# Allowed values: "error", "warn"/"warning", "info", "debug"
Expand Down

0 comments on commit 17a22ee

Please sign in to comment.