Skip to content

DRAFT: Prometheus_exec receiver *This PR is too big and will be broken into smaller pieces - keeping it here for reference until the rest is merged* #453

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 60 commits into from
Closed

DRAFT: Prometheus_exec receiver *This PR is too big and will be broken into smaller pieces - keeping it here for reference until the rest is merged* #453

wants to merge 60 commits into from

Conversation

Nicolas-MacBeth
Copy link
Contributor

@Nicolas-MacBeth Nicolas-MacBeth commented Jul 15, 2020 •
edited
Loading

Description:
PR for prometheus_exec receiver. The README contains lots of info on what this does, and feel free to ask any questions!

Testing:
87% coverage in prometheusexec package with end-to-end test and 90% in the subprocessmanager package. Most, if not all, of untested code lines are simply errors that are hard to reproduce in testing.

Documentation:
Comments are used as documentation, as well as a detailed README.