All of lore.kernel.org
 help / color / mirror / Atom feed
From: Cyril Hrubis <chrubis@suse.cz>
To: Andrea Cervesato <andrea.cervesato@suse.com>
Cc: Linux Test Project <ltp@lists.linux.it>
Subject: Re: [LTP] [kirk] Remove support for external testing suite
Date: Mon, 31 Mar 2025 18:17:24 +0200	[thread overview]
Message-ID: <Z-rAFF9yX3de8CZw@yuki.lan> (raw)
In-Reply-To: <f1833ac3-5c28-4a19-945c-2112dbd898eb@suse.com>

Hi!
> As you may know, Kirk supports the execution of certain kselftests and 
> liburing testing suites. It was initially created as an experiment.
> 
> We have made attempts to implement these features, but it seems we have 
> not succeeded. They appear to be largely unused by most users and are 
> creating a bottleneck for new Kirk implementations, such as the 
> automatic installation of LTP inside the SUT. This feature would be 
> particularly beneficial for QEMU vanilla images. We had it in the past 
> already and probably it's time to get this feature back. And we should 
> probably implement it for other testing suites due to the current API: 
> something that we really don't want to do (see eBPF kselftests compile 
> madness).
> 
> Another concern is the maintainability of supporting for additional 
> testing suites. While they are interesting, the implementation is 
> somewhat flawed. Each testing suite has its own 
> input/output/configuration/compilation methods, leading to confusion and 
> over-engineered solutions. While we understand that no solution is 
> perfect, it seems we have reached a point where Kirk = LTP. Therefore, 
> we should reconsider maintaining features that require updates due to 
> changes in external testing suites.
> 
> For these reasons, I propose that we remove support for the additional 
> testing suites and focus exclusively on LTP for Kirk maintenance.
> 
> What do you think?

Sounds reasonable to me. There is quite a bit of work to be done in
order to implement all the advanced features in kirk and it does not
help if we get distracted by side quests.

-- 
Cyril Hrubis
chrubis@suse.cz

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

  reply	other threads:[~2025-03-31 16:17 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-03-31  9:52 [LTP] [kirk] Remove support for external testing suite Andrea Cervesato via ltp
2025-03-31 16:17 ` Cyril Hrubis [this message]
2025-04-01  8:52   ` Andrea Cervesato via ltp

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=Z-rAFF9yX3de8CZw@yuki.lan \
    --to=chrubis@suse.cz \
    --cc=andrea.cervesato@suse.com \
    --cc=ltp@lists.linux.it \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.