CTDB: Patch set re-based because of updates

Martin Schwenke martin at meltin.net
Wed Mar 21 19:47:53 UTC 2018


On Wed, 21 Mar 2018 12:52:10 +0100, Swen Schillig <swen at vnet.ibm.com>
wrote:

> On Wed, 2018-03-21 at 22:30 +1100, Martin Schwenke wrote:
> > On Wed, 21 Mar 2018 08:46:36 +0100, Swen Schillig <swen at vnet.ibm.com>
> > wrote:
> >   
> > > On Wed, 2018-03-21 at 16:54 +1100, Martin Schwenke wrote:  
>  [...]  
>  [...]  
>  [...]  
> > > 
> > > Good catch !
> > > 
> > > Updated.  
> > 
> > This will eventually kill us...  
> I totally agree !
> > 
> > In commit 6, the definition of ctdb_start_revoke_ro_record() has been
> > wrapped but had more than 1 parameter per line.  Also the call to
> > ctdb_revoke_all_delegations() also has more then 1 parameter per
> > line.
> > 
> > With those 2 changes:
> > 
> > Reviewed-by: Martin Schwenke <martin at meltin.net>
> > 
> > I'm happy for a 2nd reviewer to jump in if you resend with those
> > changes.  
> Updates as requested.

For completeness:

Reviewed-by: Martin Schwenke <martin at meltin.net>

2nd team reviewer, please?

peace & happiness,
martin



More information about the samba-technical mailing list