Re: [openstack-dev] [OpenStack-Dev][Cinder] Review days? (open to ANYBODY and EVERYBODY)

2014-06-18 Thread Erlon Cruz
Great John,

I'll push fellows here to help. Couldn't  join today but I'll put in my
schedule for the next weeks. Do you plan to do this review force every
Wednesday?

Erlon


On Mon, Jun 16, 2014 at 9:31 AM, Kerr, Andrew andrew.k...@netapp.com
wrote:

 +1

 Andrew Kerr


 On 6/13/14, 10:30 AM, Duncan Thomas duncan.tho...@gmail.com wrote:

 Same as Jay, for much the same reasons. Having a fixed calendar time
 makes it easy for me to put up a 'do not disturb' sign.
 
 On 13 June 2014 05:10, Jay Bryant jsbry...@electronicjungle.net wrote:
  John,
 
  +2
 
  I am guilty of falling behind on reviews. Pulled in to a lot of other
 stuff
  since the summit ... and before.
 
  Having prescribed time on my calendar is a good idea.  Just put it on my
  calendar.
 
  Jay
 
  On Jun 12, 2014 10:49 PM, John Griffith john.griff...@solidfire.com
  wrote:
 
  Hey Everyone,
 
  So I've been noticing some issues with regards to reviews in Cinder
  lately, namely we're not keeping up very well.  Most of this is a math
  problem (submitters  reviewers).  We're up around 200+ patches in the
  queue, and a large number of them have no negative feedback but have
 just
  been waiting patiently (some  2 months).
 
  Growth is good, new contributors are FANTASTIC... but stale
 submissions in
  the queue are BAD, and I hate for people interested in contributing to
  become discouraged and just go away (almost as much as I hate emails
 asking
  me to review patches).
 
  I'd like to propose we consider one or two review days a week for a
 while
  to try and work on our backlog.  I'd like to propose that on these
 days we
  make an attempt to NOT propose new code (or at least limit it to
 bug-fixes
  [real bugs, not features disguised as bugs]) and have an agreement from
  folks to focus on actually doing reviews and using IRC to collaborate
  together and knock some of these out.
 
  We did this sort of thing over a virtual meetup and it was really
  effective, I'd like to see if we can't do something for a brief
 duration
  over IRC.
 
  I'm thinking we give it a test run, set aside a few hours next Wed
 morning
  to start (coinciding with our Cinder weekly meeting since many folks
 around
  that morning across TZ's etc) where we all dedicate some time prior to
 the
  meeting to focus exclusively on helping each other get some reviews
 knocked
  out.  As a reminder Cinder weekly meeting is 16:00 UTC.
 
  Let me know what you all think, and keep in mind this is NOT limited to
  just current regular Block-Heads but anybody in the OpenStack
 community
  that's willing to help out and of course new reviewers are MORE than
  welcome.
 
  Thanks,
  John
 
  ___
  OpenStack-dev mailing list
  OpenStack-dev@lists.openstack.org
  http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
 
 
  ___
  OpenStack-dev mailing list
  OpenStack-dev@lists.openstack.org
  http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
 
 
 
 
 --
 Duncan Thomas
 
 ___
 OpenStack-dev mailing list
 OpenStack-dev@lists.openstack.org
 http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


 ___
 OpenStack-dev mailing list
 OpenStack-dev@lists.openstack.org
 http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

___
OpenStack-dev mailing list
OpenStack-dev@lists.openstack.org
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


Re: [openstack-dev] [OpenStack-Dev][Cinder] Review days? (open to ANYBODY and EVERYBODY)

2014-06-13 Thread Duncan Thomas
Same as Jay, for much the same reasons. Having a fixed calendar time
makes it easy for me to put up a 'do not disturb' sign.

On 13 June 2014 05:10, Jay Bryant jsbry...@electronicjungle.net wrote:
 John,

 +2

 I am guilty of falling behind on reviews. Pulled in to a lot of other stuff
 since the summit ... and before.

 Having prescribed time on my calendar is a good idea.  Just put it on my
 calendar.

 Jay

 On Jun 12, 2014 10:49 PM, John Griffith john.griff...@solidfire.com
 wrote:

 Hey Everyone,

 So I've been noticing some issues with regards to reviews in Cinder
 lately, namely we're not keeping up very well.  Most of this is a math
 problem (submitters  reviewers).  We're up around 200+ patches in the
 queue, and a large number of them have no negative feedback but have just
 been waiting patiently (some  2 months).

 Growth is good, new contributors are FANTASTIC... but stale submissions in
 the queue are BAD, and I hate for people interested in contributing to
 become discouraged and just go away (almost as much as I hate emails asking
 me to review patches).

 I'd like to propose we consider one or two review days a week for a while
 to try and work on our backlog.  I'd like to propose that on these days we
 make an attempt to NOT propose new code (or at least limit it to bug-fixes
 [real bugs, not features disguised as bugs]) and have an agreement from
 folks to focus on actually doing reviews and using IRC to collaborate
 together and knock some of these out.

 We did this sort of thing over a virtual meetup and it was really
 effective, I'd like to see if we can't do something for a brief duration
 over IRC.

 I'm thinking we give it a test run, set aside a few hours next Wed morning
 to start (coinciding with our Cinder weekly meeting since many folks around
 that morning across TZ's etc) where we all dedicate some time prior to the
 meeting to focus exclusively on helping each other get some reviews knocked
 out.  As a reminder Cinder weekly meeting is 16:00 UTC.

 Let me know what you all think, and keep in mind this is NOT limited to
 just current regular Block-Heads but anybody in the OpenStack community
 that's willing to help out and of course new reviewers are MORE than
 welcome.

 Thanks,
 John

 ___
 OpenStack-dev mailing list
 OpenStack-dev@lists.openstack.org
 http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


 ___
 OpenStack-dev mailing list
 OpenStack-dev@lists.openstack.org
 http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev




-- 
Duncan Thomas

___
OpenStack-dev mailing list
OpenStack-dev@lists.openstack.org
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


Re: [openstack-dev] [OpenStack-Dev][Cinder] Review days? (open to ANYBODY and EVERYBODY)

2014-06-12 Thread Jay Bryant
John,

+2

I am guilty of falling behind on reviews. Pulled in to a lot of other stuff
since the summit ... and before.

Having prescribed time on my calendar is a good idea.  Just put it on my
calendar.

Jay
On Jun 12, 2014 10:49 PM, John Griffith john.griff...@solidfire.com
wrote:

 Hey Everyone,

 So I've been noticing some issues with regards to reviews in Cinder
 lately, namely we're not keeping up very well.  Most of this is a math
 problem (submitters  reviewers).  We're up around 200+ patches in the
 queue, and a large number of them have no negative feedback but have just
 been waiting patiently (some  2 months).

 Growth is good, new contributors are FANTASTIC... but stale submissions in
 the queue are BAD, and I hate for people interested in contributing to
 become discouraged and just go away (almost as much as I hate emails asking
 me to review patches).

 I'd like to propose we consider one or two review days a week for a while
 to try and work on our backlog.  I'd like to propose that on these days we
 make an attempt to NOT propose new code (or at least limit it to bug-fixes
 [real bugs, not features disguised as bugs]) and have an agreement from
 folks to focus on actually doing reviews and using IRC to collaborate
 together and knock some of these out.

 We did this sort of thing over a virtual meetup and it was really
 effective, I'd like to see if we can't do something for a brief duration
 over IRC.

 I'm thinking we give it a test run, set aside a few hours next Wed morning
 to start (coinciding with our Cinder weekly meeting since many folks around
 that morning across TZ's etc) where we all dedicate some time prior to the
 meeting to focus exclusively on helping each other get some reviews knocked
 out.  As a reminder Cinder weekly meeting is 16:00 UTC.

 Let me know what you all think, and keep in mind this is NOT limited to
 just current regular Block-Heads but anybody in the OpenStack community
 that's willing to help out and of course new reviewers are MORE than
 welcome.

 Thanks,
 John

 ___
 OpenStack-dev mailing list
 OpenStack-dev@lists.openstack.org
 http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


___
OpenStack-dev mailing list
OpenStack-dev@lists.openstack.org
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev