helm icon indicating copy to clipboard operation
helm copied to clipboard

Support existing claims for redis and postgresql

Open justsomebody42 opened this issue 2 years ago • 4 comments

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

  • Postgresql: primary.persistence.existingClaim
  • Redis: master.persistence.existingClaim

Describe alternatives you've considered

No response

Screenshots

No response

Additional context

No response

justsomebody42 avatar Jul 19 '23 11:07 justsomebody42

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

nijel avatar Jul 20 '23 07:07 nijel

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?

justsomebody42 avatar Jul 20 '23 07:07 justsomebody42

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!

github-actions[bot] avatar Aug 04 '23 01:08 github-actions[bot]

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.

Mart-Kuc avatar Sep 24 '24 15:09 Mart-Kuc