[Gluster-Maintainers] [Gluster-devel] IMPORTANT: New jenkins triggering method
atin.mukherjee83 at gmail.com
Tue Jan 12 14:29:39 UTC 2016
Sent from one plus one
On Jan 12, 2016 7:41 PM, "Niels de Vos" <ndevos at redhat.com> wrote:
> On Tue, Jan 12, 2016 at 07:21:37PM +0530, Raghavendra Talur wrote:
> > We have now changed the gerrit-jenkins workflow as follows:
> > 1. Developer works on a new feature/bug fix and tests it locally(run
> > run-tests.sh completely).
> > 2. Developer sends the patch to gerrit using rfc.sh.
> > +++Note that no regression runs have started automatically for this
> > at this point.+++
> > 3. Developer marks the patch as +1 verified on gerrit as a promise of
> > having tested the patch completely. For cases where patches don't have
> > verified from the developer, maintainer has the following options
> > a. just do the code-review and award a +2 code review.
> > b. pull the patch locally and test completely and award a +1 verified.
> > Both the above actions would result in triggering of regression runs for
> > the patch.
> Would it not help if anyone giving +1 code-review starts the regression
> tests too? When developers ask me to review, I prefer to see reviews
> done by others first, and any regression failures should have been fixed
> by the time I look at the change.
When this idea was originated (long back) I was in favour of having
regression triggered on a +1, however verified flag set by the developer
would still trigger the regression. Being a maintainer I would always
prefer to look at a patch when its verified flag is +1 which means the
regression result would also be available.
> > 4. Once the regression runs complete, verification verdict is passed
> > gerrit by jenkins
> > and maintainer can proceed with the merge using submit button.
> > Thanks,
> > Raghavendra Talur
> > _______________________________________________
> > maintainers mailing list
> > maintainers at gluster.org
> > http://www.gluster.org/mailman/listinfo/maintainers
> Gluster-devel mailing list
> Gluster-devel at gluster.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the maintainers