workflows.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Bird, Tim" <Tim.Bird@sony.com>
To: Don Zickus <dzickus@redhat.com>,
	"workflows@vger.kernel.org" <workflows@vger.kernel.org>,
	"automated-testing@lists.yoctoproject.org"
	<automated-testing@lists.yoctoproject.org>,
	"linux-kselftest@vger.kernel.org"
	<linux-kselftest@vger.kernel.org>,
	kernelci <kernelci@lists.linux.dev>
Cc: Nikolai Kondrashov <nkondras@redhat.com>,
	Gustavo Padovan <gustavo.padovan@collabora.com>,
	kernelci-members <kernelci-members@groups.io>,
	"laura.nao@collabora.com" <laura.nao@collabora.com>
Subject: RE: [Automated-testing] [RFC] Test catalog template
Date: Tue, 15 Oct 2024 16:01:26 +0000	[thread overview]
Message-ID: <MW5PR13MB5632C443F9D7E658168BC77DFD452@MW5PR13MB5632.namprd13.prod.outlook.com> (raw)
In-Reply-To: <CAK18DXYitS7hL1mA3QsPLmW9-R0q6Kin0C5Uv9fj=uS90WSnxA@mail.gmail.com>

> -----Original Message-----
> From: automated-testing@lists.yoctoproject.org <automated-testing@lists.yoctoproject.org> On Behalf Of Don Zickus
> Hi,
> 
> At Linux Plumbers, a few dozen of us gathered together to discuss how
> to expose what tests subsystem maintainers would like to run for every
> patch submitted or when CI runs tests.  We agreed on a mock up of a
> yaml template to start gathering info.  The yaml file could be
> temporarily stored on kernelci.org until a more permanent home could
> be found.  Attached is a template to start the conversation.
>
Don,

I'm interested in this initiative.  Is discussion going to be on a kernel mailing
list, or on this e-mail, or somewhere else?

See a few comments below.
 
> Longer story.
> 
> The current problem is CI systems are not unanimous about what tests
> they run on submitted patches or git branches.  This makes it
> difficult to figure out why a test failed or how to reproduce.
> Further, it isn't always clear what tests a normal contributor should
> run before posting patches.
> 
> It has been long communicated that the tests LTP, xfstest and/or
> kselftests should be the tests  to run. 
Just saying "LTP" is not granular enough.  LTP has hundreds of individual
test programs, and it would be useful to specify the individual tests
from LTP that should be run per sub-system.

I was particularly intrigued by the presentation at Plumbers about
test coverage.  It would be nice to have data (or easily replicable
methods) for determining the code coverage of a test or set of
tests, to indicate what parts of the kernel are being missed
and help drive new test development.

> However, not all maintainers
> use those tests for their subsystems.  I am hoping to either capture
> those tests or find ways to convince them to add their tests to the
> preferred locations.
> 
> The goal is for a given subsystem (defined in MAINTAINERS), define a
> set of tests that should be run for any contributions to that
> subsystem.  The hope is the collective CI results can be triaged
> collectively (because they are related) and even have the numerous
> flakes waived collectively  (same reason) improving the ability to
> find and debug new test failures.  Because the tests and process are
> known, having a human help debug any failures becomes easier.
> 
> The plan is to put together a minimal yaml template that gets us going
> (even if it is not optimized yet) and aim for about a dozen or so
> subsystems.  At that point we should have enough feedback to promote
> this more seriously and talk optimizations.

Sounds like a good place to start.  Do we have some candidate sub-systems
in mind?  Has anyone volunteered to lead the way?

> 
> Feedback encouraged.
> 
> Cheers,
> Don
> 
> ---
> # List of tests by subsystem
> #
> # Tests should adhere to KTAP definitions for results
> #
> # Description of section entries
> #
> #  maintainer:    test maintainer - name <email>
> #  list:                mailing list for discussion
> #  version:         stable version of the test
> #  dependency: necessary distro package for testing
> #  test:
> #    path:            internal git path or url to fetch from
> #    cmd:            command to run; ability to run locally
> #    param:         additional param necessary to run test
> #  hardware:      hardware necessary for validation

Is this something new in MAINTAINERS, or is it a separate file?

> #
> # Subsystems (alphabetical)
> 
> KUNIT TEST:
>   maintainer:
>     - name: name1
>       email: email1
>     - name: name2
>       email: email2
>   list:
>   version:
>   dependency:
>     - dep1
>     - dep2
>   test:
>     - path: tools/testing/kunit
>       cmd:
>       param:
>     - path:
>       cmd:
>       param:
>   hardware: none

Looks OK so far - it'd be nice to have a few concrete examples.
 -- Tim


  reply	other threads:[~2024-10-15 16:01 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-10-14 20:32 Donald Zickus
2024-10-15 16:01 ` Bird, Tim [this message]
2024-10-16 13:10   ` [Automated-testing] " Cyril Hrubis
2024-10-16 18:02     ` Donald Zickus
2024-10-17 11:01       ` Cyril Hrubis
2024-10-16 18:00   ` Donald Zickus
2024-10-17 12:31 ` Minas Hambardzumyan
2024-10-18 19:44   ` Donald Zickus
2024-10-18  7:21 ` David Gow
2024-10-18 14:23   ` Gustavo Padovan
2024-10-18 14:35     ` [Automated-testing] " Cyril Hrubis
2024-10-18 19:17   ` Mark Brown
2024-10-18 20:17   ` Donald Zickus
2024-10-19  6:36     ` David Gow
2024-11-06 17:01       ` Donald Zickus
2024-11-20  8:16         ` David Gow
2024-11-21 15:28           ` Donald Zickus

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=MW5PR13MB5632C443F9D7E658168BC77DFD452@MW5PR13MB5632.namprd13.prod.outlook.com \
    --to=tim.bird@sony.com \
    --cc=automated-testing@lists.yoctoproject.org \
    --cc=dzickus@redhat.com \
    --cc=gustavo.padovan@collabora.com \
    --cc=kernelci-members@groups.io \
    --cc=kernelci@lists.linux.dev \
    --cc=laura.nao@collabora.com \
    --cc=linux-kselftest@vger.kernel.org \
    --cc=nkondras@redhat.com \
    --cc=workflows@vger.kernel.org \
    /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