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