Skip to content
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

Couchbase check #690

Closed
wants to merge 0 commits into from
Closed

Couchbase check #690

wants to merge 0 commits into from

Conversation

jslatts
Copy link
Contributor

@jslatts jslatts commented Oct 10, 2013

A couchbase check based on the couchdb check. Pulls cluster, node, and bucket level metrics.

return couchbase

@staticmethod
def parse_agent_config(agentConfig):
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can remove that method.
This method is used for compatibility purposes when agent checks were not using checks.d and were configured using datadog.conf

@remh
Copy link
Contributor

remh commented Oct 17, 2013

Thanks @jslatts !

This check can be useful ?
Could you fix the comments I made and also rebase your pull request so it won't include the fix for mysql you made and that is already merged ?

Last thing could you write a test for it the same way it's done for the couchdb check ?
Thanks a lot !

@jslatts
Copy link
Contributor Author

jslatts commented Oct 23, 2013

This PR has been superseded by #700. Closing.

@jslatts jslatts closed this Oct 23, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants