Re: [PATCH 0/4] Add info on why process is down to statusfile
On 01/19/15 17:41, Laurent Bercot wrote:
> On 19/01/2015 01:04, Olivier Brunel wrote:
>
>> Sure; though I'm totally fine if you want to discuss them, or would like
>> me to fix/make changes and re-send, as you see fit. That's what I
>> expected really.
>
> I have implemented your suggestions, with minor changes.
>
> For instance, I removed the "felldown" flag: the only case where it would
> be false is at boot time, when the service hasn't been launched yet.
> Using s6-svstat at this point is very unlikely and would return a transient
> result, identified with "want up" in the output, unless the service has a
> down file, in which case the admin is supposed to know what he's doing -
> in other words, it's a corner case not worthy of ad-hoc code IMO.
>
> Please try the latest skalibs and s6 git snapshots and tell me if they're
> working for you.
Yep, works great. Thanks a lot!
Side question: I see you haven't added support of the file "ready" into
s6-svstat, any reason?
Received on Mon Jan 19 2015 - 20:55:10 UTC
This archive was generated by hypermail 2.3.0
: Sun May 09 2021 - 19:44:19 UTC