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

Missing webhook-server-cert Secret when installed with Helm #132

Closed
vinzent opened this issue Apr 22, 2022 · 1 comment
Closed

Missing webhook-server-cert Secret when installed with Helm #132

vinzent opened this issue Apr 22, 2022 · 1 comment

Comments

@vinzent
Copy link
Contributor

vinzent commented Apr 22, 2022

Affected Helm chart version: 1.3.9

Detected while upgrading from 1.3.0

When installing with helm, the kube-rbac proxy container wants to mount a webhook-server-cert Secret. Such a secret is not created by Helm.

https://github.com/redhat-cop/cert-utils-operator/blob/master/config/helmchart/templates/manager.yaml#L57-L60

Seems there is a toggle to enableCertManager that will provision this cert by cert-manager.

https://github.com/redhat-cop/cert-utils-operator/blob/master/config/helmchart/templates/certificate.yaml#L1

In https://github.com/redhat-cop/cert-utils-operator/blob/master/config/helmchart/values.yaml.tpl I can't find a reference for that variable.

As there is a secret called webhook-server-cert and RBAC roles allowing {mutating,validating}webhookconfigurations but there is no MutatingWebhookConfiguration or ValidatingWebhookConfiguration rendered by the helm chart: is it missing or will there be some webhooks in the future?

@raffaelespazzoli
Copy link
Contributor

the enableCertManager variable is not documented, but you can use it. I'll add it to the value file
the webhooks at the momnent are missing but we create the secret anyway for better alignment with the other operators.

raffaelespazzoli added a commit that referenced this issue May 28, 2022
Signed-off-by: raffaelespazzoli <raffaele.spazzoli@gmail.com>
davgordo pushed a commit to davgordo/cert-utils-operator that referenced this issue Apr 25, 2023
Signed-off-by: raffaelespazzoli <raffaele.spazzoli@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants