-
Notifications
You must be signed in to change notification settings - Fork 265
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 couchbase support #404
base: master
Are you sure you want to change the base?
Conversation
👍 merge plz? |
@zeitlinger might be worth to squash the commits first. you have multiple commits with the same message |
+ this is not a backend | ||
+ it just executes programs (usually bash scripts) in a directory | ||
+ the program name is used as the metric name | ||
+ the output of the program must be numeric |
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.
this part should probably go to a different branch, isn't related to the couchbase support part
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.
Yes, this referes to nagiorelic.
please don't merge this branch, it's broken. #457 builds up on this, fixes everything, and adds many more features. We're using it at @ Rounds for some time now and it's working great. |
No description provided.