Re: [LTP] [PATCH v1] Replace runltp-ng with kirk framework

[Petr Vorel <pvorel@suse.cz>]

> Hi,

> On 9/15/23 13:19, Petr Vorel wrote:
> > Hi Cyril, Andrea,

> > > Hi!
> > > > Kirk application is a fork of runltp-ng and it aims to merge multiple
> > > > Linux testing frameworks in one tool, providing support for remote
> > > > testing via Qemu, SSH, LTX, parallel execution and much more.
> > > > Signed-off-by: Andrea Cervesato <andrea.cervesato@suse.com>
> > > > ---
> > > >   .gitmodules     | 6 +++---
> > > >   tools/kirk      | 1 +
> > > >   tools/runltp-ng | 1 -
> > > Do we also want symlink from runltp-ng to kirk?
> > +1

> > BTW I suggested Andrea in the beginning of kirk to have runltp-ng symlink,
> > which would behave like 'kirk --framework ltp'. I.e. this:

> > ./kirk --framework ltp --run-suite syscalls

> > would be equivalent of

> > ./runltp-ng --run-suite syscalls

> > Instead of having to run:
> > ./runltp-ng --framework ltp --run-suite syscalls

> > In that case kirk would be a drop in replacement.
> > Andrea, would it be possible to implement this functionality?
> > (the default framework could be detected by symlink name, the same approach
> > busybox uses, more symlinks could be added).

> > Kind regards,
> > Petr

> The default kirk behavior is to use '--framework ltp' by default, so there's
> no difference between

> ./kirk --framework ltp --run-suite syscalls

> and

> ./kirk --run-suite syscalls

Great, then OK from my side. With adding the symlink:
Reviewed-by: Petr Vorel <pvorel@suse.cz>
(please send v2).

I'm for replacing runltp-ng with kirk before the release.
I guess Cyril will highlight it in the changelog and then we should replace all
files in the runltp-ng repo with simple README.md, which points to kirk git
repo.

I'd still recommend to implement the default framework functionality (kselftest
and liburing would benefit from it), but that can be added later.

Kind regards,
Petr

> Regards,
> Andrea


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