Re: [PATCH i-g-t] tests/intel-ci/xe.blocklist: Add SR-IOV tests

[Michal Wajdeczko <michal.wajdeczko@intel.com>]

On 09.07.2024 13:27, Jakub Kolakowski wrote:
> Add SR-IOV tests to xe blocklist so that only desired tests will be run
> in CI.
> SR-IOV tests that should be running are as follows:
> - sriov_basic@enable-vfs-autoprobe-off (all dynamic subtests)
> - sriov_basic@enable-vfs-autoprobe-on@numvfs-all

I'm not sure that we want to keep this test above in CI as it is almost
like a stress-test, and test below will provide the same functional
coverage, without being a stress

> - sriov_basic@enable-vfs-bind-unbind-each@numvfs-all
> - sriov_basic@bind-unbind-vf@vf-1
> - sriov_basic@bind-unbind-vf@vf-last

IMO those two do not bring any additional coverage compared to above:

 sriov_basic@enable-vfs-bind-unbind-each@numvfs-all

so lets drop them from the CI (and maybe consider killing them for good)

> 
> Signed-off-by: Jakub Kolakowski <jakub1.kolakowski@intel.com>
> Cc: Adam Miszczak <adam.miszczak@intel.com>
> Cc: Marcin Bernatowicz <marcin.bernatowicz@intel.com>
> Cc: Lukasz Laguna <lukasz.laguna@intel.com>
> Cc: Michal Wajdeczko <Michal.Wajdeczko@intel.com>
> ---
>  tests/intel-ci/xe.blocklist.txt | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/tests/intel-ci/xe.blocklist.txt b/tests/intel-ci/xe.blocklist.txt
> index 828baf324..57e550885 100644
> --- a/tests/intel-ci/xe.blocklist.txt
> +++ b/tests/intel-ci/xe.blocklist.txt
> @@ -83,3 +83,9 @@ igt@kms_prime@.*
>  ##################################################################
>  igt@kms_cursor_legacy@modeset-atomic-cursor-hotspot
>  ##################################################################
> +# SR-IOV tests
> +##################################################################
> +igt@sriov_basic@enable-vfs-autoprobe-on@numvfs-(?!all).*
> +igt@sriov_basic@enable-vfs-bind-unbind-each@numvfs-(?!all).*
> +igt@sriov_basic@bind-unbind-vf@vf-(?!(1$|last)).*
> +##################################################################