On 06/11/2017 01:35 μμ, William Lallemand wrote: > On Mon, Nov 06, 2017 at 12:11:13PM +0100, Pavlos Parissis wrote: >> On 06/11/2017 11:16 πμ, William Lallemand wrote: >>> The parent process supervises itself the children, we don't need to >>> store the children pids anymore in the pidfile in master-worker mode. >> >> I have a small objection against this. Having PIDs in a file allows external >> tools to monitor the >> workers. If those PIDs aren't written in a the pidfile then those tools have >> to use pgrep to find >> them, unless the master process can provide them in some way(stats socket?). >> >> My 2cents, >> Pavlos >> > > Hi Pavlos, > > This patch was made to prevent scripts to send signals to the children > directly > instead of sending them to the master which will forward them. That could end > in reporting a wrong exit code in the master for example. > > One of the problem of the pidfile, is that only the latest children were > written, so you wouldn't have the remaining PID in this list on a reload. > With pgrep -P you will have the full list of processes attached to the master. > > Unfortunately there is no stats socket on the master (yet?), so it's not > possible to do what you suggest. > > However, we can maybe add an option to write all new PIDs in the pidfile if > it's easier for supervision. >
That will be very much appreciated as it will allow us to have a smooth migration to the new master process model. Cheers, Pavlos
signature.asc
Description: OpenPGP digital signature