tests/functional: Increase the timeout of the mips64el_replay test

[PATCH] tests/functional: Increase the timeout of the mips64el_replay test

Posted by Thomas Huth 3 weeks, 6 days ago

We run the gitlab-CI with the untrusted tests enabled, and
the test_replay_mips64el_malta_5KEc_cpio subtest is rather slow,
so this already hit the standard 90 seconds timeout in the CI.
Increase the timeout for more headroom.

Reported-by: Stefan Hajnoczi <stefanha@redhat.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 tests/functional/meson.build | 1 +
 1 file changed, 1 insertion(+)

diff --git a/tests/functional/meson.build b/tests/functional/meson.build
index 3fd2652c078..97c3f4ad4e9 100644
--- a/tests/functional/meson.build
+++ b/tests/functional/meson.build
@@ -36,6 +36,7 @@ test_timeouts = {
   'intel_iommu': 300,
   'mips_malta' : 120,
   'mipsel_replay' : 480,
+  'mips64el_replay' : 180,
   'netdev_ethtool' : 180,
   'ppc_40p' : 240,
   'ppc64_hv' : 1000,
-- 
2.48.1

Re: [PATCH] tests/functional: Increase the timeout of the mips64el_replay test

Posted by Stefan Hajnoczi 3 weeks, 5 days ago

On Wed, Mar 05, 2025 at 08:43:53AM +0100, Thomas Huth wrote:
> We run the gitlab-CI with the untrusted tests enabled, and
> the test_replay_mips64el_malta_5KEc_cpio subtest is rather slow,
> so this already hit the standard 90 seconds timeout in the CI.
> Increase the timeout for more headroom.
> 
> Reported-by: Stefan Hajnoczi <stefanha@redhat.com>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  tests/functional/meson.build | 1 +
>  1 file changed, 1 insertion(+)

Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>

Re: [PATCH] tests/functional: Increase the timeout of the mips64el_replay test

Posted by Daniel P. Berrangé 3 weeks, 5 days ago

On Wed, Mar 05, 2025 at 08:43:53AM +0100, Thomas Huth wrote:
> We run the gitlab-CI with the untrusted tests enabled, and
> the test_replay_mips64el_malta_5KEc_cpio subtest is rather slow,
> so this already hit the standard 90 seconds timeout in the CI.
> Increase the timeout for more headroom.
> 
> Reported-by: Stefan Hajnoczi <stefanha@redhat.com>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  tests/functional/meson.build | 1 +
>  1 file changed, 1 insertion(+)

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 :|