+1 for Gerrit.
> -----Original Message----- > From: Sateesh Chodapuneedi [mailto:sateesh.chodapune...@citrix.com] > Sent: Monday, February 04, 2013 11:43 AM > To: cloudstack-dev@incubator.apache.org > Subject: RE: [ACS41] 4.1 branch created > > +1 for review process through Gerrit > > > -----Original Message----- > > From: Chip Childers [mailto:chip.child...@sungard.com] > > Sent: 04 February 2013 00:20 > > To: cloudstack-dev@incubator.apache.org > > Subject: Re: [ACS41] 4.1 branch created > > > > On Sun, Feb 3, 2013 at 2:23 AM, Marcus Sorensen > <shadow...@gmail.com> > > wrote: > > > Are you thinking there should be a regular moratorium or something > > > similar just before the cut, so that the quality of the features as > > > a whole can be evaluated, or are you just concerned that the last > > > minute features didn't get proper review? I think that as long as > > > there's a time-based release were going to have features rushed, we > > > either need to be OK with it and allow for time and ability to fix > > > it afterward, or have some very stringent quality control prior to > > > merge. We can maybe start with the former and work toward the latter. > > > > That's exactly right. We're either going to do time-based release > > schedules, or we're not. And if we are doing time-based releases we > > need to be concerned about the quality of commits coming into master. > > Anyone want to get gerrit up and running? I'd be more than happy if > > we had a strict review process for ALL commits coming into master, > > regardless of commit access permissions.