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.

Sign up for GitHub

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

Jump to bottom

feat: add pgbouncer.authType value #498

Merged
merged 1 commit into from
Apr 1, 2022
Merged

feat: add pgbouncer.authType value #498

merged 1 commit into from
Apr 1, 2022

Conversation

thesuperzapper
Copy link
Member

@thesuperzapper thesuperzapper commented Jan 10, 2022 •
edited

What issues does your PR fix?

What does your PR do?

  • Adds the pgbouncer.authType value (default: md5) to set the auth_type config of PgBouncer.
    • NOTE: this is important for "Azure PostgreSQL" as the default md5 will not work, but scram-sha-256 will

Checklist

For all Pull Requests

For releasing ONLY

@thesuperzapper thesuperzapper added this to Unsorted in Issue Triage and PR Tracking via automation Jan 10, 2022
@thesuperzapper thesuperzapper moved this from Unsorted to PR | Ready to Merge in Issue Triage and PR Tracking Jan 10, 2022
@thesuperzapper thesuperzapper added this to the airflow-8.6.0 milestone Jan 10, 2022
@stale stale bot added the lifecycle/stale lifecycle - this is stale label Mar 12, 2022
@stale stale bot closed this Mar 19, 2022
Issue Triage and PR Tracking automation moved this from PR | Ready to Merge to Done Mar 19, 2022
Issue Triage and PR Tracking automation moved this from Done to Unsorted Mar 21, 2022
@stale stale bot removed the lifecycle/stale lifecycle - this is stale label Mar 21, 2022
@thesuperzapper thesuperzapper moved this from Unsorted to PR | Ready to Merge in Issue Triage and PR Tracking Mar 21, 2022
@thesuperzapper thesuperzapper added the status/ready-to-merge status - this will be merged into next release label Mar 22, 2022
@airflow-helm airflow-helm deleted a comment from stale bot Mar 22, 2022
@thesuperzapper
feat: add pgbouncer.authType value
4fd9454 
Signed-off-by: Mathew Wicks <thesuperzapper@users.noreply.github.com>
@thesuperzapper thesuperzapper merged commit df07e28 into airflow-helm:main Apr 1, 2022
Issue Triage and PR Tracking automation moved this from PR | Ready to Merge to Done Apr 1, 2022
@thesuperzapper thesuperzapper deleted the pgbouncer-auth-type branch April 1, 2022 08:28
@thesuperzapper thesuperzapper mentioned this pull request Apr 3, 2022
Closed
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gsemet gsemet Awaiting requested review from gsemet

Assignees
No one assigned
Labels
status/ready-to-merge status - this will be merged into next release
Projects
Issue Triage and PR Tracking
Done
Milestone
airflow-8.6.0
Development

Successfully merging this pull request may close these issues.

add pgbouncer.authType value to set auth_type (for Azure PostgreSQL) PgBouncer fails with Azure PostgreSQL
1 participant
@thesuperzapper