oVirt site

Carl Trieloff cctrieloff at redhat.com
Thu Sep 15 16:40:48 UTC 2011


On 09/15/2011 12:17 PM, Itamar Heim wrote:
> - need to decide if patch must be verified before submitting, or
> verification could be done as part of patch review (for example, gerrit
> supports verification as part of flow. Some projects may prefer this
> approach).
> - (using gerrit terminology) anyone can +1 a patch. only maintainer can +2
> the patch. a +1 would help a maintainer do the +2.
> - going back to my earlier point on committer != maintainer: in a large
> project, we'd still like committer to have right to commit to entire git
> to make things simple, but may have maintainers (or call them sub
> maintainers) per module in the project. A committer would expect (trust
> would suffice I hope) the relevant maintainer[1] to give the +2 before
> committing the patch[2]

Itamar,

Do you want to explicitly add the +2 for projects that use gerrit?  I
can do that.

Carl.



More information about the Project-planning mailing list