<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Nov 14, 2013 at 5:59 PM, Robert Collins <span dir="ltr"><<a href="mailto:robertc@robertcollins.net" target="_blank">robertc@robertcollins.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Total reviews: 10705 (118.9/day)<br>
Total reviewers: 406<br>
Total reviews by core team: 5289 (58.8/day)<br>
Core team size: 17<br>
New patch sets in the last 90 days: 7515 (83.5/day)<br>
<br>
This is the really interesting bit. Remembering that every patch needs<br>
- at minimum - 2 +2's, the *minimum* viable core team review rate to<br>
keep up is patch sets per day * 2:<br>
30 days: 132 core reviews/day<br>
90 days: 167 core reviews/day<br>
<br>
But we're getting:<br>
30 days: 42/day or 90/day short<br>
90 days: 59/day or 108/day short<br>
<br>
One confounding factor here is that this counts (AIUI) pushed changes,<br>
not change ids - so we don't need two +2's for every push, we need two<br>
+2's for every changeid - we should add that as a separate metric I<br>
think, as the needed +2 count will be a bit lower.<br>
<br></blockquote><div><br></div><div>So I thought that can make quite a difference to the calculations you made below<br>so just for fun I added a few more stats (<a href="https://review.openstack.org/#/c/56380/">https://review.openstack.org/#/c/56380/</a>) and got:<br>
<br>Total reviews: 10751 (119.5/day)<br>Total reviewers: 405<br>Total reviews by core team: 5312 (59.0/day)<br>Core team size: 17<br>New patch sets in the last 90 days: 7501 (83.3/day)<br>Changes involved in the last 90 days: 1840 (20.4/day)<br>
  New changes in the last 90 days: 1549 (17.2/day)<br>  Changes merged in the last 90 days: 1120 (12.4/day)<br>  Changes abandoned in the last 90 days: 395 (4.4/day)<br>  Changes left in state WIP in the last 90 days: 18 (0.2/day)<br>
  Queue growth in the last 90 days: 16 (0.2/day)<br>  Average number of patches per changeset: 4.1<br><br></div>So if everyone uploaded perfect changesets we'd only need 40 core reviews per day :-)<br><div>Though in practice it takes on average about 4 tries for Nova. Obviously some of those updated patchsets<br>
are due to automatic feedback from Jenkins and -1 could come from anyone, but in practice<br></div><div>of course cores review a lot of patches in progress rather than just when they're ready. Though the more<br>non-cores picking up issues, the less that needs to occur.<br>
</div><div><br></div><div>Queue growth is a derived number which I think is correct as its based on new changes versus ones which merge or are abandoned (but I might be wrong). Its not necessarily a problem as long as the delay through the queue does <br>
not increase as the queue length is going to grow as a project becomes more active if the time taken to get through the queue remains the same. It also probably changes a bit depending on the exact time slice of the development period you look at.<br>
<br></div><div>Chris<br></div></div></div></div>