<div dir="ltr"><div><div>review response time matches up to accepted practices in infosec for quantifying risk exposure in terms of incident response times.<br><br></div>so this is actually a suggest best practice in other areas.<br>
<br></div>-matt<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jun 28, 2013 at 10:30 AM, Russell Bryant <span dir="ltr"><<a href="mailto:rbryant@redhat.com" target="_blank">rbryant@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 06/28/2013 12:18 PM, Matt Riedemann wrote:<br>
> Hey I made the list!<br>
><br>
</div>> _<a href="https://review.openstack.org/#/c/25355/_" target="_blank">https://review.openstack.org/#/c/25355/_</a><br>
<div class="im">><br>
> Just wanted to point out for nova in longest-waiting reviews based on<br>
> first revision:<br>
><br>
><br>
> 1.        94 days, 12 hours, 49 minutes<br>
</div>> - _<a href="https://review.openstack.org/25355_" target="_blank">https://review.openstack.org/25355_</a> (PowerVM resize and migrate test<br>
<div class="im">> cases)<br>
><br>
> This one is a bit skewed because it was abandoned due to inactivity and<br>
> then I picked it back up by assigning the bug to myself and contributing<br>
> to the original review.<br>
><br>
> Is there a way to take that into account in the metrics?  Or is this a<br>
> process issue, i.e. should I have left this abandoned and pushed up a<br>
> new review based on the original?<br>
<br>
</div>I think what you did is definitely the right thing to do.  The stat is<br>
still accurate for how long it is taking to get the patch to completion.<br>
 Having some really high numbers here doesn't worry me, because it's<br>
reality, and is often out of the control of reviewers.<br>
<div class="HOEnZb"><div class="h5"><br>
--<br>
Russell Bryant<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>
</div></div></blockquote></div><br></div>