Skip to content

Commit

Permalink
Merge pull request #1074 from woehrl01/optional_helm_hooks
Browse files Browse the repository at this point in the history
feat: allow to don't use helm hooks for CSIDriver creation
  • Loading branch information
k8s-ci-robot authored Sep 24, 2024
2 parents 3886bd8 + 1d5c39c commit 1e9c794
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
2 changes: 2 additions & 0 deletions charts/aws-efs-csi-driver/templates/csidriver.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,10 @@ kind: CSIDriver
metadata:
name: efs.csi.aws.com
annotations:
{{- if .Values.useHelmHooksForCSIDriver }}
"helm.sh/hook": pre-install, pre-upgrade
"helm.sh/hook-delete-policy": before-hook-creation
{{- end }}
"helm.sh/resource-policy": keep
spec:
attachRequired: false
7 changes: 5 additions & 2 deletions charts/aws-efs-csi-driver/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -114,7 +114,6 @@ controller:
leaderElectionRenewDeadline: 10s
leaderElectionLeaseDuration: 15s


## Node daemonset variables

node:
Expand Down Expand Up @@ -154,7 +153,8 @@ node:
# cpu: 100m
# memory: 128Mi
nodeSelector: {}
updateStrategy: {}
updateStrategy:
{}
# Override default strategy (RollingUpdate) to speed up deployment.
# This can be useful if helm timeouts are observed.
# type: OnDelete
Expand Down Expand Up @@ -208,3 +208,6 @@ storageClasses: []
# ensureUniqueDirectory: true
# reclaimPolicy: Delete
# volumeBindingMode: Immediate

# Specifies wether to use helm hooks to apply the CSI driver
useHelmHooksForCSIDriver: true

0 comments on commit 1e9c794

Please sign in to comment.