← Back to team overview

openstack team mailing list archive

Re: Gerrit minimum review time frame

 

On Mon, 2012-03-12 at 17:57 -0500, Josh Kearney wrote:
> Is this is really a problem that needs solving? I'd like to believe that no
> member of Nova Core would approve something that they aren't familiar with.

I think the issue is more a case of allowing some time for other
reviewers to come along and catch issues before it gets merged.

Sometimes there can be a few folks working through an issue together and
the patch gets pushed and approved so quickly that no-one else gets a
chance to review.

> IMHO, we should be making better attempts at not letting branches sit
> around for days/weeks at a time.
> 
> On Mon, Mar 12, 2012 at 4:59 PM, Joe Gordon <jogo@xxxxxxxxxxxxxxxx> wrote:
> 
> > Hi All,
> >
> > I have noticed that some Gerrit branches get approved very quickly,
> > sometimes in a matter of minutes.   While most of the time these branches
> > are vetted properly, the window for reviewing can be so small that a
> > non-trivial branch lands but without enough vetting.  If someone is in a
> > meeting for half on hour they may miss the entire review window.  To fix
> > this problem I propose a minimum time frame (should be overridable in an
> > emergency) for a branch to be approved, perhaps 2 hours.  This time frame
> > would start on 'Upload time.'

I like the idea, personally. Even in the case of "emergencies", I don't
think 2 hours is unreasonable.

Cheers,
Mark.



Follow ups

References