WeblateOrg / helm

Helm Chart for Weblate
https://docs.weblate.org/en/latest/admin/install/kubernetes.html
Apache License 2.0
31 stars 41 forks source link

Support existing claims for redis and postgresql #331

Open justsomebody42 opened 1 year ago

justsomebody42 commented 1 year ago

Describe the problem

Both redis and postgresql charts support the configuration of an existing claim, but it seems to be impossible to use this with the weblate helm chart

Describe the solution you'd like

Add support for existingClaims for both redis and postgresql subcharts

Describe alternatives you've considered

No response

Screenshots

No response

Additional context

No response

nijel commented 1 year ago

There is already support for using external services, see https://github.com/WeblateOrg/helm/pull/317 or https://github.com/WeblateOrg/helm/pull/116.

justsomebody42 commented 1 year ago

Hi @nijel, thanks for the note. I was looking to be able to deploy the full stack (weblate + postgresql + redis) with a single chart, rather than setting up each service individually. Wouldn't it be possible and suffice to expose the values from the subcharts to this chart?

github-actions[bot] commented 1 year ago

This issue has been automatically marked as stale because there wasn’t any recent activity.

It will be closed soon if no further action occurs.

Thank you for your contributions!

Mart-Kuc commented 1 month ago

Hi @justsomebody42, you can use existing claim in values.yaml. For instance if you want to mount existing PVC for Redis, you can add master.persistence.existingClaim in values.yaml how you mentioned in your Issue. All values for Upstream (redis/postgres) helm charts are taking.

Feel free to correct me if I misunderstood.