search

airflow-helm / charts

The User-Community Airflow Helm Chart is the standard way to deploy Apache Airflow on Kubernetes with Helm. Originally created in 2017, it has since helped thousands of companies create production-ready deployments of Airflow on Kubernetes.
https://github.com/airflow-helm/charts/tree/main/charts/airflow
Apache License 2.0
647 stars 475 forks source link

feat: add `pgbouncer.statsUsers` value to set `stats_user` in pgbouncer #820

Closed jbvaningen closed 5 months ago

jbvaningen commented 8 months ago

What issues does your PR fix?

What does your PR do?

This PR adds the following values:

Checklist

For all Pull Requests

For releasing ONLY

jbvaningen commented 7 months ago

@thesuperzapper are you or somebody else available to review this PR please?

stale[bot] commented 5 months ago

This issue has been automatically marked as stale because it has not had activity in 60 days. It will be closed in 7 days if no further activity occurs.

Thank you for your contributions.

Issues never become stale if any of the following is true:

  1. they are added to a Project
  2. they are added to a Milestone
  3. they have the lifecycle/frozen label
jbvaningen commented 5 months ago

@thesuperzapper we are still interested in getting this merged. Is anyone available to review this?

thesuperzapper commented 5 months ago

@jbvaningen sorry for missing this!

We are currently trying to get airflow 2.9 working in a new release, so I guess this could be part of that.

Is this single config really all you need?

Is there anything else we should add or another way of doing this?

jbvaningen commented 5 months ago

@thesuperzapper nice!

yes this single config is all that is needed to support pgbouncer metrics exporting.