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

fix(helm): Add ClusterOriginIssuers to approver ClusterRole #132

Closed
wants to merge 1 commit into from

Conversation

xorinzor
Copy link
Contributor

cert-manager was unable to process the Approval action of a certificate when using ClusterOriginIssuer, the pod was showing:

 1 conditions.go:263] Setting lastTransitionTime for CertificateRequest "test-tls-1" condition "Approved" to 2024-07-10 08:57:28.083240782 +0000 UTC m=+600872.378883246
E0710 08:57:28.093648       1 controller.go:162] "re-queuing item due to error processing" err="admission webhook \"webhook.cert-manager.io\" denied the request: status.conditions: Forbidden: user \"system:serviceaccount:cert-manager:cert-manager\" does not have permissions to set approved/denied conditions for issuer {cf-cluster-origin-issuer ClusterOriginIssuer cert-manager.k8s.cloudflare.com}" logger="cert-manager.controller" key="my-site/test-tls-1"

Turns out the ClusterRole manifest in the helm chart was missing the clusteroriginissuers.cert-manager.k8s.cloudflare.com/* resourceName, adding it fixed the issue.

@terinjokes
Copy link
Contributor

You'll need to bump the chart version too.

@xorinzor
Copy link
Contributor Author

good call, done!

@terinjokes
Copy link
Contributor

dankjewel!

@xorinzor xorinzor deleted the fix/approver-role branch July 10, 2024 12:45
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

Successfully merging this pull request may close these issues.

None yet

2 participants