+1 to Doug’s addition. He’s been a workhorse in moving lbaas and the services split forward and his being able to finally merge code in the main repo should be a boon to our merge velocity.
Many thanks to Sumit for his long-serving dedication as a core reviewer. I hope to see him return to the fold soon! Maru > On Jan 15, 2015, at 2: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