Skip to content
This repository has been archived by the owner on Sep 17, 2021. It is now read-only.

Replace monit with another process monitoring system #22

Open
marc- opened this issue May 20, 2015 · 3 comments
Open

Replace monit with another process monitoring system #22

marc- opened this issue May 20, 2015 · 3 comments

Comments

@marc-
Copy link

marc- commented May 20, 2015

Monit is not suitable for containers. Please see https://bitbucket.org/tildeslash/monit/issue/53/ . It ends up crowded with zombie processes and you'll have to kill the container.

32714 ?        Z      0:00 [service] <defunct>
32730 ?        Z      0:00 [service] <defunct>
32736 ?        Z      0:00 [service] <defunct>
32744 ?        Z      0:00 [service] <defunct>
32753 ?        Z      0:00 [service] <defunct>
32754 ?        Z      0:00 [service] <defunct>

I would suggest to move to supervisord .
Please also consider splitting mysql and httpd+php into separate containers. In this case process monitoring system is not required at all.

@berngp
Copy link
Owner

berngp commented Jun 1, 2015

Thanks @marc-, I'll look into moving things into supervisord, I noticed that issue too. Now, the decission to bundle mysql and httpd+php into the same image/container was to help folks run Zabbix for development and test environments without too much hassle and dependencies. I know that composer is a tool that can aid on this but this will require users to install such tool. In a nutshell, I am a bit hesitant on making such change since it will affect users that assume that the image has no dependencies.

@marc-
Copy link
Author

marc- commented Jun 1, 2015

That makes sense. Anyways, I've already created image without mysql and java based on minimalistic Alpine Linux distro.

@marc- marc- closed this as completed Jun 1, 2015
@berngp
Copy link
Owner

berngp commented Jun 1, 2015

great, thanks @marc-

@berngp berngp reopened this Jun 19, 2015
berngp added a commit that referenced this issue Aug 13, 2015
This commit references #22 and establishes the first attempt to migrate
from `monit` to `supervisor`. The current code doesn't address the fact
that we can't start `zabbix-*` services without daemonizaing them. This
is required since `supervisor` must spawn such processes and hold their
reference and therefore the ability to assess if they are up or not.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants