[openstack-dev] [Ironic] March core team review
As March has come to a close, and Juno is open for development, I would like to look at our review stats and see if the core review team should be adjusted to reflect current activity. Also, since I believe that our development pace needs to accelerate, I would like to increase the size of the team from its current size of six. As a quick outline of what core means within this team: in my view, it's a combination of how active and effective someone's reviews are, and how much they participate in relevant discussions. Ideally, cores should aim for about two reviews per work day, or about 40 per month, but it's not a hard limit, and I don't believe we should remove folks simply because their review stats slip below this line if their input is still felt and valued within the project. With ~160 reviews submitted in the last month, in an ideal situation, we would be able to keep up with submissions if we had a review team size of 8 (since it takes a minimum of two cores to land a patch). Given that reviews have been taking an average of 2.8 patch sets, we would actually fall behind if reviewers didn't do more than 2/day - and for a large part of Icehouse, we were pretty far behind... For this reason, reviews by non-core members are extremely helpful because they often catch issues early and allow core reviewers to focus on patches that have already received some +1's. Here are the current 90-day stats, cut at the point where folks are meeting the suggested quantity of reviews. http://russellbryant.net/openstack-stats/ironic-reviewers-90.txt ++---++ | Reviewer | Reviews -2 -1 +1 +2 +A+/- % | Disagreements* | ++---++ |devananda **| 358 22 101 7 228 13265.6% | 10 ( 2.8%) | | lucasagomes ** | 3164 99 4 209 6167.4% |8 ( 2.5%) | |nobodycam **| 1990 24 0 175 8187.9% | 11 ( 5.5%) | |rloo| 1800 74 106 0 058.9% |8 ( 4.4%) | | whaom| 1530 56 97 0 063.4% | 15 ( 9.8%) | | yuriyz | 1360 57 79 0 058.1% | 14 ( 10.3%) | |max_lobur **| 1311 44 38 48 465.6% |7 ( 5.3%) | So, I'd like to formally propose that Ruby (rloo), Haomeng (whaom), and Yuriy (yuriyz) be added to the core team at this time. I believe they have all been very helpful over the last few months. Regards, Devananda ___ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
Re: [openstack-dev] [Ironic] March core team review
So, I'd like to formally propose that Ruby (rloo), Haomeng (whaom), and Yuriy (yuriyz) be added to the core team at this time. I believe they have all been very helpful over the last few months. +1 for all! Good stuff :) Cheers, Lucas ___ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
Re: [openstack-dev] [Ironic] March core team review
+1 for those guys. On Tue, Apr 8, 2014 at 6:35 PM, Lucas Alvares Gomes lucasago...@gmail.comwrote: So, I'd like to formally propose that Ruby (rloo), Haomeng (whaom), and Yuriy (yuriyz) be added to the core team at this time. I believe they have all been very helpful over the last few months. +1 for all! Good stuff :) Cheers, Lucas ___ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev ___ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
Re: [openstack-dev] [Ironic] March core team review
Nice! Definitely +1 for each :) Best regards, Max Lobur, Python Developer, Mirantis, Inc. Mobile: +38 (093) 665 14 28 Skype: max_lobur 38, Lenina ave. Kharkov, Ukraine www.mirantis.com www.mirantis.ru On Tue, Apr 8, 2014 at 6:37 PM, Roman Prykhodchenko rprikhodche...@mirantis.com wrote: +1 for those guys. On Tue, Apr 8, 2014 at 6:35 PM, Lucas Alvares Gomes lucasago...@gmail.com wrote: So, I'd like to formally propose that Ruby (rloo), Haomeng (whaom), and Yuriy (yuriyz) be added to the core team at this time. I believe they have all been very helpful over the last few months. +1 for all! Good stuff :) Cheers, Lucas ___ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev ___ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev ___ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
Re: [openstack-dev] [Ironic] March core team review
+1 for all. Each has demonstrated a good understanding of the project and its goals. Chris On Apr 8, 2014 9:31 AM, Devananda van der Veen devananda@gmail.com wrote: As March has come to a close, and Juno is open for development, I would like to look at our review stats and see if the core review team should be adjusted to reflect current activity. Also, since I believe that our development pace needs to accelerate, I would like to increase the size of the team from its current size of six. As a quick outline of what core means within this team: in my view, it's a combination of how active and effective someone's reviews are, and how much they participate in relevant discussions. Ideally, cores should aim for about two reviews per work day, or about 40 per month, but it's not a hard limit, and I don't believe we should remove folks simply because their review stats slip below this line if their input is still felt and valued within the project. With ~160 reviews submitted in the last month, in an ideal situation, we would be able to keep up with submissions if we had a review team size of 8 (since it takes a minimum of two cores to land a patch). Given that reviews have been taking an average of 2.8 patch sets, we would actually fall behind if reviewers didn't do more than 2/day - and for a large part of Icehouse, we were pretty far behind... For this reason, reviews by non-core members are extremely helpful because they often catch issues early and allow core reviewers to focus on patches that have already received some +1's. Here are the current 90-day stats, cut at the point where folks are meeting the suggested quantity of reviews. http://russellbryant.net/openstack-stats/ironic-reviewers-90.txt ++---++ | Reviewer | Reviews -2 -1 +1 +2 +A+/- % | Disagreements* | ++---++ |devananda **| 358 22 101 7 228 13265.6% | 10 ( 2.8%) | | lucasagomes ** | 3164 99 4 209 6167.4% |8 ( 2.5%) | |nobodycam **| 1990 24 0 175 8187.9% | 11 ( 5.5%) | |rloo| 1800 74 106 0 058.9% |8 ( 4.4%) | | whaom| 1530 56 97 0 063.4% | 15 ( 9.8%) | | yuriyz | 1360 57 79 0 058.1% | 14 ( 10.3%) | |max_lobur **| 1311 44 38 48 465.6% |7 ( 5.3%) | So, I'd like to formally propose that Ruby (rloo), Haomeng (whaom), and Yuriy (yuriyz) be added to the core team at this time. I believe they have all been very helpful over the last few months. Regards, Devananda ___ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev ___ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
