<div dir="ltr">+1 </div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 16, 2015 at 12:03 PM, Carl Baldwin <span dir="ltr"><<a href="mailto:carl@ecbaldwin.net" target="_blank">carl@ecbaldwin.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">+1<br>
<div class="HOEnZb"><div class="h5"><br>
On Thu, Jan 15, 2015 at 3:31 PM, Kyle Mestery <<a href="mailto:mestery@mestery.com">mestery@mestery.com</a>> wrote:<br>
> The last time we looked at core reviewer stats was in December [1]. In<br>
> looking at the current stats, I'm going to propose some changes to the core<br>
> team. Reviews are the most important part of being a core reviewer, so we<br>
> need to ensure cores are doing reviews. The stats for the 90 day period [2]<br>
> indicate some changes are needed for core reviewers who are no longer<br>
> reviewing on pace with the other core reviewers.<br>
><br>
> First of all, I'm removing Sumit Naiksatam from neutron-core. Sumit has been<br>
> a core reviewer for a long time, and his past contributions are very much<br>
> thanked by the entire OpenStack Neutron team. If Sumit jumps back in with<br>
> thoughtful reviews in the future, we can look at getting him back as a<br>
> Neutron core reviewer. But for now, his stats indicate he's not reviewing at<br>
> a level consistent with the rest of the Neutron core reviewers.<br>
><br>
> As part of the change, I'd like to propose Doug Wiegley as a new Neutron<br>
> core reviewer. Doug has been actively reviewing code across not only all the<br>
> Neutron projects, but also other projects such as infra. His help and work<br>
> in the services split in December were the reason we were so successful in<br>
> making that happen. Doug has also been instrumental in the Neutron LBaaS V2<br>
> rollout, as well as helping to merge code in the other neutron service<br>
> repositories.<br>
><br>
> I'd also like to take this time to remind everyone that reviewing code is a<br>
> responsibility, in Neutron the same as other projects. And core reviewers<br>
> are especially beholden to this responsibility. I'd also like to point out<br>
> that +1/-1 reviews are very useful, and I encourage everyone to continue<br>
> reviewing code even if you are not a core reviewer.<br>
><br>
> Existing neutron cores, please vote +1/-1 for the addition of Doug to the<br>
> core team.<br>
><br>
> Thanks!<br>
> Kyle<br>
><br>
> [1]<br>
> <a href="http://lists.openstack.org/pipermail/openstack-dev/2014-December/051986.html" target="_blank">http://lists.openstack.org/pipermail/openstack-dev/2014-December/051986.html</a><br>
> [2] <a href="http://russellbryant.net/openstack-stats/neutron-reviewers-90.txt" target="_blank">http://russellbryant.net/openstack-stats/neutron-reviewers-90.txt</a><br>
><br>
</div></div><div class="HOEnZb"><div class="h5">> __________________________________________________________________________<br>
> OpenStack Development Mailing List (not for usage questions)<br>
> Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</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>
><br>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</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>
</div></div></blockquote></div><br></div>