<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<br>
<div class="moz-cite-prefix">On 9/8/2014 2:30 PM, Anne Gentle wrote:<br>
</div>
<blockquote
cite="mid:CAD0KtVHYKcUWzsaepoDq3f+RZtYgjw2b8wPQsZUFMxtNaVM9Gw@mail.gmail.com"
type="cite">
<div dir="ltr">
<div class="gmail_extra"><br>
<div class="gmail_quote">On Mon, Sep 8, 2014 at 1:23 PM,
Christian Berendt <span dir="ltr"><<a
moz-do-not-send="true"
href="mailto:berendt@b1-systems.de" target="_blank">berendt@b1-systems.de</a>></span>
wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px
0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">I
would like to propose the introduction of a minimum time
before<br>
approving a new review request.<br>
<br>
Example: <a moz-do-not-send="true"
href="https://review.openstack.org/#/c/119783/"
target="_blank">https://review.openstack.org/#/c/119783/</a><br>
<br>
Uploaded: Sep 8, 2014 4:59 PM<br>
First +2: 6:10 PM<br>
Second +2: 8:15 PM<br>
Approval: 8:16 PM<br>
<br>
I think we should wait at least 24 hours before approving
a new review<br>
request (exception: urgent or generated changes). This way
everybody of<br>
us has the chance to realize and review new review
requests.<br>
<br>
</blockquote>
<div><br>
</div>
<div>Interesting proposal, we did not address in <a
moz-do-not-send="true"
href="https://wiki.openstack.org/wiki/Documentation/ReviewGuidelines">https://wiki.openstack.org/wiki/Documentation/ReviewGuidelines</a>
-- my feeling is that we can't all review all patches so
it's better to not enforce a minimum wait time. The
impatience people have with our reviews should be reversed
and we shouldn't have a policy of an enforced wait time.</div>
<div><br>
</div>
<div>To me, it's preferred to have hundreds of patches go
through efficiently rather than ensure everyone sees every
patch.</div>
<div><br>
</div>
<div>Just my thinking though, would love to hear what others
have to say.</div>
<br>
</div>
</div>
</div>
</blockquote>
<br>
I think that it's a good thought to make sure that patches have
enough oversight, but it's unrealistic to have "everybody" look at a
patch; there's just not enough time, as evidenced by the backlog we
already have. If a patch goes through and someone has a concern,
opening a new bug is just a button-click away.<br>
<br>
I also agree that people (all over OpenStack, not just in Docs) are
already frustrated enough with the review process. ENSURING that it
takes a day will only make things worse, I would think.<br>
<br>
But perhaps if there's a patch that people are concerned about --
say something major is going on -- there's a way for us to flag it,
or maybe just to have the cores to +1 it temporarily just to show
that they're in favor, while it's being further reviewed?<br>
<br>
---- Nick<br>
<br>
<div class="moz-signature">-- <br>
Nick Chase <br>
1-650-567-5640<br>
Technical Marketing Manager, Mirantis<br>
Editor, OpenStack:Now<br>
<a href="http://openstacksv.com"><img
src="cid:part4.04020703.09050105@mirantis.com"></a></div>
</body>
</html>