apache / couchdb-helm

Apache CouchDB Helm Chart
https://couchdb.apache.org/
Apache License 2.0
49 stars 64 forks source link

Support IngressClass #69

Closed discostur closed 1 year ago

discostur commented 2 years ago

Hi,

would it be possible to support setting a IngressClass?

Thanks Regards, Kilian

discostur commented 2 years ago

@kocolosk it seems that you have released a new version of the helm chart which should contain this merge / fix - but if i download the helm chart i cannot see the changes.

I cannot see any merge in your master branch, too. Did there anything go wrong with your release?

edit: might be related to https://github.com/apache/couchdb-helm/issues/88

willholley commented 2 years ago

@discostur I think this is blocked on merging #75 - when the CI issues are resolved we can look at merging/releasing.

broomfn commented 2 years ago

In case it's of any use to anyone, I used the following command to patch the ingress:

kubectl patch ingress/couch-couchdb --type json --patch '[{"op": "add", "path": "/spec/ingressClassName", value: "nginx" }]'