Improve default machine type selection

Jim Fehlig jfehlig at suse.com
Fri Apr 14 21:19:08 UTC 2023


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



More information about the libvir-list mailing list