[Openstack] Gerrit minimum review time frame
Johannes Erdfelt
johannes at erdfelt.com
Tue Mar 13 15:20:54 UTC 2012
On Tue, Mar 13, 2012, Mark McLoughlin <markmc at redhat.com> wrote:
> 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.
And what's the problem if an occasional issue gets missed?
> 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.
Everyone has an opportunity to review even after a patch gets merged.
JE
More information about the Openstack
mailing list