cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Anthony Xu <Xuefei...@citrix.com>
Subject RE: [DISCUSS][INFRA] Setting up gerrit
Date Fri, 08 Feb 2013 19:11:42 GMT
+1

> -----Original Message-----
> From: Chip Childers [mailto:chip.childers@sungard.com]
> Sent: Friday, February 08, 2013 11:09 AM
> To: Alex Huang
> Cc: cloudstack-dev@incubator.apache.org
> Subject: Re: [DISCUSS][INFRA] Setting up gerrit
> 
> On Fri, Feb 08, 2013 at 11:06:37AM -0800, Alex Huang wrote:
> > Hi everyone,
> >
> > I like to propose that we setup gerrit as the review mechanism.  Here
> are my reasons
> >
> > - Committer status in Apache is a reflection of one's commitment to
> the community, not a reflection of understanding of code.  So to me
> just because you have committer status shouldn't mean code does not
> need review.  Chip's been doing a great job monitoring the merges and
> commits but one person handling all that just means things will slip
> through.
> > - This also has the side effect of contributors' code contribution to
> be treated as a second class citizen with delays in reviews because
> review is not common place within our community.
> > - Direct commits have to be reverted if there are -1 votes, directly
> impacting the time and effort of the code contributor.
> >
> > It makes a lot of sense to make code commit and review a normal
> process in the CloudStack Community.
> >
> > --Alex
> >
> >
> >
> 
> +1 - And I think that other threads have come to a similar conclusion.
> 
> Does anyone want to volunteer to get that up and running?  We need
> someone to step up and do the work for this to become real.
> 
> -chip

Mime
View raw message