-
Notifications
You must be signed in to change notification settings - Fork 477
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
feat: add pgbouncer.statsUsers
value to set stats_user
in pgbouncer
#820
Conversation
Signed-off-by: jbvaningen <joost@dexterenergy.nl>
Signed-off-by: jbvaningen <joost@dexterenergy.nl>
@thesuperzapper are you or somebody else available to review this PR please? |
This issue has been automatically marked as stale because it has not had activity in 60 days. Thank you for your contributions. Issues never become stale if any of the following is true:
|
@thesuperzapper we are still interested in getting this merged. Is anyone available to review this? |
@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? |
@thesuperzapper nice! yes this single config is all that is needed to support pgbouncer metrics exporting. |
Signed-off-by: Mathew Wicks <5735406+thesuperzapper@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jbvaningen thanks, I pushed a small cosmetic update in d43bbf3, but this is good to merge.
stats_users
configuration option for pgbouncer
stats_users
configuration option for pgbouncerpgbouncer.statsUsers
value to set stats_user
in pgbouncer
What issues does your PR fix?
What does your PR do?
This PR adds the following values:
pgbouncer.statsUsers
(default:""
)stats_users
PgBouncer config in thepgbouncer.ini
file.Checklist
For all Pull Requests
For releasing ONLY