From: "Daniel P. Berrangé" <berrange@redhat.com>
To: "Alex Bennée" <alex.bennee@linaro.org>
Cc: "Nicholas Piggin" <npiggin@gmail.com>,
qemu-ppc@nongnu.org, "Cleber Rosa" <crosa@redhat.com>,
"Philippe Mathieu-Daudé" <philmd@linaro.org>,
"Wainer dos Santos Moschetta" <wainersm@redhat.com>,
"Beraldo Leal" <bleal@redhat.com>,
qemu-devel@nongnu.org
Subject: Re: [RFC PATCH 07/11] tests/avocado: Add ppc boot tests for non-free AIX images
Date: Tue, 10 Oct 2023 14:01:04 +0100 [thread overview]
Message-ID: <ZSVLEO4hAX3zJRFZ@redhat.com> (raw)
In-Reply-To: <87h6mypryo.fsf@linaro.org>
On Tue, Oct 10, 2023 at 01:43:16PM +0100, Alex Bennée wrote:
>
> Nicholas Piggin <npiggin@gmail.com> writes:
>
> > An AIX image can be provided by setting AIX_IMAGE environment
> > variable when running avocado.
> >
> > It's questionable whether we should carry these in upstream QEMU.
> > It's convenient to see how to run these things, but simple enough
> > to maintain in out of tree branch. I just wanted to see opinions
> > about it.
>
> Yeah there is no point adding a test no one else can run. We already
> have tests that utilise dead URLs that can only run if you happen to
> have the image in the avocado cache which should arguably be removed.
I can understand the appeal of wanting to sanity check QEMU
with esentially arbitrary guest OS, whether modern, or obsolete,
whether OSS or proprietary.
The appeal of getting the test integrated into QEMU is you don't
have to worry about rebasing / merging local git changes forever
more.
I feel like this tells us we should not require users to be writing
new avocado python test code merely to get a boring old guest OS
boot up smoke test integrated into avocado.
I think we ought to have a 'guest_smoke_test.py' avocado test, that
pulls in guest OS scenarios from external YAML/JSON files. eg
$ cat ppc_aix.yaml
image:
url: https:////some/path
checksum: xxxxxx
console:
expect: ...some console message...
vm:
arch: ppc64
machine: pseries
....something something extra cli args something something...
Users could then set
export QEMU_SMOKE_TEST_PATHS=$HOME/my-guestos-library:$HOME/shared-guestos-library
make check-avocado
to load all the guest OS scenarios from these dirs, in addition to
any scenarios that are shipped in qemu.git by default.
With regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
next prev parent reply other threads:[~2023-10-10 13:02 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-10-10 7:52 [RFC PATCH 00/11] ppc: avocado test additions and new defaults Nicholas Piggin
2023-10-10 7:52 ` [RFC PATCH 01/11] tests/avocado: ppc add powernv10 boot_linux_console test Nicholas Piggin
2023-10-10 7:52 ` [RFC PATCH 02/11] tests/avocado: Add ppc pseries and powernv Hash MMU tests Nicholas Piggin
2023-10-16 8:40 ` Cédric Le Goater
2023-10-10 7:52 ` [RFC PATCH 03/11] tests/avocado: Add pseries KVM boot_linux test Nicholas Piggin
2023-10-10 7:52 ` [RFC PATCH 04/11] tests/avocado: ppc add hypervisor tests Nicholas Piggin
2023-10-16 6:39 ` Harsh Prateek Bora
2023-10-10 7:52 ` [RFC PATCH 05/11] testing/avocado: ppc add new BookE boot_linux_console.py tests Nicholas Piggin
2023-10-10 8:02 ` Philippe Mathieu-Daudé
2023-10-10 12:03 ` Joel Stanley
2023-10-10 20:53 ` Nicholas Piggin
2023-10-16 8:07 ` Cédric Le Goater
2023-10-16 8:32 ` Cédric Le Goater
2023-10-10 7:52 ` [RFC PATCH 06/11] tests/avocado: Add FreeBSD distro boot tests for ppc Nicholas Piggin
2023-10-10 21:55 ` Warner Losh
2023-10-11 0:36 ` Nicholas Piggin
2023-10-11 4:24 ` Warner Losh
2023-10-11 2:23 ` Nicholas Piggin
2023-10-11 4:25 ` Warner Losh
2023-10-10 7:52 ` [RFC PATCH 07/11] tests/avocado: Add ppc boot tests for non-free AIX images Nicholas Piggin
2023-10-10 12:43 ` Alex Bennée
2023-10-10 12:49 ` Philippe Mathieu-Daudé
2023-10-10 21:21 ` Nicholas Piggin
2023-10-10 13:01 ` Daniel P. Berrangé [this message]
2023-10-10 21:14 ` Nicholas Piggin
2023-10-10 7:52 ` [RFC PATCH 08/11] tests/avocado: Add ppc MacOS tests Nicholas Piggin
2023-10-10 7:58 ` Philippe Mathieu-Daudé
2023-10-11 0:25 ` Nicholas Piggin
2023-10-10 7:52 ` [RFC PATCH 09/11] tests/avocado: Use default CPU for pseries machine Nicholas Piggin
2023-10-10 7:52 ` [RFC PATCH 10/11] ppc/spapr: change pseries machine default to POWER10 CPU Nicholas Piggin
2023-10-10 12:08 ` Joel Stanley
2023-10-16 8:16 ` Cédric Le Goater
2023-10-10 7:52 ` [RFC PATCH 11/11] ppc/pnv: Change powernv default to powernv10 Nicholas Piggin
2023-10-10 12:05 ` Joel Stanley
2023-10-10 20:57 ` Nicholas Piggin
2023-10-16 1:59 ` Joel Stanley
2023-10-16 8:15 ` Cédric Le Goater
2023-10-10 7:59 ` [RFC PATCH 00/11] ppc: avocado test additions and new defaults Philippe Mathieu-Daudé
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=ZSVLEO4hAX3zJRFZ@redhat.com \
--to=berrange@redhat.com \
--cc=alex.bennee@linaro.org \
--cc=bleal@redhat.com \
--cc=crosa@redhat.com \
--cc=npiggin@gmail.com \
--cc=philmd@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=qemu-ppc@nongnu.org \
--cc=wainersm@redhat.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;
as well as URLs for NNTP newsgroup(s).