Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support overriding the --tls-min-version controller manager flag in the Helm chart #2288

Closed
cbuto opened this issue Sep 26, 2022 · 0 comments · Fixed by #2289
Closed

Support overriding the --tls-min-version controller manager flag in the Helm chart #2288

cbuto opened this issue Sep 26, 2022 · 0 comments · Fixed by #2289
Labels
enhancement New feature or request

Comments

@cbuto
Copy link

cbuto commented Sep 26, 2022

Describe the solution you'd like
The --tls-min-version flag was added in #1424; however, there is not an easy way to set this flag using the gatekeeper helm chart.

I'd like to be able to pass in a value to the Helm chart to set the flag to --tls-min-version=1.2.

Anything else you would like to add:
Instead of adding the option directly in the args, should the chart support passing any arbitrary arguments (extra args) to the deployment?

i.e.

{{- if .Values.extraArgs }}
{{ toYaml .Values.extraArgs | indent 8 }}
{{- end }}

Environment:

  • Gatekeeper version: 3.8.1
  • Kubernetes version: (use kubectl version):
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant