GNU bug report logs - #70936
[PATCH gnome-team 0/3] Fix some broken tests.

Previous Next

Full log

View this message in rfc822 format

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
Cc: 70936 <at> debbugs.gnu.org, vivien <at> planete-kraus.eu
Subject: [bug#70936] [PATCH gnome-team 3/3] gnu: qemu: Disable more tests.
Date: Sun, 23 Jun 2024 23:14:55 -0400

Hello!

Maxim Cournoyer <maxim.cournoyer <at> gmail.com> writes:

> Liliana Marie Prikler <liliana.prikler <at> gmail.com> writes:
>
>> * gnu/packages/virtualization.scm (qemu)[disable-unusable-tests]: Also disable
>> qtest/ahci-test and qemu-iotests/copy-before-write.
>> ---
>>  gnu/packages/virtualization.scm | 7 ++++++-
>>  1 file changed, 6 insertions(+), 1 deletion(-)
>>
>> diff --git a/gnu/packages/virtualization.scm b/gnu/packages/virtualization.scm
>> index 34cccd6550..5389af62f0 100644
>> --- a/gnu/packages/virtualization.scm
>> +++ b/gnu/packages/virtualization.scm
>> @@ -305,7 +305,12 @@ (define-public qemu
>>                (substitute* "tests/qtest/meson.build"
>>                  ;; These tests fail to get the expected number of tests
>>                  ;; on arm platforms.
>> -                (("'arm-cpu-features',") ""))))
>> +                (("'arm-cpu-features',") "")
>> +                ;; XXX: Why does this (sometimes) fail?
>
> Someone reported it already upstream:
> <https://gitlab.com/qemu-project/qemu/-/issues/2121>.  Please add a
> reference to it.
>
>> +                (("\\['ahci-test'\\]") "[]"))
>> +              ;; This test appears to be missing permissions.
>> +              ;; Very sus 🤔
>
> How can it be missing permissions if it is sometimes working?  Perhaps
> more something racy where the file hasn't been created yet.  I'd
> personally rather not see emojis or comments of dubious utility ("Very
> suspect") in the source.  Perhaps add a short excerpt of the error for
> the next curious person.
>
>> +              (delete-file "tests/qemu-iotests/tests/copy-before-write")))
>>            #$@(if (target-riscv64?)
>>                   '((add-after 'unpack 'disable-some-tests
>>                       (lambda _
>
> Otherwise, LGTM, with the added reference to the upstream issue.  Thanks
> for taking the time to address flaky tests!  They are a real drag when
> left unchecked.

Ping, in case it had fallen in cracks :-).

-- 
Thanks,
Maxim

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.