<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">On Fri, Oct 11, 2013 at 1:34 PM, Clint Byrum <span dir="ltr"><<a href="mailto:clint@fewbar.com" target="_blank">clint@fewbar.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Recently in the TripleO meeting we identified situations where we need<br>
to make it very clear that it is ok to pick up somebody else's patch<br>
and finish it. We are broadly distributed, time-zone-wise, and I know<br>
other teams working on OpenStack projects have the same situation. So<br>
when one of us starts the day and sees an obvious issue with a patch,<br>
we have decided to take action, rather than always -1 and move on. We<br>
clarified for our core reviewers that this does not mean that now both<br>
of you cannot +2. We just need at least one person who hasn't been in<br>
the code to also +2 for an approval*.<br>
<br>
I think all projects can benefit from this model, as it will raise<br>
velocity. It is not perfect for everything, but it is really great when<br>
running up against deadlines or when a patch has a lot of churn and thus<br>
may take a long time to get through the "rebase gauntlet".<br>
<br>
So, all of that said, I want to encourage all OpenStack developers to<br>
say "thanks for fixing my patch" when somebody else does so. It may seem<br>
obvious, but publicly expressing gratitude will make it clear that you<br>
do not take things personally and that we're all working together.<br>
<br>
Thanks for your time -Clint<br>
<br>
* If all core reviewers have been in on the patch, then any two +2's<br>
work.<br>
<br></blockquote><div><br></div><div>+1 across the board -- keystone-core follows this approach, especially around feature freeze / release candidate time.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div><br></div>-Dolph
</div></div>