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

Pgbouncer auth type cannot be configured #19654

Closed
1 of 2 tasks
freget opened this issue Nov 17, 2021 · 3 comments · Fixed by #21999
Closed
1 of 2 tasks

Pgbouncer auth type cannot be configured #19654

freget opened this issue Nov 17, 2021 · 3 comments · Fixed by #21999
Labels
area:helm-chart Airflow Helm Chart kind:feature Feature Requests

Comments

@freget
Copy link
Contributor

freget commented Nov 17, 2021

Official Helm Chart version

1.3.0 (latest released)

Apache Airflow version

2.2.1

Kubernetes Version

1.20.9

Helm Chart configuration

No response

Docker Image customisations

No response

What happened

The default pgbouncer config generated by the helm chart looks something like

[databases]
...

[pgbouncer]
pool_mode = transaction
listen_port = 6543
listen_addr = *
auth_type = md5
auth_file = /etc/pgbouncer/users.txt
stats_users = <user>
ignore_startup_parameters = extra_float_digits
max_client_conn = 200
verbose = 0
log_disconnections = 0
log_connections = 0

server_tls_sslmode = require
server_tls_ciphers = normal

If the database to connect against is Azure Postgresl, the auth type md5 does not seem to be supported. Hence, we need to add a configuration flag to change the auth type to something else in the helm chart.

What you expected to happen

No response

How to reproduce

No response

Anything else

No response

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Code of Conduct

@freget freget added area:helm-chart Airflow Helm Chart kind:bug This is a clearly a bug labels Nov 17, 2021
@freget
Copy link
Contributor Author

freget commented Nov 17, 2021

kind:feature would probably be the better tag, but I'm not able to change this (or I do not find how).

@potiuk potiuk added kind:feature Feature Requests and removed kind:bug This is a clearly a bug labels Nov 17, 2021
@potiuk
Copy link
Member

potiuk commented Nov 17, 2021

changed it

@potiuk
Copy link
Member

potiuk commented Nov 17, 2021

(BTW. In the future - you can choose "new feature" from the issue types when creating new issue @freget )

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 kind:feature Feature Requests
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants