Re: [RFC PATCH] tests/vm: Remove obsolete Fedora VM test

[Alex Bennée]

Philippe Mathieu-Daudé <f4bug@amsat.org> writes:

> On 24/8/22 19:29, Alex Bennée wrote:
>> Thomas Huth <thuth@redhat.com> writes:
>> 
>>> It's still based on Fedora 30 - which is not supported anymore by QEMU
>>> since years. Seems like nobody is using (and refreshing) this, and it's
>>> easier to test this via a container anyway, so let's remove this now.
>>>
>>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>> Queued to testing/next, thanks.
>
> FYI this is the image I'm using to test x86 guest on aarch64/mips64
> hosts, but I can test a BSD-based instead; I suppose Linux as a guest
> is already well tested.

We already have a lot of coverage in avocado for that. If we want to
keep a VM image around though it needs updating to something supported
(either an LTS or more recent Fedora).

>
>>> ---
>>>   tests/vm/Makefile.include |   3 +-
>>>   tests/vm/fedora           | 190 --------------------------------------
>>>   2 files changed, 1 insertion(+), 192 deletions(-)
>>>   delete mode 100755 tests/vm/fedora
>>>
>>> diff --git a/tests/vm/Makefile.include b/tests/vm/Makefile.include
>>> index 8d2a164552..2cc2203d09 100644
>>> --- a/tests/vm/Makefile.include
>>> +++ b/tests/vm/Makefile.include
>>> @@ -15,7 +15,7 @@ endif
>>>     EFI_AARCH64 = $(wildcard
>>> $(BUILD_DIR)/pc-bios/edk2-aarch64-code.fd)
>>>   -X86_IMAGES := freebsd netbsd openbsd fedora haiku.x86_64
>>> +X86_IMAGES := freebsd netbsd openbsd haiku.x86_64
>>>   ifneq ($(GENISOIMAGE),)
>>>   X86_IMAGES += centos
>>>   ifneq ($(EFI_AARCH64),)
>>> @@ -45,7 +45,6 @@ vm-help vm-test:
>>>     @echo "  vm-build-freebsd                - Build QEMU in FreeBSD VM"
>>>     @echo "  vm-build-netbsd                 - Build QEMU in NetBSD VM"
>>>     @echo "  vm-build-openbsd                - Build QEMU in OpenBSD VM"
>>> -   @echo "  vm-build-fedora                 - Build QEMU in Fedora VM"
>>>   ifneq ($(GENISOIMAGE),)
>>>     @echo "  vm-build-centos                 - Build QEMU in CentOS VM, 
>>> with Docker"
>>>   ifneq ($(EFI_AARCH64),)


-- 
Alex Bennée