<div dir="ltr">Hi all!<div><br></div><div>It's that time again -- time to look at the core team and see if any changes are in order. Over the last month, we've continued to see an increase in the size of the review queue, and folks are (understandably) getting frustrated at how long it is taking to land any changes. We have had an average of 21 patch sets per day, with a total average of only 12 reviews by core team members per day! Granted, reviews slowed down when most of the core team was at the summit, but if I look at the previous month, the average isn't much better (15 per day). So this email is as much to look at who should be added to the team as it is to remind existing core reviewers of their responsibility to help keep the pace of development by committing to review at least a few patch sets per day.</div>
<div></div><div><br></div><div>In addition to adding Dmitry to the core team, which I proposed in another email, I'd like to remove Max Lobur from the team. He hasn't been active over the last three months, but if he returns to working on Ironic, I'd be happy to fast-track him back into core.</div>
<div><br></div><div>Without further ado, here are the stats!</div><div><br></div><div><font face="courier new, monospace">Reviews for the last 30 days in ironic</font></div><div><font face="courier new, monospace">** -- ironic-core team member</font></div>
<div><font face="courier new, monospace">+------------------+---------------------------------------+----------------+</font></div><div><font face="courier new, monospace">| Reviewer | Reviews -2 -1 +1 +2 +A +/- % | Disagreements* |</font></div>
<div><font face="courier new, monospace">+------------------+---------------------------------------+----------------+</font></div><div><font face="courier new, monospace">| dtantsur | 124 0 56 68 0 0 54.8% | 17 ( 13.7%) |</font></div>
<div><font face="courier new, monospace">| lucasagomes ** | 114 4 32 4 74 16 68.4% | 4 ( 3.5%) |</font></div><div><font face="courier new, monospace">| mrda | 102 0 14 88 0 0 86.3% | 10 ( 9.8%) |</font></div>
<div><font face="courier new, monospace">| rloo ** | 56 0 24 2 30 12 57.1% | 2 ( 3.6%) |</font></div><div><font face="courier new, monospace">| devananda ** | 54 7 24 2 21 11 42.6% | 0 ( 0.0%) |</font></div>
<div><font face="courier new, monospace">| whaom ** | 46 0 4 4 38 8 91.3% | 6 ( 13.0%) |</font></div><div><font face="courier new, monospace">| yuriyz ** | 38 0 12 3 23 4 68.4% | 8 ( 21.1%) |</font></div>
<div><font face="courier new, monospace">| jimrollenhagen | 34 1 12 14 7 3 61.8% | 8 ( 23.5%) |</font></div><div><font face="courier new, monospace">| ghe.rivero | 33 0 1 32 0 0 97.0% | 9 ( 27.3%) |</font></div>
<div><font face="courier new, monospace">| JoshNang | 32 0 3 13 16 3 90.6% | 1 ( 3.1%) |</font></div><div><font face="courier new, monospace">| dshrews | 29 0 5 24 0 0 82.8% | 0 ( 0.0%) |</font></div>
<div><span style="font-family:'courier new',monospace">...</span><br></div><div><font face="courier new, monospace"><div><div>Total reviews: 836 (27.9/day)</div><div>Total reviewers: 37 (avg 0.8 reviews/day)</div>
<div>Total reviews by core team: 352 (11.7/day)</div><div>Core team size: 9 (<b>avg 1.3 reviews/day</b>)</div><div>New patch sets in the last 30 days: 642 (21.4/day)</div><div>Changes involved in the last 30 days: 153 (5.1/day)</div>
<div> New changes in the last 30 days: 96 (3.2/day)</div><div> Changes merged in the last 30 days: 62 (2.1/day)</div><div> Changes abandoned in the last 30 days: 12 (0.4/day)</div><div> Changes left in state WIP in the last 30 days: 0 (0.0/day)</div>
<div> <b>Queue growth in the last 30 days: 22 (0.7/day)</b></div><div> Average number of patches per changeset: 4.2</div></div><div><br></div><div><br></div><div><br></div><div><br></div><div><div>Reviews for the last 90 days in ironic</div>
<div>** -- ironic-core team member</div><div>+------------------+---------------------------------------+----------------+</div><div>| Reviewer | Reviews -2 -1 +1 +2 +A +/- % | Disagreements* |</div><div>
+------------------+---------------------------------------+----------------+</div><div>| lucasagomes ** | 346 7 104 12 223 59 67.9% | 15 ( 4.3%) |</div><div>| devananda ** | 322 19 108 10 185 109 60.6% | 13 ( 4.0%) |</div>
<div>| dtantsur | 238 0 87 151 0 0 63.4% | 37 ( 15.5%) |</div><div>| mrda | 178 0 17 161 0 0 90.4% | 19 ( 10.7%) |</div><div>| rloo ** | 167 0 74 47 46 19 55.7% | 6 ( 3.6%) |</div>
<div>| nobodycam ** | 142 0 15 2 125 64 89.4% | 8 ( 5.6%) |</div><div>| jimrollenhagen | 142 2 45 60 35 16 66.9% | 20 ( 14.1%) |</div><div>| whaom ** | 124 0 15 61 48 8 87.9% | 18 ( 14.5%) |</div>
<div>| yuriyz ** | 123 0 46 36 41 7 62.6% | 18 ( 14.6%) |</div><div>| JoshNang | 119 0 9 68 42 18 92.4% | 20 ( 16.8%) |</div><div>| JayF | 108 5 29 37 37 16 68.5% | 8 ( 7.4%) |</div>
<div>| dshrews | 81 0 18 63 0 0 77.8% | 7 ( 8.6%) |</div><div>| russell_h | 78 1 19 36 22 7 74.4% | 10 ( 12.8%) |</div><div>| mdurnosvistov | 71 0 21 50 0 0 70.4% | 12 ( 16.9%) |</div>
<div>| ghe.rivero | 70 0 8 62 0 0 88.6% | 15 ( 21.4%) |</div><div>| agordeev | 63 0 20 43 0 0 68.3% | 6 ( 9.5%) |</div><div>| prykhodchenko ** | 50 3 10 0 37 14 74.0% | 1 ( 2.0%) |</div>
<div>| cbehrens | 43 0 32 11 0 0 25.6% | 1 ( 2.3%) |</div><div>| max_lobur ** | 34 0 13 8 13 0 61.8% | 5 ( 14.7%) |</div><div>| lifeless ** | 32 2 20 0 10 2 31.2% | 0 ( 0.0%) |</div>
<div>| YuikoTakada | 25 0 6 19 0 0 76.0% | 4 ( 16.0%) |</div><div>| kozhukalov | 22 0 11 11 0 0 50.0% | 0 ( 0.0%) |</div><div>| linggao | 17 0 6 11 0 0 64.7% | 2 ( 11.8%) |</div>
<div>| jaegerandi | 14 0 1 13 0 0 92.9% | 0 ( 0.0%) |</div><div>| snikitin | 14 0 0 14 0 0 100.0% | 4 ( 28.6%) |</div><div>| matty_dubs | 11 0 1 10 0 0 90.9% | 1 ( 9.1%) |</div>
<div>| sbookworm | 10 0 0 10 0 0 100.0% | 2 ( 20.0%) |</div></div><div><br></div><div><br></div><div><br></div><div><div>Total reviews: 2708 (30.1/day)</div><div>Total reviewers: 52 (avg 0.6 reviews/day)</div>
<div>Total reviews by core team: 1340 (14.9/day)</div><div>Core team size: 9 (<b>avg 1.7 reviews/day</b>)</div><div>New patch sets in the last 90 days: 1947 (21.6/day)</div><div>Changes involved in the last 90 days: 453 (5.0/day)</div>
<div> New changes in the last 90 days: 403 (4.5/day)</div><div> Changes merged in the last 90 days: 298 (3.3/day)</div><div> Changes abandoned in the last 90 days: 67 (0.7/day)</div><div> Changes left in state WIP in the last 90 days: 0 (0.0/day)</div>
<div><b> Queue growth in the last 90 days: 38 (0.4/day)</b></div><div> Average number of patches per changeset: 4.3</div></div><div><br></div></font></div></div>