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

check-zookeeper-mode isn't working #14

Open
Albex opened this issue Nov 3, 2017 · 2 comments
Open

check-zookeeper-mode isn't working #14

Albex opened this issue Nov 3, 2017 · 2 comments

Comments

@Albex
Copy link

Albex commented Nov 3, 2017

The check to verify the mode of a node is not working. The code used to send the four letter word is not correct hence the result of the request is empty.
This is the only script that uses a different strategy to send the request. You should perhaps unify the scripts.

@majormoses
Copy link
Member

Thanks for bringing this up, I would be more than happy to review a PR to fix that and bring things more inline with the way the other checks work. pinging @fsniper as he is probably the most active contributor to this plugin.

@fsniper
Copy link
Contributor

fsniper commented Nov 6, 2017

I just checked my installed instance (1.4.0) and the check seems to be working correctly.

/opt/sensu/embedded/bin/check-zookeeper-mode.rb -s localhost -p 2181 -m master
CheckZookeeperMode CRITICAL: Zookeeper mode is follower and it does not match master /opt/sensu/embedded/bin/check-zookeeper-mode.rb -s localhost -p 2181 -m follower
CheckZookeeperMode OK: follower

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants