-
Notifications
You must be signed in to change notification settings - Fork 198
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
Support modules configuration #146
Support modules configuration #146
Conversation
Any maintainer/reviewers for this repo? Maybe cc @SuperQ? |
Sorry I missed this review ping. This is a great idea. I'll give it a look over asap. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work! Please update the Readme documentation as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please also update the example config(s) in the README.
I have updated the doc. PTAL. |
Signed-off-by: Ben Ye <[email protected]>
Signed-off-by: Ben Ye <[email protected]>
Signed-off-by: Ben Ye <[email protected]>
Signed-off-by: Ben Ye <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Please take another look @rustycl0ck thank you! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Breaking Change: The exporter config file format has changed. It now supports multiple modules to scrape different endpoints. * [FEATURE] Support custom valuetype #145 * [FEATURE] Support modules configuration #146 Signed-off-by: SuperQ <[email protected]>
Breaking Change: The exporter config file format has changed. It now supports multiple modules to scrape different endpoints. * [FEATURE] Support custom valuetype #145 * [FEATURE] Support modules configuration #146 * [FEATURE] Accept non-2xx HTTP status #161 Signed-off-by: SuperQ <[email protected]>
Breaking Change: The exporter config file format has changed. It now supports multiple modules to scrape different endpoints. * [FEATURE] Support custom valuetype #145 * [FEATURE] Support modules configuration #146 * [FEATURE] Accept non-2xx HTTP status #161 Signed-off-by: SuperQ <[email protected]>
Upstream changed the config format in prometheus-community/json_exporter#146
Signed-off-by: Ben Ye [email protected]
Fixes #86
This PR supports modules configuration. Similar to black-box exporter, scrape requests can add a
module
parameter to specify which module to use. If not specified,default
module will be used.