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

[HELM] - Add guide how to PgBouncer with Kubernetes Secret #42460

Merged
merged 2 commits into from
Oct 2, 2024

Conversation

andormarkus
Copy link
Contributor

Hi All,

Do company polices we can not store secrets in the values.yaml and we need to use Kubernetes Secrets. These requirements made the PgBouncer implementation ver challenging because the the helm chart is able to handle external database until it is fully defined in the values.yaml however if it is defined the Kubernetes Secret than the PgBouncer does not pick up these changes and wan't to connect to the local Postgres DB which is not deployed because external database was used.

To be able to PgBouncer work we need to create two Kubernetes Secrets manually and reference is in the chart.

I have extended the helm documentation if someone else in the future can have countless hours on deployment

@potiuk
Copy link
Member

potiuk commented Oct 2, 2024

Nice!

@potiuk potiuk merged commit 835ff27 into apache:main Oct 2, 2024
40 checks passed
joaopamaral pushed a commit to joaopamaral/airflow that referenced this pull request Oct 21, 2024
)

* feat: Add guide how to PgBouncer with Kubernetes Secret

* feat: Add guide how to PgBouncer with Kubernetes Secret

---------

Co-authored-by: Andor Markus (AllCloud) <andor.rudolf-markus@glassbox.com>
ellisms pushed a commit to ellisms/airflow that referenced this pull request Nov 13, 2024
)

* feat: Add guide how to PgBouncer with Kubernetes Secret

* feat: Add guide how to PgBouncer with Kubernetes Secret

---------

Co-authored-by: Andor Markus (AllCloud) <andor.rudolf-markus@glassbox.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants