Skip to content

Commit

Permalink
[bitnami/external-dns] ability to disable validation
Browse files Browse the repository at this point in the history
There are some uses cases where the chart value validation should not
be ran, this provides a way to disable it.

fixes bitnami#19942

Signed-off-by: Jon Whitcraft <[email protected]>
  • Loading branch information
jwhitcraft committed Oct 10, 2023
1 parent d67f91a commit 3fc0ad5
Show file tree
Hide file tree
Showing 4 changed files with 15 additions and 2 deletions.
2 changes: 1 addition & 1 deletion bitnami/external-dns/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -28,4 +28,4 @@ maintainers:
name: external-dns
sources:
- https://github.com/bitnami/charts/tree/main/bitnami/external-dns
version: 6.26.4
version: 6.26.5
1 change: 1 addition & 0 deletions bitnami/external-dns/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -349,6 +349,7 @@ The command removes all the Kubernetes components associated with the chart and
| `metrics.googlePodMonitor.namespace` | Namespace in which PodMonitoring created | `""` |
| `metrics.googlePodMonitor.interval` | Interval at which metrics should be scraped by Google Managed Prometheus | `60s` |
| `metrics.googlePodMonitor.endpoint` | The endpoint for Google Managed Prometheus scraping the metrics | `/metrics` |
| `validation.enabled` | Enable chart validation | `true` |

Specify each parameter using the `--set key=value[,key=value]` argument to `helm install`. For example,

Expand Down
6 changes: 5 additions & 1 deletion bitnami/external-dns/templates/_helpers.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -262,8 +262,9 @@ compartment: {{ .Values.oci.compartmentOCID }}
{{ end }}

{{/*
Compile all warnings into a single message, and call fail.
Compile all warnings into a single message, and call fail if the validation is enabled
*/}}
{{- if .Values.validation.enabled -}}
{{- define "external-dns.validateValues" -}}
{{- $messages := list -}}
{{- $messages := append $messages (include "external-dns.validateValues.provider" .) -}}
Expand Down Expand Up @@ -312,6 +313,9 @@ Compile all warnings into a single message, and call fail.
{{- printf "\nVALUES VALIDATION:\n%s" $message | fail -}}
{{- end -}}
{{- end -}}
{{- end -}}

{{/*
{{/*
Validate values of External DNS:
Expand Down
8 changes: 8 additions & 0 deletions bitnami/external-dns/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -1110,3 +1110,11 @@ metrics:
## @param metrics.googlePodMonitor.endpoint The endpoint for Google Managed Prometheus scraping the metrics
##
endpoint: /metrics


## Chart Validation
##
validation:
## @param validation.enabled Enable chart validation
##
enabled: true

0 comments on commit 3fc0ad5

Please sign in to comment.