Re: Improve default machine type selection

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 4/14/23 09:24, Andrea Bolognani wrote:
On Fri, Apr 14, 2023 at 03:33:21PM +0200, Peter Krempa wrote:
On Fri, Apr 14, 2023 at 06:26:02 -0700, Andrea Bolognani wrote:
I don't think we rely on default machine types anywhere in the test
suite, and even if we were only minor touch-ups would probably be
necessary.

The test suite does use/validate/assign machine types, but I've already
removed the all of the fake-machine type infrastructure for RISC-V so
you only get to use the qemuCapabilities object populated from what we
have in tests/qemucapabilitiesdata/.

There doesn't seem to be any qemuxml2argv test case referencing 'spike'
so obviously the default machine is not covered by tests. That also
means that the test suite will not break :)

Yeah, exactly: for all existing test cases we pick the machine type
explicitly, so changing the default shouldn't affect them at all.

And the fact that we don't have test cases covering integratorcp or
spike is not unexpected, since for both Arm and RISC-V we've only
ever really considered the virt machine type.


I needed to adjust several of the tests as well. Or something else needs tweaked that I'm missing :-). V1 is available for your viewing pleasure

https://listman.redhat.com/archives/libvir-list/2023-April/239392.html

Regards,
Jim




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux