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 pg_database_size and custom metrics to postgres integration #1302

Closed
wants to merge 8 commits into from

Conversation

ipolishchuk
Copy link

Custom metrics may be defined in postgres.yaml in the same format as metrics are defined in postgres.py
As an example, Londiste 3 replication lag metric is added to postgres.yaml.example.

@ipolishchuk ipolishchuk changed the title Add pg_database_size and custome metrics to postgres integration Add pg_database_size and custom metrics to postgres integration Jan 15, 2015
@remh
Copy link

remh commented Jan 15, 2015

Thanks a lot @ipolishchuk !

We've frozen the changelog of our 5.2.0 release so we'll review this PR once 5.2.0 is out and we'll aim at releasing your work for the 5.3.0 release.

Thanks again!

@ipolishchuk
Copy link
Author

I see a message 'Travis CI build failed', I suspect this might be because I've added a new metric postgresql.database_size, which i snot present in the test suite.

@remh remh self-assigned this Feb 23, 2015
LeoCavaille pushed a commit to DataDog/puppet-datadog-agent that referenced this pull request Feb 23, 2015
This can be merged with its upstream buddy
DataDog/dd-agent#1302
remh pushed a commit that referenced this pull request Feb 25, 2015
- Fix tests (and add one more)
- Add a limit to the number of results custom queries can return
- small copy changes to improve readability
@remh
Copy link

remh commented Feb 25, 2015

Thanks a lot @ipolishchuk i made a few changes and opened a new pul request that will be merged very soon ( #1395 ) so i'm closing this one.

Thanks again!

@remh remh closed this Feb 25, 2015
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.

2 participants