Re: [Gluster-devel] Reducing the time to test patches which doesn't modify code

2017-06-22 Thread Nigel Babu
On Wed, Jun 21, 2017 at 10:35:05AM +0530, Amar Tumballi wrote:
> Today, any changes to glusterfs code base (other than 'doc/') triggers the
> regression runs when +1 Verified is voted. But we noticed that patches
> which are changes in 'extras/' or just updating README file, need not run
> regressions.
>
> So, Nigel proposed the idea of a .testignore file (like .gitignore)[1].
> Content in which are paths for files to ignore from testing. If a patch has
> all the files belonging to this file, then the tests wouldn't be triggered.
>
> Anyone sending patches in future, and the patch needs to add a new file,
> see if you need to update .testignore file too.
>
> [1] - https://review.gluster.org/17522

This feature is now in production on the regression machines. If you're only
touching files in .testignore, the centos-regression run will only take
1 minute. When you're adding a new script to the codebase that does not affect
gluster itself, please consider adding those files to .testignore

--
nigelb
___
Gluster-devel mailing list
Gluster-devel@gluster.org
http://lists.gluster.org/mailman/listinfo/gluster-devel


[Gluster-devel] Reducing the time to test patches which doesn't modify code

2017-06-20 Thread Amar Tumballi
Today, any changes to glusterfs code base (other than 'doc/') triggers the
regression runs when +1 Verified is voted. But we noticed that patches
which are changes in 'extras/' or just updating README file, need not run
regressions.

So, Nigel proposed the idea of a .testignore file (like .gitignore)[1].
Content in which are paths for files to ignore from testing. If a patch has
all the files belonging to this file, then the tests wouldn't be triggered.

Anyone sending patches in future, and the patch needs to add a new file,
see if you need to update .testignore file too.

[1] - https://review.gluster.org/17522

Regards,
Amar

-- 
Amar Tumballi (amarts)
___
Gluster-devel mailing list
Gluster-devel@gluster.org
http://lists.gluster.org/mailman/listinfo/gluster-devel