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

Add support for external postgress server in values chart #7688

Open
mpsOxygen opened this issue Apr 10, 2024 · 4 comments
Open

Add support for external postgress server in values chart #7688

mpsOxygen opened this issue Apr 10, 2024 · 4 comments
Labels
kind/proposal An issue that reports a new feature proposal to be discussed

Comments

@mpsOxygen
Copy link

Summary
It would be very useful to add support in the chart to configure an external postgress db. For example I use cloudnative-pg to provision postgress DBs in my cluster.

Background and rationale
The cloudnative-pg operator does a way better job of managing postgress instances in kubernetes than the bitnami chart.

Description
In the values file I would like to have the option to disable the postgress datbase and specify data for an external postgress server.

Acceptance criteria
Being able to install the chart with an external database.

@mpsOxygen mpsOxygen added the kind/proposal An issue that reports a new feature proposal to be discussed label Apr 10, 2024
@github-project-automation github-project-automation bot moved this to 🗂 Backlog in Kubeapps Apr 10, 2024
@antgamdia
Copy link
Contributor

Hi @mpsOxygen , thanks for creating the proposal.

From memory, I'd say we already have support for external databases in our chart. You can set both externalDatabase.port and externalDatabase.host having set postgresql.enabled=false.

Please give it a try (have a look at the chart templates, helper.tpl file, etc) and let us know if it works for you.
Also, please also feel free to contribute and send a PR with an updated chart (in the bitnami/charts repo, so that it get published sooner).

@antgamdia antgamdia added the awaiting-more-evidence Need more info to actually get it done. label Apr 10, 2024
@antgamdia antgamdia added this to the Community requests milestone Apr 10, 2024
@mpsOxygen
Copy link
Author

I have tried it like that but it still creates the postgres pod even with postgresql.enabled=false and host and port set.

@antgamdia
Copy link
Contributor

Ah, that's a pity :S We usually don't test our Kubeapps chart against external databases, so, likely, the support for it has been accidentally removed when refactoring the chart some time ago.

As said, please feel free to contribute and send a PR to the repo and thanks for reporting the issue!

@antgamdia antgamdia removed the awaiting-more-evidence Need more info to actually get it done. label Apr 10, 2024
@brunomedeirosdedalus
Copy link

As far as I know the variables needed to setup the external DB are not available on the values.yaml. I have made a PR for it #7743, I have some more ideas on how to improve this part, if there is any interest on it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/proposal An issue that reports a new feature proposal to be discussed
Projects
Status: 🗂 Backlog
Development

No branches or pull requests

3 participants