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