[PATCH] Fix CTDB PID file handling
Volker Lendecke
vl at samba.org
Thu Sep 22 13:01:56 UTC 2016
Hi, Martin!
On Thu, Sep 22, 2016 at 04:04:15PM +1000, Martin Schwenke wrote:
> At the moment ctdbd_wrapper is solid but multiple ctdbd's can start at
> the same time. If they overtake each other in interesting ways then
> one ctdbd can exit early and remove another's PID file.
>
> Add a PID file abstraction, use it, make PID file creation the first
> action, clean up some surrounding code.
>
> Please review and maybe push...
We have lib/util/pidfile.c. You do destruction with a talloc context,
but the locking engine is also there in lib/util/pidfile.c. It's not
rocket science, but also not entirely trivial, so -- can't we share
code here between ctdb/common and lib/util?
Volker
More information about the samba-technical
mailing list