Re: [igt-dev] [PATCH i-g-t] intel-ci: update blacklists to work with the Xe driver

[Petri Latvala <adrinael@adrinael.net>]

On Thu, Mar 23, 2023 at 03:15:37PM +0100, Mauro Carvalho Chehab wrote:
> On Thu, 23 Mar 2023 15:18:18 +0200
> Petri Latvala <adrinael@adrinael.net> wrote:
> 
> > On Wed, Mar 22, 2023 at 08:29:47AM +0100, Mauro Carvalho Chehab wrote:
> > > From: Mauro Carvalho Chehab <mchehab@kernel.org>
> > > 
> > > It doesn't make sense to run i915 tests with Xe driver and
> > > vice-versa. Update testlists to make it clear.
> > > 
> > > In order to avoid disrupts at exiting intel-ci tests, keep a
> > > symlink:
> > > 	i915-blacklist.txt -> blacklist.txt
> > > 
> > > Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
> > > ---
> > >  tests/intel-ci/blacklist.txt      | 130 +----------------------------
> > >  tests/intel-ci/i915-blacklist.txt | 133 ++++++++++++++++++++++++++++++
> > >  tests/intel-ci/xe-blacklist.txt   |  21 +++++
> > >  3 files changed, 155 insertions(+), 129 deletions(-)
> > >  mode change 100644 => 120000 tests/intel-ci/blacklist.txt
> > >  create mode 100644 tests/intel-ci/i915-blacklist.txt
> > >  create mode 100644 tests/intel-ci/xe-blacklist.txt  
> > 
> > Since a rename is happening anyway, let's rename to inclusive language
> > at the same time? i915-blocklist.txt + xe-blocklist.txt
> 
> Currently, we have 2 such files:
> 
> 	tests/intel-ci/blacklist-pre-merge.txt
> 	tests/intel-ci/blacklist.txt
> 
> IMO, a rename from <blacklist> to whatever now-preferred namespace
> should touch both.
> 
> Yet, renames like that will break CI, so symlinks to the old names
> are required.
> 
> If you're ok with that, I can prepare an upcoming series with
> the renames, including renaming from "*blacklist*" to
> "*blocklist*" namespace and adding the needed symlinks to preserve
> CI from regressions.

Yeah, that sounds good. Get an ack from someone in the CI team as well
for it, just so they know in case something really weird breaks deeper
in the CI machinery.


-- 
Petri Latvala