Open Stack

Hi all!

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.

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.

Without further ado, here are the stats!

Reviews for the last 30 days in ironic
** -- ironic-core team member
+------------------+---------------------------------------+----------------+
|     Reviewer     | Reviews   -2  -1  +1  +2  +A    +/- % | Disagreements*
|
+------------------+---------------------------------------+----------------+
|     dtantsur     |     124    0  56  68   0   0    54.8% |   17 ( 13.7%)
 |
|  lucasagomes **  |     114    4  32   4  74  16    68.4% |    4 (  3.5%)
 |
|       mrda       |     102    0  14  88   0   0    86.3% |   10 (  9.8%)
 |
|     rloo **      |      56    0  24   2  30  12    57.1% |    2 (  3.6%)
 |
|   devananda **   |      54    7  24   2  21  11    42.6% |    0 (  0.0%)
 |
|     whaom **     |      46    0   4   4  38   8    91.3% |    6 ( 13.0%)
 |
|    yuriyz **     |      38    0  12   3  23   4    68.4% |    8 ( 21.1%)
 |
|  jimrollenhagen  |      34    1  12  14   7   3    61.8% |    8 ( 23.5%)
 |
|    ghe.rivero    |      33    0   1  32   0   0    97.0% |    9 ( 27.3%)
 |
|     JoshNang     |      32    0   3  13  16   3    90.6% |    1 (  3.1%)
 |
|     dshrews      |      29    0   5  24   0   0    82.8% |    0 (  0.0%)
 |
...
Total reviews: 836 (27.9/day)
Total reviewers: 37 (avg 0.8 reviews/day)
Total reviews by core team: 352 (11.7/day)
Core team size: 9 (*avg 1.3 reviews/day*)
New patch sets in the last 30 days: 642 (21.4/day)
Changes involved in the last 30 days: 153 (5.1/day)
  New changes in the last 30 days: 96 (3.2/day)
  Changes merged in the last 30 days: 62 (2.1/day)
  Changes abandoned in the last 30 days: 12 (0.4/day)
  Changes left in state WIP in the last 30 days: 0 (0.0/day)
  *Queue growth in the last 30 days: 22 (0.7/day)*
  Average number of patches per changeset: 4.2




Reviews for the last 90 days in ironic
** -- ironic-core team member
+------------------+---------------------------------------+----------------+
|     Reviewer     | Reviews   -2  -1  +1  +2  +A    +/- % | Disagreements*
|
+------------------+---------------------------------------+----------------+
|  lucasagomes **  |     346    7 104  12 223  59    67.9% |   15 (  4.3%)
 |
|   devananda **   |     322   19 108  10 185 109    60.6% |   13 (  4.0%)
 |
|     dtantsur     |     238    0  87 151   0   0    63.4% |   37 ( 15.5%)
 |
|       mrda       |     178    0  17 161   0   0    90.4% |   19 ( 10.7%)
 |
|     rloo **      |     167    0  74  47  46  19    55.7% |    6 (  3.6%)
 |
|   nobodycam **   |     142    0  15   2 125  64    89.4% |    8 (  5.6%)
 |
|  jimrollenhagen  |     142    2  45  60  35  16    66.9% |   20 ( 14.1%)
 |
|     whaom **     |     124    0  15  61  48   8    87.9% |   18 ( 14.5%)
 |
|    yuriyz **     |     123    0  46  36  41   7    62.6% |   18 ( 14.6%)
 |
|     JoshNang     |     119    0   9  68  42  18    92.4% |   20 ( 16.8%)
 |
|       JayF       |     108    5  29  37  37  16    68.5% |    8 (  7.4%)
 |
|     dshrews      |      81    0  18  63   0   0    77.8% |    7 (  8.6%)
 |
|    russell_h     |      78    1  19  36  22   7    74.4% |   10 ( 12.8%)
 |
|  mdurnosvistov   |      71    0  21  50   0   0    70.4% |   12 ( 16.9%)
 |
|    ghe.rivero    |      70    0   8  62   0   0    88.6% |   15 ( 21.4%)
 |
|     agordeev     |      63    0  20  43   0   0    68.3% |    6 (  9.5%)
 |
| prykhodchenko ** |      50    3  10   0  37  14    74.0% |    1 (  2.0%)
 |
|     cbehrens     |      43    0  32  11   0   0    25.6% |    1 (  2.3%)
 |
|   max_lobur **   |      34    0  13   8  13   0    61.8% |    5 ( 14.7%)
 |
|   lifeless **    |      32    2  20   0  10   2    31.2% |    0 (  0.0%)
 |
|   YuikoTakada    |      25    0   6  19   0   0    76.0% |    4 ( 16.0%)
 |
|    kozhukalov    |      22    0  11  11   0   0    50.0% |    0 (  0.0%)
 |
|     linggao      |      17    0   6  11   0   0    64.7% |    2 ( 11.8%)
 |
|    jaegerandi    |      14    0   1  13   0   0    92.9% |    0 (  0.0%)
 |
|     snikitin     |      14    0   0  14   0   0   100.0% |    4 ( 28.6%)
 |
|    matty_dubs    |      11    0   1  10   0   0    90.9% |    1 (  9.1%)
 |
|    sbookworm     |      10    0   0  10   0   0   100.0% |    2 ( 20.0%)
 |



Total reviews: 2708 (30.1/day)
Total reviewers: 52 (avg 0.6 reviews/day)
Total reviews by core team: 1340 (14.9/day)
Core team size: 9 (*avg 1.7 reviews/day*)
New patch sets in the last 90 days: 1947 (21.6/day)
Changes involved in the last 90 days: 453 (5.0/day)
  New changes in the last 90 days: 403 (4.5/day)
  Changes merged in the last 90 days: 298 (3.3/day)
  Changes abandoned in the last 90 days: 67 (0.7/day)
  Changes left in state WIP in the last 90 days: 0 (0.0/day)
*  Queue growth in the last 90 days: 38 (0.4/day)*
  Average number of patches per changeset: 4.3
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20140526/e980b876/attachment.html>