diff mbox series

tests/functional/test_ppc64_replay: Mark the e500 test as flaky

Message ID 20250320065012.309520-1-thuth@redhat.com (mailing list archive)
State New
Headers show
Series tests/functional/test_ppc64_replay: Mark the e500 test as flaky | expand

Commit Message

Thomas Huth March 20, 2025, 6:50 a.m. UTC
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(-)

Comments

Daniel P. Berrangé March 20, 2025, 9:32 a.m. UTC | #1
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
diff mbox series

Patch

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'