From: Kamil Konieczny <kamil.konieczny@linux.intel.com>
To: igt-dev@lists.freedesktop.org
Cc: "Kamil Konieczny" <kamil.konieczny@linux.intel.com>,
"Christoph Manszewski" <christoph.manszewski@intel.com>,
"Zbigniew Kempczyński" <zbigniew.kempczynski@intel.com>,
"Jari Tahvanainen" <jari.tahvanainen@intel.com>,
"Katarzyna Piecielska" <katarzyna.piecielska@intel.com>,
"Mauro Carvalho Chehab" <mchehab@kernel.org>
Subject: [PATCH i-g-t 0/1] relax doc generation for non-compiled tests
Date: Thu, 12 Sep 2024 16:04:00 +0200 [thread overview]
Message-ID: <20240912140401.137060-1-kamil.konieczny@linux.intel.com> (raw)
New tests added with conditional compilation result in error reported
when generating testlist. For example, with a recent series with new
eudebug igt tests from here:
https://patchwork.freedesktop.org/series/136623/
("Test coverage for GPU debug support")
when --exclude-files option was removed from version 6:
https://patchwork.freedesktop.org/series/136623/#rev6
then compiling with testplan report warnings:
xe_eudebug.testlist: Testlist file not found.
xe_eudebug_online.testlist: Testlist file not found.
xe_exec_sip_eudebug.testlist: Testlist file not found.
Warning: Documented igt@xe_eudebug@attach-debug-metadata doesn't exist on source files
Warning: Documented igt@xe_eudebug@basic-client doesn't exist on source files
Warning: Documented igt@xe_eudebug@basic-client-th doesn't exist on source files
[...rest omitted...]
and exits with an error.
Relax that check to only report error on missing documentation, and treat
new added tests without a compiled binary as a new tests under development
or as tests outside a compilation scope (such as chamelium). This should
allow to have new tests in documentation and track them.
With dropped --exclude-files patch and dropped changes in docs/testplan/meson.build
generation of testplan succeeds and in report it can be find:
[1897/1905] Generating docs/testplan/xe_tests.rst with a custom command
xe_eudebug.testlist: Testlist file not found.
xe_eudebug_online.testlist: Testlist file not found.
xe_exec_sip_eudebug.testlist: Testlist file not found.
xe_eudebug: No binary found, test was not compiled.
xe_eudebug_online: No binary found, test was not compiled.
xe_exec_sip_eudebug: No binary found, test was not compiled.
Info: Documented igt@xe_eudebug@attach-debug-metadata don't have a binary xe_eudebug
Info: Documented igt@xe_eudebug_online@basic-breakpoint don't have a binary xe_eudebug_online
Info: Documented igt@xe_exec_sip_eudebug@breakpoint-waitsip don't have a binary xe_exec_sip_eudebug
[1898/1905] Generating docs/testplan/i915_tests.rst with a custom command
[1899/1905] Generating docs/testplan/kms_tests.rst with a custom command
[1900/1905] Generating docs/testplan/intel-ci-tests with a custom command
[1901/1905] Generating docs/testplan/xe_tests.html with a custom command
[1902/1905] Generating docs/testplan/i915_tests.html with a custom command
[1903/1905] Compiling C object runner/runner_test.p/runner_tests.c.o
[1904/1905] Generating docs/testplan/kms_tests.html with a custom command
[1905/1905] Linking target runner/runner_test
Cc: Christoph Manszewski <christoph.manszewski@intel.com>
Cc: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
Cc: Jari Tahvanainen <jari.tahvanainen@intel.com>
Cc: Katarzyna Piecielska <katarzyna.piecielska@intel.com>
Cc: Mauro Carvalho Chehab <mchehab@kernel.org>
Kamil Konieczny (1):
scripts/test_list: relax treatment of non-compiled tests
scripts/test_list.py | 31 +++++++++++++++++++++++++++++--
1 file changed, 29 insertions(+), 2 deletions(-)
--
2.43.0
next reply other threads:[~2024-09-12 14:04 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-09-12 14:04 Kamil Konieczny [this message]
2024-09-12 14:04 ` [PATCH i-g-t 1/1] scripts/test_list: relax treatment of non-compiled tests Kamil Konieczny
2024-09-13 11:03 ` Manszewski, Christoph
2024-09-12 16:47 ` ✓ Fi.CI.BAT: success for relax doc generation for " Patchwork
2024-09-12 17:06 ` ✓ CI.xeBAT: " Patchwork
2024-09-13 5:11 ` ✗ CI.xeFULL: failure " Patchwork
2024-09-13 6:02 ` ✗ Fi.CI.IGT: " Patchwork
2024-09-13 10:58 ` [PATCH i-g-t 0/1] " Manszewski, Christoph
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240912140401.137060-1-kamil.konieczny@linux.intel.com \
--to=kamil.konieczny@linux.intel.com \
--cc=christoph.manszewski@intel.com \
--cc=igt-dev@lists.freedesktop.org \
--cc=jari.tahvanainen@intel.com \
--cc=katarzyna.piecielska@intel.com \
--cc=mchehab@kernel.org \
--cc=zbigniew.kempczynski@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox