From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga06.intel.com (mga06b.intel.com [134.134.136.31]) by gabe.freedesktop.org (Postfix) with ESMTPS id 0108D10EAB3 for ; Thu, 23 Mar 2023 14:15:49 +0000 (UTC) Date: Thu, 23 Mar 2023 15:15:37 +0100 From: Mauro Carvalho Chehab To: Petri Latvala Message-ID: <20230323151537.75d5efe3@maurocar-mobl2> In-Reply-To: References: <20230322072947.1760988-1-mauro.chehab@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [igt-dev] [PATCH i-g-t] intel-ci: update blacklists to work with the Xe driver List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: igt-dev@lists.freedesktop.org Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: On Thu, 23 Mar 2023 15:18:18 +0200 Petri Latvala wrote: > On Wed, Mar 22, 2023 at 08:29:47AM +0100, Mauro Carvalho Chehab wrote: > > From: Mauro Carvalho Chehab > > > > 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 > > --- > > 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 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. Regards, Mauro