[PATCH] CTDB fixups

Amitay Isaacs amitay at gmail.com
Tue Nov 7 02:52:08 UTC 2017


On Mon, Nov 6, 2017 at 12:38 PM, Martin Schwenke via samba-technical <
samba-technical at lists.samba.org> wrote:

> * ctdb-common: Ensure unused bytes in union are initialised
>
>   Initialised data can make it to ctdbd.  We should probably fix this
>   in smbd too.
>
> * ctdb-tests: Have fake daemon log when it parses public IPs
>
>   Other initialisation is logged, so the absence here was confusing.
>
> * ctdb-scripts: Don't bother checking PID file when starting ctdbd
>
>   From commit message:
>
>     This is an optimisation that can cause incorrect results.  If ctdbd
>     was killed and there is a stale PID file then this will often cause
>     "CTDB exited during initialisation".  The wrapper reads the old PID
>     from the PID file, finds the PID gone, complains and exits.
>
>     It is better to drop this code and finally get this right.  If
>     ctdbd does exit early then it will take CTDB_STARTUP_TIMEOUT
>     (default 10) seconds before the wrapper fails.  That's not too
>     bad...
>
> * ctdb-common: Fix stale socket removal
>
>   Fix a regression in master.  Patch by Amitay, already has my review.
>
> * ctdb-tests: Add sock_daemon test for stale socket handling
>
>   Add a new test to avoid repeat of regression.
>
> Please review and maybe push...
>
> peace & happiness,
> martin
>

Pushed to autobuild.

Amitay.


More information about the samba-technical mailing list