Re: [openstack-dev] [reviews] putting numbers on -core team load

2013-11-14 Thread Christopher Yeoh
On Thu, Nov 14, 2013 at 5:59 PM, Robert Collins robe...@robertcollins.netwrote: Total reviews: 10705 (118.9/day) Total reviewers: 406 Total reviews by core team: 5289 (58.8/day) Core team size: 17 New patch sets in the last 90 days: 7515 (83.5/day) This is the really interesting bit.

Re: [openstack-dev] [reviews] putting numbers on -core team load

2013-11-14 Thread Christopher Yeoh
On Fri, Nov 15, 2013 at 12:04 AM, Christopher Yeoh cbky...@gmail.comwrote: On Thu, Nov 14, 2013 at 5:59 PM, Robert Collins robe...@robertcollins.net wrote: So if everyone uploaded perfect changesets we'd only need 40 core reviews per day :-) And I should add in terms of changesets that are

Re: [openstack-dev] [reviews] putting numbers on -core team load

2013-11-14 Thread Daniel P. Berrange
On Thu, Nov 14, 2013 at 08:29:36PM +1300, Robert Collins wrote: At the summit there were lots of discussions about reviews, and I made the mistake of sending a mail to Russell proposing a few new stats we could gather. I say mistake, because he did so and then some... we new have extra info

Re: [openstack-dev] [reviews] putting numbers on -core team load

2013-11-14 Thread Sean Dague
On 11/14/2013 09:03 AM, Daniel P. Berrange wrote: On Thu, Nov 14, 2013 at 08:29:36PM +1300, Robert Collins wrote: At the summit there were lots of discussions about reviews, and I made the mistake of sending a mail to Russell proposing a few new stats we could gather. I say mistake, because

Re: [openstack-dev] [reviews] putting numbers on -core team load

2013-11-14 Thread Robert Collins
On 15 November 2013 03:03, Daniel P. Berrange berra...@redhat.com wrote: On Thu, Nov 14, 2013 at 08:29:36PM +1300, Robert Collins wrote: One confounding factor here is that this counts (AIUI) pushed changes, not change ids - so we don't need two +2's for every push, we need two +2's for every

Re: [openstack-dev] [reviews] putting numbers on -core team load

2013-11-14 Thread Robert Collins
On 15 November 2013 03:27, Sean Dague s...@dague.net wrote: here, but in general). Having a lot of eager contributors, a small number of people that understand the implications to Nova core, and a finite time to build shared culture to make sure it's not a bag of parts that doesn't hold

[openstack-dev] [reviews] putting numbers on -core team load

2013-11-13 Thread Robert Collins
At the summit there were lots of discussions about reviews, and I made the mistake of sending a mail to Russell proposing a few new stats we could gather. I say mistake, because he did so and then some... we new have extra info - consider: