Code review required for commits - formal Team vote.
Karolin Seeger
kseeger at samba.org
Mon Oct 15 04:11:32 MDT 2012
On Fri, Oct 12, 2012 at 10:48:32AM -0700, Jeremy Allison wrote:
> Ok, I'd like to bring this to a conclusion before I have to go
> on my European trip next week :-).
>
> Sorry for asking for another vote after people have already
> +1 and -1'ed, but there have been so many discussions about
> what exactly the details of the policy should be that I think
> it might be useful to re-iterate exactly what we're voting on.
>
> Just to be clear - the formal proposal, originally from
> Ira (but slightly tweaked by me after the discussions)
> is here:
>
> --------------------------------------------------------
> No team member commits their own code. All code will be "signed off"
> by two team members, as a team member you may sign off your own code.
> The "non-author" team member will be responsible for pushing the code.
> If there are two they can agree among themselves.
>
> Tools for code review to be decided amongst reviewers, but any
> patch posted to samba-technical is an explicit request for review.
>
> Release branches work as they do today, though the + should be treated
> as a sign-off in bugzilla. All release branch bug commits must contain
> their bug number in them, so we can track back what happened.
> --------------------------------------------------------
>
> Given this policy (no author push, no free pass) as an initial
> start I think we can move to a vote.
>
> Yes, there will be things to sort out and discuss as we go
> along, but we're all reasonable (-ish :-) people and can work
> out the minor details to make it work as we go along.
>
> So I know many people have voted back and forth on various
> versions, but I'd like to have a formal Team vote (anyone
> on the Team with a samba.org address) to make a decision on
> this. No reply means abstention.
>
> Let's wait until the end of next week to allow everyone
> who might be travelling or not reading email to respond
> and think carefully about it, and tally the votes on
> Friday 19th Oct (I'll be travelling on that date so
> someone else will have probably to do the count, volunteers
> welcome :-). If it passes let's target implementation on
> Monday 22nd Oct.
>
> I'm voting +1 (in case you wondered).
+1
I do really like the reviews. The procedure works very well in the release
branches (from my point of view). Sometimes, even reviewed patches do not
compile at all, so the non-author push sounds like a very good idea to me.
What I would not like is sending each patch to samba-technical. I would
prefer private git repos, mails, whatever the developers like.
Alternatively, we might create a new mailing list.
So I would like to give it a try.
Cheers,
Karolin
--
Samba http://www.samba.org
SerNet http://www.sernet.de
sambaXP http://www.sambaxp.org
More information about the samba-technical
mailing list