Re: [Gluster-Maintainers] Update: Gerrit review system has one more command now

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



This will be very useful. Thank you.

On Mon, May 21, 2018 at 11:45 PM, Vijay Bellur <vbellur@xxxxxxxxxx> wrote:


On Mon, May 21, 2018 at 2:29 AM, Amar Tumballi <atumball@xxxxxxxxxx> wrote:
Hi all,

As a push towards more flexibility to our developers, and options to run more tests without too much effort, we are moving towards more and more options to trigger tests from Gerrit during reviews.

One such example was 'regression-on-demand-multiplex' tests, where any one can ask for a brick-mux regression for a particular patch.

In the same way, in certain cases where developers are making changes, and more than 1 tests would be impacted, there was no easy way to run all the regression, other than sending one patchset with changes to 'run-tests.sh' to not fail on failures. This was tedious, and also is not known to many new developers. Hence a new command is added to gerrit, where one can trigger all the runs (if something is failing), by entering 'run full regression' in a single line at the top of your review comments.

With this, a separate job will be triggered which will run the full regression suite with the patch. So, no more requirement to make 'run-tests.sh' changes.





Thank you, Amar! I think it will be quite useful for us.

I am not sure if there's a document that details all possible options & tricks with gerrit. If there's none, we could add one to our repository/developer-guide so that new developers find it easy to use these options.

Regards,
Vijay

_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
http://lists.gluster.org/mailman/listinfo/gluster-devel



--
Thanks and Regards,
Kotresh H R
_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
http://lists.gluster.org/mailman/listinfo/gluster-devel

[Index of Archives]     [Gluster Users]     [Ceph Users]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux