-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
collectors/GoCollector: Add tests and examples #1109
Conversation
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! Some comments (:
97bb078
to
22bbb17
Compare
Signed-off-by: Kemal Akkoyun <[email protected]>
22bbb17
to
9c7a9d5
Compare
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, just small nit (:
Signed-off-by: Kemal Akkoyun <[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.
👍🏽
…uration. (#1102) * goCollector: Added rule support for granular metric configuration. Fixes: #1089 Signed-off-by: bwplotka <[email protected]> * Added compatibility mode with old options. (#1107) * Added compatibility mode with old options. Signed-off-by: bwplotka <[email protected]> * Copyright header. Signed-off-by: bwplotka <[email protected]> * Remove bucket option for now. (#1108) Signed-off-by: bwplotka <[email protected]> * collectors/GoCollector: Add tests and examples (#1109) * Add tests and examples Signed-off-by: Kemal Akkoyun <[email protected]> * Add docs for the presets Signed-off-by: Kemal Akkoyun <[email protected]> Co-authored-by: Kemal Akkoyun <[email protected]>
Signed-off-by: Kemal Akkoyun [email protected]