openstack team mailing list archive
-
openstack team
-
Mailing list archive
-
Message #08587
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