[PATCH] Fix implementation of CTDB control PROCESS_EXISTS (bug 13012)
Martin Schwenke
martin at meltin.net
Tue Sep 12 07:11:04 UTC 2017
On Tue, 12 Sep 2017 16:00:35 +1000, Amitay Isaacs <amitay at gmail.com>
wrote:
> On Tue, Sep 12, 2017 at 3:30 PM, Martin Schwenke via samba-technical <
> samba-technical at lists.samba.org> wrote:
>
> > On Fri, 8 Sep 2017 21:51:16 +1000, Martin Schwenke via samba-technical
> > <samba-technical at lists.samba.org> wrote:
> >
> > > On Fri, 8 Sep 2017 16:50:03 +1000, Martin Schwenke via samba-technical
> > > <samba-technical at lists.samba.org> wrote:
> > >
> [...]
> [...]
> > as
> [...]
> [...]
> > >
> > > ... and pushed.
> >
> > There's a little race in the test here. The attached patch seems to
> > fix it.
> >
> >
> Thanks for identifying the race condition.
>
> We also need a fix to test_wrap to correctly set up the PATH.
>
> Attached patches include:
> - Martin's fix with missing BUG tag and my reviewed-by.
> - Fix to test_wrap.
>
> Please review and push.
Reviewed-by: Martin Schwenke <martin at meltin.net>
Mega-push coming... :-)
peace & happiness,
martin
More information about the samba-technical
mailing list