On Wed, Aug 27, 2025 at 14:41:38 -0500, Praveen K Paladugu wrote:
> 
> 
> On 8/26/2025 11:14 AM, Peter Krempa wrote:
> > On Tue, Aug 26, 2025 at 09:33:59 -0500, Praveen K Paladugu wrote:
> > > 
> > > 
> > > On 8/25/2025 3:22 PM, Peter Krempa wrote:
> > > > On Mon, Aug 25, 2025 at 10:57:52 -0500, Praveen K Paladugu wrote:
> > > > > Introduce mshv variant in capabilities tests.
> > > > > 
> > > > > Signed-off-by: Praveen K Paladugu <pra...@linux.microsoft.com>
> > > > > ---
> > > > >    .../caps_11.3.0_x86_64+mshv.replies           | 46912 
> > > > > ++++++++++++++++
> > > > >    .../caps_11.3.0_x86_64+mshv.xml               |  4996 ++
> > > > >    .../qemucaps2xmloutdata/caps.x86_64+mshv.xml  |    29 +
> > > > >    3 files changed, 51937 insertions(+)
> > > > >    create mode 100644 
> > > > > tests/qemucapabilitiesdata/caps_11.3.0_x86_64+mshv.replies
> > > > >    create mode 100644 
> > > > > tests/qemucapabilitiesdata/caps_11.3.0_x86_64+mshv.xml
> > > > >    create mode 100644 tests/qemucaps2xmloutdata/caps.x86_64+mshv.xml
> > > > 
> > > > Few issues:
> > > > 
> > > >    1) the test-suite fails after this patch:
> > > > 
> > > >    $ VIR_TEST_REGENERATE_OUTPUT=1 ./tests/domaincapstest
> > > > TEST: domaincapstest
> > > >         
> > > > ............................../home/pipo/libvirt/tests/domaincapsdata/qemu_11.3.0.x86_64+mshv.xml:
> > > >  failed to open: No such file or directory
> > > > !/home/pipo/libvirt/tests/domaincapsdata/qemu_11.3.0-q35.x86_64+mshv.xml:
> > > >  failed to open: No such file or directory
> > > > !/home/pipo/libvirt/tests/domaincapsdata/qemu_11.3.0-tcg.x86_64+mshv.xml:
> > > >  failed to open: No such file or directory
> > > > !....... 40
> > > >         ..................................       74  FAIL
> > > > 3 tests failed. Run them using:
> > > > VIR_TEST_DEBUG=1 VIR_TEST_RANGE=31-33 ./tests/domaincapstest
> > > > 
> > > > You seem to have forgotten to commit the two output files.
> > > > 
> > > > 
> > > I didn't notice this failure among the other failures I am seeing with 
> > > this
> > > test locally. I will investigate this further.
> > 
> > Can you elaborate what other failures you are seeing?
> > 
> 
> I should have clarified this. The test failures I am noticing were not
> introduced by my code changes. I see these failures even while using latest
> upstream code (commit: 5b69c37265e4ec876d7a46bd865427388bc37da2):

This logic is flawed. If the tests don't pass before your changes ...

> 
> Summary of Failures:
> 
>  73/300 libvirt:bin / domaincapstest       FAIL            1.23s   exit
> status 1

... how can you tell that your changes didn't break these?

> 123/300 libvirt:bin / qemufirmwaretest      FAIL            0.04s   exit
> status 1
> 131/300 libvirt:bin / qemuvhostusertest      FAIL            0.03s   exit
> status 1
> 182/300 libvirt:bin / qemuxmlconftest      FAIL            7.45s   exit
> status 1
> 
> 
> These could be related to missing components or incorrect configuration
> on my development host (Fedora 42). I didn't investigate these failures,

A sizable part of libvirt developers do use Fedora, so the tests do pass
almost all the time.

> yet.

So, and how do you configure your build?


> 
> domaincapstest:
> 
>  VIR_TEST_DEBUG=1 
> VIR_TEST_RANGE=2,4-7,9-10,12,14,16,19-21,23-24,26-27,29-31,34,36-37,39-40,42-43,47,49,51,53-54,56-57,59-60,62-63,65,67-68
> /home/prapal/Source/libvirt/build/tests/domaincapstest
> 
> Most of these tests fail with following signature:
> 
> 2) qemu_7.2.0.x86_64                                                 ...
> In
> '/home/username/Source/libvirt/tests/domaincapsdata/qemu_7.2.0.x86_64.xml':
> Offset 275
> Expect [  <value>efi</value>
>     <]
> Actual [<]
> 
> 
> qemufirmwaretest:
> 
> $ VIR_TEST_DEBUG=1 VIR_TEST_RANGE=19-20,22,24-26
> /home/prapal/Source/libvirt/build/tests/qemufirmwaretest
> TEST: qemufirmwaretest
> 19) QEMU FW precedence test ... Unexpected path. Expected
> //share/qemu/firmware/30-edk2-ovmf-4m-qcow2-x64-sb-enrolled.json got
> /etc/qemu/firmware/59-combined.json
> FAILED
> 20) QEMU FW SUPPORTED pc-i440fx-3.1 VIR_ARCH_X86_64 ... Mismatch in
> supported interfaces. Expected 0x6 got 0x2
> FAILED
> 22) QEMU FW SUPPORTED pc-q35-3.1 VIR_ARCH_X86_64 ... Unexpected FW image:
> /usr/share/edk2/ovmf/OVMF.secboot.fd NVRAM: <null>
> Expected: /usr/share/edk2/ovmf/OVMF_CODE_4M.secboot.qcow2 NVRAM:
> /usr/share/edk2/ovmf/OVMF_VARS_4M.secboot.qcow2
> FAILED
> 24) QEMU FW SUPPORTED microvm VIR_ARCH_X86_64 ... Mismatch in supported
> interfaces. Expected 0x4 got 0x0
> FAILED
> 25) QEMU FW SUPPORTED virt-3.1 VIR_ARCH_AARCH64 ... Mismatch in supported
> interfaces. Expected 0x4 got 0x0
> FAILED
> 26) QEMU FW SUPPORTED virt VIR_ARCH_RISCV64 ... Mismatch in supported
> interfaces. Expected 0x4 got 0x0
> FAILED
> 6 tests failed. Run them using:
> VIR_TEST_DEBUG=1 VIR_TEST_RANGE=19-20,22,24-26
> /home/prapal/Source/libvirt/build/tests/qemufirmwaretest

My hunch would be that the mocks, which are meant to make the tests
independent from your host system are not applied.

Is your source directory the git tree? How did you configure the build?

Did you follow any of our guides? e.g.
https://libvirt.org/compiling.html I want do understand if any of our
documentation is broken.

Reply via email to