Attila Fabian
Attila Fabian
@chavacava I'd be happy contribute these updates to the docs.
https://github.com/mgechev/revive/pull/750
@fedepaol I'd be happy to contribute this. Just to double check: we want to provide a configuration option that controls whether the [`metallb-webhook-configuration` resource](https://github.com/metallb/metallb/blob/main/charts/metallb/templates/webhooks.yaml) is deployed or not, correct? Do...
>or an alternative method is to drive the failure policy of the webhook Sounds good to me! If we go this path, how about calling the config option `permissiveWebhooks` or...
I can open a PR for this later today, if that's fine with you.
PR opened: https://github.com/metallb/metallb/pull/1623
# test: default value helm install --dry-run --debug metallb . ``` install.go:192: [debug] Original chart version: "" install.go:209: [debug] CHART PATH: /Users/attiss/git/metallb/charts/metallb NAME: metallb LAST DEPLOYED: Mon Oct 3 19:34:10...
# test: `Fail` helm install --dry-run --debug --set crds.validationFailurePolicy=Fail metallb . ``` install.go:192: [debug] Original chart version: "" install.go:209: [debug] CHART PATH: /Users/attiss/git/metallb/charts/metallb NAME: metallb LAST DEPLOYED: Mon Oct 3...
# test: `Ignore` helm install --dry-run --debug --set crds.validationFailurePolicy=Ignore metallb . ``` install.go:192: [debug] Original chart version: "" install.go:209: [debug] CHART PATH: /Users/attiss/git/metallb/charts/metallb NAME: metallb LAST DEPLOYED: Mon Oct 3...
# test: invalid value helm install --dry-run --debug --set crds.validationFailurePolicy=Invalid metallb . ``` install.go:192: [debug] Original chart version: "" install.go:209: [debug] CHART PATH: /Users/attiss/git/metallb/charts/metallb Error: INSTALLATION FAILED: values don't meet...