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

Make cert-manager dependency optional #1922

Open
avishefi opened this issue May 31, 2022 · 0 comments
Open

Make cert-manager dependency optional #1922

avishefi opened this issue May 31, 2022 · 0 comments
Labels
help wanted Extra attention is needed

Comments

@avishefi
Copy link

avishefi commented May 31, 2022

Requirement - what kind of business use case are you trying to solve?

Remove non-strict dependency on cert-manager, have the option to pass a tls certificate secret name instead of creating Issuer and Certificate and fix the documentation to reflect that.

Proposal - what do you suggest to solve the problem or improve the existing situation?

Make cert-manager kustomize resources optional, and allow disabling them with a value and pass the tls secret name instead which will be used for the webhooks.

The following PR for the jaeger-operator Helm chart already makes the dependency on cert-manager optional by providing a way to disable the creation of Issuer and Certificate and passing a tls certificate secret name instead: jaegertracing/helm-charts#368

@rubenvp8510 rubenvp8510 added the help wanted Extra attention is needed label Jun 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants