[PATCH] tests/functional/test_ppc64_replay: Mark the e500 test as flaky

[PATCH] tests/functional/test_ppc64_replay: Mark the e500 test as flaky

[Thomas Huth]

The test_ppc64_e500 occasionally fails (it just hangs and finally times
out). The same issue could also be reproduced with the former Avocado
test already (especially if the host system is under heavy load), so
it's not a new regression. It's very likely the issue that has been
filed at https://gitlab.com/qemu-project/qemu/-/issues/2523 instead (e.g.
I could not reproduce the issue in older commits before commit 578912ad),
so use this URL for the reasoning in the the decorator.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 Here are some examples where the test failed:
  https://gitlab.com/thuth/qemu/-/jobs/9391454847
  https://gitlab.com/thuth/qemu/-/jobs/9457171828
  https://gitlab.com/thuth/qemu/-/jobs/9458371668

 tests/functional/test_ppc64_replay.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/tests/functional/test_ppc64_replay.py b/tests/functional/test_ppc64_replay.py
index 48ce1b7f1e1..e8c9c4bcbf8 100755
--- a/tests/functional/test_ppc64_replay.py
+++ b/tests/functional/test_ppc64_replay.py
@@ -5,7 +5,7 @@
 #
 # SPDX-License-Identifier: GPL-2.0-or-later
 
-from qemu_test import Asset
+from qemu_test import Asset, skipFlakyTest
 from replay_kernel import ReplayKernelBase
 
 
@@ -16,6 +16,7 @@ class Ppc64Replay(ReplayKernelBase):
          'day19.tar.xz'),
         '20b1bb5a8488c664defbb5d283addc91a05335a936c63b3f5ff7eee74b725755')
 
+    @skipFlakyTest('https://gitlab.com/qemu-project/qemu/-/issues/2523')
     def test_ppc64_e500(self):
         self.set_machine('ppce500')
         self.cpu = 'e5500'
-- 
2.48.1

Re: [PATCH] tests/functional/test_ppc64_replay: Mark the e500 test as flaky

[Daniel P. Berrangé]

On Thu, Mar 20, 2025 at 07:50:12AM +0100, Thomas Huth wrote:
> The test_ppc64_e500 occasionally fails (it just hangs and finally times
> out). The same issue could also be reproduced with the former Avocado
> test already (especially if the host system is under heavy load), so
> it's not a new regression. It's very likely the issue that has been
> filed at https://gitlab.com/qemu-project/qemu/-/issues/2523 instead (e.g.
> I could not reproduce the issue in older commits before commit 578912ad),
> so use this URL for the reasoning in the the decorator.
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  Here are some examples where the test failed:
>   https://gitlab.com/thuth/qemu/-/jobs/9391454847
>   https://gitlab.com/thuth/qemu/-/jobs/9457171828
>   https://gitlab.com/thuth/qemu/-/jobs/9458371668
> 
>  tests/functional/test_ppc64_replay.py | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)

Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>


With regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|