[igt-dev] [PATCH i-g-t 1/2] intel-ci: Tweak blacklist for very long running stability tests

[Petri Latvala]

On Mon, Mar 16, 2020 at 10:54:26AM +0000, Chris Wilson wrote:
> To exclude yynamic tests just use  their group name?

Yes, the igt_subtest_with_dynamic("somename") macro creates a subtest
entry point just like igt_subtest, for the purposes of testlists and
blacklists.


> 
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> Cc: Petri Latvala <petri.latvala at intel.com>
> ---
>  tests/intel-ci/blacklist.txt | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/tests/intel-ci/blacklist.txt b/tests/intel-ci/blacklist.txt
> index 948b47569..184c23c37 100644
> --- a/tests/intel-ci/blacklist.txt
> +++ b/tests/intel-ci/blacklist.txt
> @@ -60,9 +60,9 @@ igt at gem_sync@(?!.*basic).*
>  igt at gem_tiled_swapping@(?!non-threaded).*
>  igt at gem_userptr_blits@(major|minor|forked|mlocked|swapping).*
>  igt at gem_wait@.*hang.*
> -igt at sysfs_heartbeat_timeout@long.*
> -igt at sysfs_preemption_timeout@off.*
> -igt at sysfs_timeslice_duration@off.*
> +igt at sysfs_heartbeat_timeout@long
> +igt at sysfs_preemption_timeout@off
> +igt at sysfs_timeslice_duration@off

Please fix the test names along with this change. I spent some minutes
trying to figure out what changes, before I realized
sysfs_heartbeat_timeout doesn't exist. It's
sysfs_heartbeat_interval. sysfs_preemption_timeout is
sysfs_preempt_timeout.

Ways to doublecheck:

igt_runner -L -t igt at sysfs_heartbeat_timeout@long build/tests

igt_runner -L -b tests/intel-ci/blacklist.txt build/tests

https://patchwork.freedesktop.org/series/74263/


Acked-by: Petri Latvala <petri.latvala at intel.com>