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 support for database metrics #2777

Merged
merged 1 commit into from
Jan 28, 2019

Conversation

joekohlsdorf
Copy link
Contributor

What does this PR do?

This PR adds per database metrics to the pgBouncer check.
See SHOW DATABASES on https://pgbouncer.github.io/usage.html

Motivation

These metrics are vital for monitoring pgBouncer, especially when having more than one database configured in one instance.

@joekohlsdorf joekohlsdorf requested a review from a team as a code owner December 17, 2018 21:58
@codecov-io
Copy link

Codecov Report

Merging #2777 into master will decrease coverage by 11.2%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           master    #2777       +/-   ##
===========================================
- Coverage   84.75%   73.54%   -11.21%     
===========================================
  Files         657        6      -651     
  Lines       37463      223    -37240     
  Branches     4502       26     -4476     
===========================================
- Hits        31750      164    -31586     
+ Misses       4397       42     -4355     
+ Partials     1316       17     -1299

@joekohlsdorf
Copy link
Contributor Author

What do I have to do to get this simple change merged?

Copy link
Contributor

@ofek ofek left a comment

Choose a reason for hiding this comment

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

Works locally for me, thanks!!!

@ofek ofek merged commit fd71e1b into DataDog:master Jan 28, 2019
@ofek ofek changed the title pgBouncer: add support for database metrics Add support for database metrics Jan 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants