-
Notifications
You must be signed in to change notification settings - Fork 803
Capture usename and application_name for pg_stat_activity #673
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
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 <[email protected]>
Signed-off-by: cezmunsta <[email protected]>
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. |
SuperQ
left a comment
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!
sysadmind
left a comment
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