Skip to content
This repository has been archived by the owner on Nov 5, 2021. It is now read-only.

Postgres Surfacer #130

Closed
dmicanzerofox opened this issue May 8, 2018 · 4 comments
Closed

Postgres Surfacer #130

dmicanzerofox opened this issue May 8, 2018 · 4 comments
Assignees

Comments

@dmicanzerofox
Copy link
Contributor

Currently, I am auditing using cloudprober with prometheus and postgres remote endpoint. It would simplify things drastically to be able to write to postgres directly from cloudprober. I would be happy to contribute this if it's something that could be in cloudprober.

@manugarg
Copy link
Contributor

Sorry for the late response @dmicanzerofox. Postgres surfacer for Cloudprober sounds good to me.

dmicanzerofox added a commit to dmicanzerofox/cloudprober that referenced this issue May 25, 2018
dmicanzerofox added a commit to dmicanzerofox/cloudprober that referenced this issue May 28, 2018
Removed Redundant Postgres Surfacer Event log

Postgres: Factors out EventMetric param from distribution functions`
@manugarg
Copy link
Contributor

manugarg commented Jun 29, 2018

2513471 by @dmicanzerofox adds the Posgres surfacer.

@dmicanzerofox
Copy link
Contributor Author

Aw yes!! Thank you for all your time @manugarg ! We're going to be able to start using this shortly. Would you still like the test scripts / documentation? Should that be part of this issue??

@manugarg
Copy link
Contributor

@dmicanzerofox Sounds great. Once you start using it, would you mind sharing how you use cloudprober at #123.

Adding test scripts / documentation doesn't have to be part of this issue. However, you're probably the right person to add that, so it will be great if you can do that whenever you get a chance.

@manugarg manugarg closed this as completed Dec 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants