+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