Open Stack

Greetings,

The key metric I have been using for knowing whether we are keeping up
with review requests is the average wait time for getting a review.  In
a previous thread, we set a goal of keeping that under 4 days (at least
by the end of the week, may be higher after a weekend).  This is
calculated using the time that the *latest* patch revision was posted.
We have been keeping up with this (Nova at 3.5 days right now).

I've been getting a lot of complaints this week about review turnaround.
 It's important to me that we're doing this well, but action needs to be
based on real data.

One of the theories was that patches are having to be rebased a bunch,
so they have been waiting longer than the stats say.  True, but by how
much?  The answer is now in the stats:

http://russellbryant.net/openstack-stats/all-openreviews.html

The results are much better than I was afraid of.  On average across all
projects, patches waiting for review have an age of just under 14 days
since they were first posted.  Nova is below average, sitting at an
average of just over 10 days.  That doesn't seem bad at all, to me.

So, if we have a problem, it's not Nova specific, at least.  It's harder
to set a goal for this metric since it's not entirely in the hands of
reviewers like the other one.

Suggestions for additional tweaks welcome.

Thanks,

-- 
Russell Bryant