I have worked with both Max and Roman for sometime now, Both are very active in the Ironic IRC channel and are a pleasure to work with. They both have good grasp of the projects goals, and I look forward to having them as Ironic Core members.
So with that I fully +1 both Max and Roman. (do two +1's equal one +2?) Chris Krelle NobodyCam On Tue, Feb 4, 2014 at 11:42 AM, Devananda van der Veen < devananda....@gmail.com> wrote: > The last month and a half had most of our team out for holiday leave at > some point, and the review stats reflect that. I had hoped our review queue > would come down once we all got back from the holidays, but that hasn't > happened. In fact, our review queue has grown significantly .... Perhaps > it's a combination of the usual nearing-end-of-cycle-rush and our gate > breaking twice in the last 10 days.... > > Here are the stats for the last month [1] > > Total reviews: 569 (19.0/day) > Total reviewers: 28 (avg 0.7 reviews/day) > Total reviews by core team: 211 (7.0/day) > Core team size: 6 (avg 1.2 reviews/day) > New patch sets in the last 30 days: 347 (11.6/day) > Changes involved in the last 30 days: 119 (4.0/day) > New changes in the last 30 days: 93 (3.1/day) > Changes merged in the last 30 days: 56 (1.9/day) > Changes abandoned in the last 30 days: 13 (0.4/day) > Changes left in state WIP in the last 30 days: 4 (0.1/day) > Queue growth in the last 30 days: 20 (0.7/day) > Average number of patches per changeset: 2.9 > > > And here are the current / average stats [2] > > Total Open Reviews: 48 > Waiting on Submitter: 16 > Waiting on Reviewer: 32 > Stats since the latest revision: > Average wait time: 6 days, 21 hours, 8 minutes > 1rd quartile wait time: 3 days, 15 hours, 59 minutes > Median wait time: 5 days, 13 hours, 4 minutes > 3rd quartile wait time: 10 days, 6 hours, 2 minutes > Number waiting more than 7 days: 14 > > I would very much like to add a few people to our core review team. We > need to increase the pace of reviews to keep up with development, > particularly as we approach our most aggressive milestone and prepare for > our first release. I'd also like to improve our non-US-timezone coverage. > > So, I'd like to nominate the following two additions to the ironic-core > team: > > Max Lobur > > https://review.openstack.org/#/q/reviewer:mlobur%2540mirantis.com+project:openstack/ironic,n,z > > Roman Prykhodchenko > > https://review.openstack.org/#/q/reviewer:rprikhodchenko%2540mirantis.com+project:openstack/ironic,n,z > > I believe that the review feedback that I've seen from both of them shows > a good understanding of the project architecture and the direction that I'd > like Ironic to go. > > Max has been consistently reviewing patches for the last few months. His > input has been very valuable in spotting issues early on, and clearly show > a good grasp of the project's architecture. He is frequently engaged with > the existing team during discussions in IRC and in the weekly meetings. > > Roman was involved in Ironic early on, then spent a few months focusing on > our devstack and tempest patches [3]. He continues to help with the > ironic-related work in those projects, is engaged in discussions both in > channel and during meetings, and has resumed doing reviews on a regular > basis. > > With this, we would have one core in NZ, two in US, and three in EU time > zones. > > > Regards, > Devananda > > > [1] - http://russellbryant.net/openstack-stats/ironic-reviewers-30.txt > [2] - http://russellbryant.net/openstack-stats/ironic-openreviews.html > [3] - > https://review.openstack.org/#/q/owner:rprikhodchenko%2540mirantis.com,n,z > > _______________________________________________ > 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