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 healthcheck #9

Open
jvperrin opened this issue Aug 16, 2017 · 4 comments
Open

Add healthcheck #9

jvperrin opened this issue Aug 16, 2017 · 4 comments
Assignees

Comments

@jvperrin
Copy link
Member

jvperrin commented Aug 16, 2017

This would be nice to have so that the bot can be restarted automatically if it becomes unhealthy. We do need to make sure to avoid restarting over and over, but Marathon Kubernetes will rate-limit it a little bit so that it doesn't spam way too much, but that would not be very nice for people on IRC with all the joins/quits from bots.

This would also help with #8 so that if the bot crashes and can't reach Slack, then it would be restarted to avoid being down for a long time.

@kkuehlz kkuehlz self-assigned this Oct 16, 2017
@jvperrin jvperrin changed the title Add Marathon/Mesos healthcheck Add healthcheck Jun 26, 2019
@dkess
Copy link
Member

dkess commented Jul 6, 2019

How do you envision the mechanics of this actually working?

@jvperrin
Copy link
Member Author

jvperrin commented Jul 7, 2019

Not sure yet, but I was loosely envisioning running a command that would check all threads (the bridge bot and all the IRC bots) are running, but the actual mechanics of how that would function are up for debate.

@cg505
Copy link
Member

cg505 commented Jul 8, 2019

I think we might want to write a general way to interact with IRC that can be used both by slackbridge and ircbot. Then we can do something like connect to IRC, maybe send a message, and wait for a response. This could be in combination with a simple slack bot that just will just connect and send a message/wait for a response.
This is pretty complicated for a healthcheck though, idk

@dkess
Copy link
Member

dkess commented Jul 8, 2019

We can't use a message-based healthcheck because it would fill up Slack history

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