<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 8, 2014 at 6:12 PM, Michael Still <span dir="ltr"><<a href="mailto:mikal@stillhq.com" target="_blank">mikal@stillhq.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class=""><div class="h5">On Thu, Oct 9, 2014 at 11:58 AM, Joe Gordon <<a href="mailto:joe.gordon0@gmail.com">joe.gordon0@gmail.com</a>> wrote:<br>
> On Wed, Oct 8, 2014 at 4:30 PM, Joe Gordon <<a href="mailto:joe.gordon0@gmail.com">joe.gordon0@gmail.com</a>> wrote:<br>
>><br>
>> Recently there has been a lot of discussion around the development growing<br>
>> pains in nova. Instead of guessing about how bad some of the issues are, I<br>
>> tried to answer a few questions that may help us better understand the<br>
>> issues.<br>
>><br>
>><br>
>> Q: How many revisions does it take to merge a patch?<br>
>><br>
>> Average: 6.76 revisions<br>
>> median: 4.0 revisions<br>
>><br>
>><br>
>> Q: How many rechecks/verifies does it take to merge a patch (ignoring<br>
>> rechecks where the same job failed before and after)?<br>
>><br>
>> Average: 0.749 rechecks per patch revision<br>
>> median: 0.4285  rechecks per patch revision<br>
>><br>
>> For comparison here are the same results for tempest, which has a lot more<br>
>> gating tests:<br>
>><br>
>> Average: 1.01591525738<br>
>> median: 0.6<br>
>><br>
>><br>
>> Q: How long does it take for a patch to get approved?<br>
>><br>
>> Average: 28 days<br>
>> median: 11 days<br>
>><br>
>><br>
>> Q: How long does it take for a patch to get approved that touches<br>
>> 'nova/virt/'?<br>
>><br>
>> Average: 34 days<br>
>> median: 18 days<br>
>><br>
><br>
> To expand on these numbers, same results for last 6 months of commits:<br>
><br>
> all of nova (1723 patches):<br>
> Average: 28.8<br>
> median: 11.0<br>
><br>
> nova/virt (476 patches):<br>
>  Average: 34.5<br>
<br>
</div></div>I think it would be interesting to break this up by driver<br>
directory... Are there drivers which take longer to land code for than<br>
others?<br>
<span class=""><br></span></blockquote><div><br></div><div>Like this?</div><div><br></div><div>subtree: None (1724 patches):<br></div><div><div>Average: 28.7</div><div>median: 11.0</div><div>subtree: nova/virt/ (476 patches):</div><div>Average: 34.5</div><div>median: 18.0</div><div>subtree: nova/virt/hyperv/ (38 patches):</div><div>Average: 46.8</div><div>median: 33.0</div><div>subtree: nova/virt/libvirt/ (224 patches):</div><div>Average: 35.9</div><div>median: 18.0</div><div>subtree: nova/virt/xenapi/ (72 patches):</div><div>Average: 39.5</div><div>median: 20.0</div><div>subtree: nova/virt/vmwareapi/ (134 patches):</div><div>Average: 38.7</div><div>median: 26.0</div></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span class="">
>> When looking at these numbers two things stick out out:<br>
>><br>
>> * We successfully use recheck an awful lot. More then I expected<br>
>> * Patches that touch 'nova/virt' take about 20% more time to land or about<br>
>> 6 days. While that is definitely a difference, its smaller then I expected<br>
>><br>
>><br>
>> Dataset: last 800 patches in nova<br>
>> Code: <a href="https://github.com/jogo/gerrit-fun" target="_blank">https://github.com/jogo/gerrit-fun</a><br>
><br>
><br>
><br>
</span>> _______________________________________________<br>
> OpenStack-dev mailing list<br>
> <a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
><br>
<span class=""><font color="#888888"><br>
Michael<br>
<br>
--<br>
Rackspace Australia<br>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</font></span></blockquote></div><br></div></div>