[PATCH] Make loadparm more common

[Andrew Bartlett]

On Mon, 2014-03-31 at 11:46 +0200, Andreas Schneider wrote:
> On Friday 28 March 2014 13:42:02 Andrew Bartlett wrote:
> > Even more patches are at
> > http://git.catalyst.net.nz/gitweb?p=samba.git;a=shortlog;h=refs/heads/loadpa
> > rm-talloc-to-review3
> > 
> > Please review!
> 
> No! I've reviewed this patchset 3 or 4 times now. You constantly keep changing 
> and adding patches. I will not waste more time on reviewing stuff which is 
> still WIP and not tested.

I can understand your frustration.  As with other large patch sets, what
appears to be OK can have other issues when further investigated.  

The patch set at
http://git.catalyst.net.nz/gitweb?p=samba.git;a=shortlog;h=refs/heads/loadparm-talloc-to-review4 not only passes a full autobuild, it has passed a more limited test on each and every commit.

One of the big challenges with this patch set is that many issues have
shown up that are not present at the end, but which would impede a
bisect into the series later.  I've been working hard over the past few
days to address both these and other issues that showed up in a full
autobuild, but not in the expanded docs.py test that we aimed
specifically at this under-tested area of code.

> I will not start guessing which 55 patches are ready for master and pick them 
> out of the tree, that is your task.

At this point, what I would like to see is these patches reviewed:
http://git.catalyst.net.nz/gitweb?p=samba.git;a=commitdiff;h=5fa1a3cd62e0dcefc5364f83046db025fc0e65b9
http://git.catalyst.net.nz/gitweb?p=samba.git;a=commitdiff;h=909fb5a6c7df2e2cc7e48d5814eb2cff53ee6664
http://git.catalyst.net.nz/gitweb?p=samba.git;a=commitdiff;h=5c9f5892c4e3304e2e3f11be0a41d087f9df6d9c
http://git.catalyst.net.nz/gitweb?p=samba.git;a=commitdiff;h=746cbfcd93c38e5bb66e892fa2ef0d0d11888703

Getting these reviewed should allow the set that was previously reviewed
by you and others to be merged, because gerrit should, if we are lucky,
pick them up as a trivial rebase.  Getting those marked out onto the
patches will hopefully temper a little of the remaining frustration,
because I agree re-reviewing patches again and again just isn't fun. 

> I'm will review patches if you have a clean tree for me which can really be 
> pushed at the end ...

While further testing may again reveal issues, I expect that
loadparm-talloc-to-review4 is such a tree. 

Jeremy,

Per our discussion today, see above for the patches I was particularly
looking for you OK on.

Thanks,

Andrew Bartlett

-- 
Andrew Bartlett
http://samba.org/~abartlet/
Authentication Developer, Samba Team  http://samba.org
Samba Developer, Catalyst IT          http://catalyst.net.nz/services/samba