Skip to content

Conversation

@dnskr
Copy link
Contributor

@dnskr dnskr commented Jan 28, 2023

The PR changes default pgbouncer.auth_type value to scram-sha-256.
It fixes the case when installing the chart with enabled pgbouncer, i.e. the following deployment:

helm install test chart --set pgbouncer.enabled=true

The issue was introduced when bitnami/postgres subchart was upgraded to 12.1.9 version. Previous bitnami/postgres subchart deploys Postgres 11 with default auth type md5, current version deploys Postgres 15 with default auth type scram-sha-256.

related: #29071 and #29207

@boring-cyborg boring-cyborg bot added the area:helm-chart Airflow Helm Chart label Jan 28, 2023
@dnskr dnskr force-pushed the complete_bitnami_postgresql_upgrade branch from 0a64f47 to 6aebcab Compare January 28, 2023 22:07
@potiuk
Copy link
Member

potiuk commented Jan 29, 2023

Needs tests fix as well.

@dnskr dnskr force-pushed the complete_bitnami_postgresql_upgrade branch from 6aebcab to ecc0c80 Compare January 29, 2023 11:44
@dnskr dnskr marked this pull request as draft January 30, 2023 19:14
@dnskr
Copy link
Contributor Author

dnskr commented Jan 30, 2023

Closed in favor of pinning Postgres 11 version for the helm chart 1.8.0, see #29207.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:helm-chart Airflow Helm Chart

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants