Re: register runsvdir as subreaper

2017-02-02 Thread Olivier Brunel
On Thu, 02 Feb 2017 19:30:17 + "Laurent Bercot" wrote: > >There's also the possible case of having this for getty/login > >session. So any process spawned from there won't be reparented to > >PID1 but to e.g. the session's supervisor; That can include things >

Re: register runsvdir as subreaper

2017-02-02 Thread Laurent Bercot
Or put your script as /usr/local/bin/runsv (assuming /usr/local/bin is in your PATH ofc) so the original runsv is left untouched, which is nice (I feel), and certainly better when using a package manager. Indeed, if runsvdir uses PATH resolution to find runsv, it's easier and better to

Re: register runsvdir as subreaper

2017-02-02 Thread Laurent Bercot
There's also the possible case of having this for getty/login session. So any process spawned from there won't be reparented to PID1 but to e.g. the session's supervisor; That can include things such as pulseaudio, settings daemon, dbus, window manager, etc Those are more examples of what you

Re: register runsvdir as subreaper

2017-02-02 Thread John Regan
On 02/02/2017 03:29 PM, Mitar wrote: Hi! Hm, when I read the runit man page I got scared because of its trying to reboot and halt the machine. I am not sure how will that interact with a Docker container. I also didn't want one extra process to be in every container. But you are right, it seems

Re: register runsvdir as subreaper

2017-02-02 Thread Laurent Bercot
Hm, when I read the runit man page I got scared because of its trying to reboot and halt the machine. I am not sure how will that interact with a Docker container. Don't worry about that. When run from an unshared PID namespace, the reboot() system call is diverted by the Linux kernel and

Re: register runsvdir as subreaper

2017-02-02 Thread Jonathan de Boyne Pollard
Laurent Bercot: You want runsvdir to be your reaper, so you'd just run "local-reaper runsvdir scandir" instead of "runsvdir scandir". Actually you'd run > local-reaper true runsvdir scandir

Re: register runsvdir as subreaper

2017-02-02 Thread Olivier Brunel
On Thu, 02 Feb 2017 16:50:03 + "Laurent Bercot" wrote: > If your question was about the mechanism of wrapping runsv: > > mv /bin/runsv /bin/runsv.real > cat > /bin/runsv < #!/bin/sh > exec local-reaper /bin/runsv.real "$@" > EOF > chmod 755 /bin/runsv Or

Re: register runsvdir as subreaper

2017-02-02 Thread Laurent Bercot
Except in this case, runsvdir executes runsv, so you have to tell runsvdir to chain-load. You want runsvdir to be your reaper, so you'd just run "local-reaper runsvdir scandir" instead of "runsvdir scandir". If you wanted runsv to be your reaper instead, it would indeed be somewhat more

Re: register runsvdir as subreaper

2017-02-02 Thread Laurent Bercot
If your question was about the mechanism of wrapping runsv: mv /bin/runsv /bin/runsv.real cat > /bin/runsv <

Re: register runsvdir as subreaper

2017-02-02 Thread Olivier Brunel
On Wed, 01 Feb 2017 16:48:15 + "Laurent Bercot" wrote: (...) > You want a clean process tree with a visually pleasing "ps afuxww" > output? Fix your services so they don't leave orphans in the first There's also the possible case of having this for getty/login