Re: [libvirt PATCH] ci: print test suite logs on failure for Cirrus jobs

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

 



On Tue, Apr 26, 2022 at 10:12:17AM +0100, Daniel P. Berrangé wrote:
> We don't have access to the 'testlog.txt' file, so we need meson to
> print the failures for any broken tests directly.
>
> Signed-off-by: Daniel P. Berrangé <berrange@xxxxxxxxxx>
> ---
>  ci/cirrus/build.yml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Andrea Bolognani <abologna@xxxxxxxxxx>

> The CI pipeline for macOS started failing a few days ago. It was not
> triggered by any commit, as the pipeline immediately preceeding the
> first failure used the same commit hash on master. The logs show
> glib2 being updated from 2.72.0 to 2.72.1
>
> With this patch applied I can see the test logs
>
>    https://gitlab.com/berrange/libvirt/-/jobs/2376891003
>
> and all the failing tests are hitting:
>
> (process:50961): GLib-WARNING **: 01:56:14.162: poll(2) failed due to:
> Bad file descriptor.
>
> so something todo with the QEMU monitor/event loop AFAIK, but not
> sure what.

Looking at

  https://gitlab.gnome.org/GNOME/glib/-/releases/2.72.1

the interesting change seems to be

  * Fix detection of broken poll() function on macOS

which would correspond to

  https://gitlab.gnome.org/GNOME/glib/-/merge_requests/2571

So it looks like this was very intentional, and motivated by the
needs of QEMU of all projects... I wonder what they're doing
differently from us?

-- 
Andrea Bolognani / Red Hat / Virtualization





[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