[openstack-dev] doubling our core review bandwidth

Alexis Lee alexisl at hp.com
Mon Sep 8 15:07:40 UTC 2014


Sean Dague said on Mon, Sep 08, 2014 at 09:22:56AM -0400:
> > On 09/08/2014 05:17 AM, Steven Hardy wrote:
> >> I think this may be a sensible move, but only if it's used primarily to
> >> land the less complex/risky patches more quickly.
> 
> 2 +2 has been part of OpenStack culture for a long time, and there is a
> good reason for it, it really does keep bugs out.
> 
> It should also be clear that the subject of this email really should
> have been "merging code faster", because nothing in here doubles the
> review bandwidth, it just provides us with less review coverage.

For these reasons, I'm also wary of changing this in general.

Sometimes I yell in IRC if I +2 something important, this shortens loop
time as hopefully I already understand the patch and can answer
questions.

Single-approvals for small + simple changes could be worth trying.
Perhaps also for large + simple changes like whitespace fixes.


Alexis
-- 
Nova Engineer, HP Cloud.  AKA lealexis, lxsli.



More information about the OpenStack-dev mailing list