Skip to content

Commit

Permalink
Removed unused properties from chart defaults. When webhook patching …
Browse files Browse the repository at this point in the history
…is disabled, the other properties are unused. For example, `{{- with .Values.admissionWebhooks.patch.tolerations }}` is nested within `{{- if and .Values.admissionWebhooks.enabled .Values.admissionWebhooks.patch.enabled (not .Values.admissionWebhooks.certManager.enabled) }}`.
  • Loading branch information
mattwelke committed Jun 21, 2024
1 parent 42c3697 commit c03f4b6
Showing 1 changed file with 2 additions and 9 deletions.
11 changes: 2 additions & 9 deletions packs/oam-app-controller-0.0.4/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -164,18 +164,11 @@ charts:
certificate:
mountPath: /etc/k8s-webhook-certs
patch:
enabled: true
image:
repository: cgr.dev/chainguard/kube-webhook-certgen
tag: latest
pullPolicy: IfNotPresent
nodeSelector: {}
affinity: {}
tolerations: []
enabled: false
appConversion:
enabled: false
certManager:
enabled: false
enabled: true
revisionHistoryLimit: 3

## @param kubeClient.qps The qps for reconcile clients
Expand Down

0 comments on commit c03f4b6

Please sign in to comment.