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

Drop / enable explicitly cert-manager dep in operator helm chart #1206

Open
pschiffe opened this issue Jan 13, 2025 · 0 comments
Open

Drop / enable explicitly cert-manager dep in operator helm chart #1206

pschiffe opened this issue Jan 13, 2025 · 0 comments
Labels
bug Something isn't working

Comments

@pschiffe
Copy link

Currently there's a hard dep on cert-manager helm chart when operator is instructed to use cert-manager's CRD.

Cert-manager is widely used and usually already present in the cluster, installed according to the needs of the cluster. Also the cert-manager version specified in https://github.com/OT-CONTAINER-KIT/redis-operator/blob/master/charts/redis-operator/Chart.yaml#L25 is outdated.

It would be great if the redis operator could create cert-manager certificate without installing cert-manager. For example by either dropping cert-manager dep from Chart.yaml or using different condition variable, such as certmanager.install or similar.

Related to #784

redis-operator version: 0.19.0

@pschiffe pschiffe added the bug Something isn't working label Jan 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant