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

Deployment process #8

Open
ltnilaysahu opened this issue Sep 20, 2017 · 5 comments
Open

Deployment process #8

ltnilaysahu opened this issue Sep 20, 2017 · 5 comments
Assignees

Comments

@ltnilaysahu
Copy link

There is no readme for the deployment process. is this "kafka-health-check' supposed to be deployed on every broker or a separate machine can be used for deployment with broker config.

@sachinlala
Copy link

As the utility is communicating through zookeeper, I don't think it's necessary to deploy/run from each broker. It should be fine to run from a separate host. I am currently testing the utility for a multi-node Kafka setup.

@andreas-schroeder
Copy link
Owner

Hi @ltnilaysahu,

good point. I'm currently deploying it on every node of my cluster in order to monitor the status of every broker independently.

It is also possible to be used when deployed on a separate machine having access to the brokers if you are interested in the overall cluster health only.

I'll keep this one open until I come around writing up some docs on the deployment options.

@wilson
Copy link
Contributor

wilson commented Sep 24, 2017

While we're talking about the deployment/life-cycle.. I immediately found myself wanting a cli option for "wait up to N seconds for a healthy cluster, exit with zero status if one is found, nonzero otherwise."

Is that a thing you'd be OK with in this tool? If so I could take a swing at a PR implementing it.
I want to be able to use it as a docker/docker-compose "healthcheck", and they have to be phrased as periodic commands vs. persistent endpoints.

@andreas-schroeder
Copy link
Owner

Hi @wilson,

sounds like an interesting idea, let's continue to discuss it as a separate issue under #12

Please check the issue description to validate that I got it right, then we'll continue working on that idea there.

@ltnilaysahu
Copy link
Author

Hi @andreas-schroeder I have created a pull request #17 for the multi broker support.

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

4 participants