<div dir="ltr"><div>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....</div>
<div><br></div><div>Here are the stats for the last month [1]</div><div><span style="font-family:'courier new',monospace;white-space:pre-wrap"><br></span></div><div><span style="font-family:'courier new',monospace;white-space:pre-wrap"> Total reviews: 569 (19.0/day)</span></div>
<div><span style="font-family:'courier new',monospace;white-space:pre-wrap"> Total reviewers: 28 (avg 0.7 reviews/day)</span></div><div><span style="font-family:'courier new',monospace;white-space:pre-wrap"> Total reviews by core team: 211 (7.0/day)</span></div>
<div><span style="font-family:'courier new',monospace;white-space:pre-wrap"> Core team size: 6 (avg 1.2 reviews/day)</span></div><div><span style="font-family:'courier new',monospace;white-space:pre-wrap"> New patch sets in the last 30 days: 347 (11.6/day)</span></div>
<div><span style="font-family:'courier new',monospace;white-space:pre-wrap"> Changes involved in the last 30 days: 119 (4.0/day)</span></div><div><span style="font-family:'courier new',monospace;white-space:pre-wrap"> New changes in the last 30 days: 93 (3.1/day)</span></div>
<div><span style="font-family:'courier new',monospace;white-space:pre-wrap"> Changes merged in the last 30 days: 56 (1.9/day)</span></div><div><span style="font-family:'courier new',monospace;white-space:pre-wrap"> Changes abandoned in the last 30 days: 13 (0.4/day)</span></div>
<div><span style="font-family:'courier new',monospace;white-space:pre-wrap"> Changes left in state WIP in the last 30 days: 4 (0.1/day)</span></div><div><span style="font-family:'courier new',monospace;white-space:pre-wrap"> Queue growth in the last 30 days: 20 (0.7/day)</span></div>
<div><span style="font-family:'courier new',monospace;white-space:pre-wrap"> Average number of patches per changeset: 2.9</span></div><div><font face="courier new, monospace"><br></font></div><div><br></div><div>
And here are the current / average stats [2]<font face="courier new, monospace"><br></font></div><div><font face="courier new, monospace"><br></font></div><div><font face="courier new, monospace"> Total Open Reviews: 48<br>
Waiting on Submitter: 16<br> Waiting on Reviewer: 32<br> Stats since the latest revision:<br> Average wait time: 6 days, 21 hours, 8 minutes<br> 1rd quartile wait time: 3 days, 15 hours, 59 minutes<br> Median wait time: 5 days, 13 hours, 4 minutes<br>
3rd quartile wait time: 10 days, 6 hours, 2 minutes<br> Number waiting more than 7 days: 14</font><div><br></div><div>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.</div>
<div><br></div><div>So, I'd like to nominate the following two additions to the ironic-core team:</div><div><br></div><div>Max Lobur</div><div><a href="https://review.openstack.org/#/q/reviewer:mlobur%2540mirantis.com+project:openstack/ironic,n,z" style="font-family:arial,sans-serif;font-size:13px">https://review.openstack.org/#/q/reviewer:mlobur%2540mirantis.com+project:openstack/ironic,n,z</a></div>
<div><br></div><div>Roman Prykhodchenko</div><div><a href="https://review.openstack.org/#/q/reviewer:rprikhodchenko%2540mirantis.com+project:openstack/ironic,n,z" style="font-family:arial,sans-serif;font-size:13px">https://review.openstack.org/#/q/reviewer:rprikhodchenko%2540mirantis.com+project:openstack/ironic,n,z</a><br>
</div><div><br></div><div>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.</div><div><br></div>
<div>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.</div>
<div><br></div><div>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.</div>
<div><br></div><div>With this, we would have one core in NZ, two in US, and three in EU time zones.</div><div><br></div><div><br></div><div>Regards,</div><div>Devananda</div><div><br></div><div><br></div><div><div style="font-family:arial,sans-serif;font-size:13px">
[1] - <a href="http://russellbryant.net/openstack-stats/ironic-reviewers-30.txt" target="_blank">http://russellbryant.net/openstack-stats/ironic-reviewers-30.txt</a><br>
</div><div style="font-family:arial,sans-serif;font-size:13px">[2] - <a href="http://russellbryant.net/openstack-stats/ironic-openreviews.html" target="_blank">http://russellbryant.net/openstack-stats/ironic-openreviews.html</a></div>
</div><div style="font-family:arial,sans-serif;font-size:13px">[3] - <a href="https://review.openstack.org/#/q/owner:rprikhodchenko%2540mirantis.com,n,z">https://review.openstack.org/#/q/owner:rprikhodchenko%2540mirantis.com,n,z</a></div>
</div></div>