-
Notifications
You must be signed in to change notification settings - Fork 756
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
Capture usename and application_name for pg_stat_activity #673
Capture usename and application_name for pg_stat_activity #673
Conversation
f8c825f
to
567af30
Compare
Hi, thank you for this work. I've been able to test it myself on a few instances and it lgtm. |
This PR is out of date with the base branch. |
It is necessary to be able to exclude backups from long-running transaction alerts, as they are to be expected. With the current pg_stat_activity metric there is no ability to filter out specific users or application names. Resolves prometheus-community#668 Signed-off-by: cezmunsta <github@incoming-email.co.uk>
Signed-off-by: cezmunsta <github@incoming-email.co.uk>
567af30
to
d9ac4be
Compare
@sysadmind rebase complete |
Hi, small bump as I'm hoping to see this reviewed soon. |
Hopefully, someone will merge it for you soon @Arnoways, if you need to use this version. |
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.
Makes sense, thanks!
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.
LGTM
It is necessary to be able to exclude backups from long-running
transaction alerts, as they are to be expected. With the current
pg_stat_activity metric there is no ability to filter out
specific users or application names.
Resolves #668