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.

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

Add connection limits metrics for pg_roles and pg_database #997

Merged

Conversation

jocelynthode
Copy link
Contributor

Fixes #996

This PR adds the possibility to gather metrics for Database and roles connection limits.

collector/pg_database.go Outdated Show resolved Hide resolved
@jocelynthode
Copy link
Contributor Author

@SuperQ: Hey, could you review my latest changes please? :)

collector/pg_database.go Outdated Show resolved Hide resolved
Signed-off-by: Jocelyn Thode <jocelyn@thode.email>
Signed-off-by: Jocelyn Thode <jocelyn@thode.email>
collector/pg_database.go Outdated Show resolved Hide resolved
collector/pg_roles.go Outdated Show resolved Hide resolved
const rolesSubsystem = "roles"

func init() {
registerCollector(rolesSubsystem, defaultEnabled, NewPGRolesCollector)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@SuperQ How do you feel about this being default enabled?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any update @SuperQ ?

collector/pg_roles.go Outdated Show resolved Hide resolved
collector/pg_roles.go Outdated Show resolved Hide resolved
jocelynthode and others added 3 commits January 23, 2024 16:35
Co-authored-by: Joe Adams <github@joeadams.io>
Signed-off-by: Jocelyn Thode <jocelynthode@users.noreply.github.com>
Co-authored-by: Joe Adams <github@joeadams.io>
Signed-off-by: Jocelyn Thode <jocelynthode@users.noreply.github.com>
Co-authored-by: Joe Adams <github@joeadams.io>
Signed-off-by: Jocelyn Thode <jocelynthode@users.noreply.github.com>
Signed-off-by: Jocelyn Thode <jocelyn@thode.email>
@jocelynthode
Copy link
Contributor Author

@SuperQ any update on this PR?

@jocelynthode
Copy link
Contributor Author

@sysadmind Is there something I can do to get this PR moving forward? I'm looking forward to be able to use these new metrics.

@sysadmind sysadmind merged commit 8f39f5b into prometheus-community:master Feb 22, 2024
10 checks passed
@sysadmind
Copy link
Contributor

It will be available in master. I'm not sure when the next release will be. I haven't had much time recently so I haven't been able to work on this project.

@jocelynthode
Copy link
Contributor Author

Thanks for the work you do on this project it's much appreciated :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Adding connection limit stats for database and roles
3 participants