[Engine-devel] Development process wiki page

Mike Kolesnik mkolesni at redhat.com
Tue Jan 31 10:04:29 UTC 2012


----- Original Message -----
> On 01/31/2012 03:03 AM, Eli Mesika wrote:
> >
> >
> > ----- Original Message -----
> >> From: "Oved Ourfalli"<ovedo at redhat.com>
> >> To: engine-devel at ovirt.org
> >> Sent: Monday, January 30, 2012 9:45:47 AM
> >> Subject: [Engine-devel] Development process wiki page
> >>
> >> Hey all,
> >>
> >> I've wrote a wiki page on the oVirt development process.
> >> It contains mostly information on the patch review process, patch
> >> submission and some git guidelines.
> >>
> >> http://www.ovirt.org/wiki/DevProcess
> >>
> >> I added a link to it from the main wiki page.
> >>
> >> Your comments are welcome.
> >
> > my 2 cents : (obvious , but should be in the wiki)
> >
> > 1) Regarding patch separation, still each commit should not break
> > the build
> > 2) When having more than one patch set due to comments and applied
> > fixes add to the commit message V[n] :<changes description>  where
> > [n] is the patch set number
> 
> I agree we need this, but the problem is we are lacking the [0/nnn]
> cover letter email you get for such commentary when using gerrit.
> and there is a concept of the commit itself should not include the
> history of the patch being reviewed.
> so we need to adapt this to gerrit.
> maybe a convention that when uploading a new patch set, contributor
> also
> adds a review comment of the changes from previous version (similar
> to
> the cover letter).

I think if the commit history already exists in gerrit, and the commit itself is pushed through gerrit, then this will be redundant.

> I wonder if the git/gerrit utility some adopted from openstack can
> help
> with such a step to allow adding a comment when pushing the patch for
> review.
> _______________________________________________
> Engine-devel mailing list
> Engine-devel at ovirt.org
> http://lists.ovirt.org/mailman/listinfo/engine-devel
> 



More information about the Devel mailing list