[openstack-dev] [TripleO] Spec Minimum Review Proposal

Ben Nemec openstack at nemebean.com
Wed Jul 23 13:21:11 UTC 2014


For everyone's reference, the tripleo-specs stats can be found here:
http://www.nemebean.com/reviewstats/tripleo-specs-30.txt

Note that looking at the stats, over 30 days 1 review per week is only
4, which most of our cores are already doing anyway.  I'm not sure
codifying a requirement to do at least that is going to help much.  To
move the needle I'm thinking we would need at least 3 - most of our
cores aren't meeting that today so it would actually require everyone to
do more reviews.  Spec reviews are difficult and tend to take a
significant amount of time, so that would be a considerable increase in
time commitments for cores.  I'm not sure how I feel about that,
although I'm probably biased because I'm not at 3 per week right now.
:-)

-Ben

On 2014-07-22 15:18, Jay Dobies wrote:
> At the meetup today, the topic of our spec process came up. The
> general sentiment is that the process is still young and the hiccups
> are expected, but we do need to get better about making sure we're
> staying on top of them.
> 
> As a first step, it was proposed to add 1 spec review a week to the
> existing 3 reviews per day requirement for cores.
> 
> Additionally, we're going to start to capture and review the metrics
> on spec patches specifically during the weekly meeting. That should
> help bring to light how long reviews are sitting in the queue without
> being touched.
> 
> What are everyone's feelings on adding a 1 spec review per week
> requirement for cores?
> 
> Not surprisingly, I'm +1 for it  :)
> 
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev



More information about the OpenStack-dev mailing list