[Gluster-Maintainers] IMPORTANT: New jenkins triggering method

Niels de Vos ndevos at redhat.com
Tue Jan 12 14:11:03 UTC 2016


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 patch
> 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 a +1
> 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.

Niels

> 
> 4. Once the regression runs complete, verification verdict is passed onto
> 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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://www.gluster.org/pipermail/maintainers/attachments/20160112/2b1ada99/attachment.sig>


More information about the maintainers mailing list