Re: [PATCH 0/4] Add info on why process is down to statusfile

2015-01-18 Thread Laurent Bercot
Hi Olivier ! Thanks for the patches. A bit of discussion below. So I wanted to have the return code/signal that occured when a process went down available on the statusfile, as I believe this is useful information to have for all services. Yes, that makes sense. Looking into it, I

[PATCH] svstat: Account for file ready

2015-01-18 Thread Olivier Brunel
Signed-off-by: Olivier Brunel j...@jjacky.com --- src/supervision/s6-svstat.c | 12 1 file changed, 12 insertions(+) diff --git a/src/supervision/s6-svstat.c b/src/supervision/s6-svstat.c index c986b8d..bc27c32 100644 --- a/src/supervision/s6-svstat.c +++

[PATCH 3/4] Add info on why process is down to statusfile

2015-01-18 Thread Olivier Brunel
That is, add the wstat allowing to know the return code/signal that occured when process went down. Because in addition to those info being provided to the optional finish script, it is useful to have them always for all services. Signed-off-by: Olivier Brunel j...@jjacky.com ---

[PATCH 4/4] svstat: Show signal name as well

2015-01-18 Thread Olivier Brunel
Signed-off-by: Olivier Brunel j...@jjacky.com --- src/supervision/s6-svstat.c | 93 - 1 file changed, 92 insertions(+), 1 deletion(-) diff --git a/src/supervision/s6-svstat.c b/src/supervision/s6-svstat.c index b98e69e..766d04c 100644 ---

[PATCH 2/4] supervise: Document arguments to ./finish

2015-01-18 Thread Olivier Brunel
Signed-off-by: Olivier Brunel j...@jjacky.com --- doc/s6-supervise.html | 2 ++ 1 file changed, 2 insertions(+) diff --git a/doc/s6-supervise.html b/doc/s6-supervise.html index b080626..dca437c 100644 --- a/doc/s6-supervise.html +++ b/doc/s6-supervise.html @@ -45,6 +45,8 @@ If it already exists,