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

Add a service check for Couchbase. #1200

Merged
merged 1 commit into from
Dec 18, 2014
Merged

Conversation

conorbranagan
Copy link
Member

  • With a test that should work without Couchbase running (testing
    failure)
  • Also cleaned up some pre-omnibus code checking the Python version.

@conorbranagan conorbranagan force-pushed the conor/couchbase-service-check branch from 7b16648 to 2495f34 Compare November 15, 2014 19:15
@LeoCavaille
Copy link
Member

Looks good, thanks!
@remh could confirm but the test was againt python >= 2.6 and I don't think we're supporting any older version anymore even in the source install.

@remh
Copy link
Contributor

remh commented Nov 17, 2014

Yup. As the deb package and rpm packages bundle python 2.7 and the source installation script has always just been working with python2.6 we can be sure that agent > 5.0 only run on python 2.6 or 2.7.

@LeoCavaille LeoCavaille self-assigned this Dec 10, 2014
@LeoCavaille
Copy link
Member

@conorbranagan could you rebase on master please?

- With a test that should work without Couchbase running (testing
  failure)
- Also cleaned up some pre-omnibus code checking the Python version.
@conorbranagan conorbranagan force-pushed the conor/couchbase-service-check branch from 2495f34 to f0df22c Compare December 10, 2014 17:54
LeoCavaille added a commit that referenced this pull request Dec 18, 2014
@LeoCavaille LeoCavaille merged commit 3271a2c into master Dec 18, 2014
@LeoCavaille LeoCavaille deleted the conor/couchbase-service-check branch December 18, 2014 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants