[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