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

Support existing claims for redis and postgresql #331

Open
justsomebody42 opened this issue Jul 19, 2023 · 4 comments
Open

Support existing claims for redis and postgresql #331

justsomebody42 opened this issue Jul 19, 2023 · 4 comments
Labels
enhancement Adding or requesting a new feature.

Comments

@justsomebody42
Copy link

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

@nijel
Copy link
Member

nijel commented Jul 20, 2023

There is already support for using external services, see #317 or #116.

@justsomebody42
Copy link
Author

justsomebody42 commented Jul 20, 2023

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
Copy link

github-actions bot commented Aug 4, 2023

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 github-actions bot added the wontfix Nobody will work on this. label Aug 4, 2023
@nijel nijel added enhancement Adding or requesting a new feature. and removed wontfix Nobody will work on this. labels Aug 4, 2023
@Mart-Kuc
Copy link
Contributor

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Adding or requesting a new feature.
Projects
None yet
Development

No branches or pull requests

3 participants