[PATCH] .gitlab-ci.d/crossbuilds.yml: Force 'make check' to -j2 for cross-i686-tci

Peter Maydell posted 1 patch 2 months, 1 week ago
.gitlab-ci.d/crossbuilds.yml | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
[PATCH] .gitlab-ci.d/crossbuilds.yml: Force 'make check' to -j2 for cross-i686-tci
Posted by Peter Maydell 2 months, 1 week ago
In commit 1374ed49e1453c300 we forced the cross-i686-tci job to -j1 to
see if this helped with test timeouts. It seems to help with that but
on the other hand we now sometimes run into the overall 60 minute
job timeout. Try -j2 instead.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
---
 .gitlab-ci.d/crossbuilds.yml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/.gitlab-ci.d/crossbuilds.yml b/.gitlab-ci.d/crossbuilds.yml
index 1e21d082aa4..95dfc392244 100644
--- a/.gitlab-ci.d/crossbuilds.yml
+++ b/.gitlab-ci.d/crossbuilds.yml
@@ -62,11 +62,11 @@ cross-i686-tci:
     IMAGE: debian-i686-cross
     ACCEL: tcg-interpreter
     EXTRA_CONFIGURE_OPTS: --target-list=i386-softmmu,i386-linux-user,aarch64-softmmu,aarch64-linux-user,ppc-softmmu,ppc-linux-user --disable-plugins --disable-kvm
-    # Force tests to run in series, to see whether this
+    # Force tests to run with reduced parallelism, to see whether this
     # reduces the flakiness of this CI job. The CI
     # environment by default shows us 8 CPUs and so we
     # would otherwise be using a parallelism of 9.
-    MAKE_CHECK_ARGS: check check-tcg -j1
+    MAKE_CHECK_ARGS: check check-tcg -j2
 
 cross-mipsel-system:
   extends: .cross_system_build_job
-- 
2.34.1
Re: [PATCH] .gitlab-ci.d/crossbuilds.yml: Force 'make check' to -j2 for cross-i686-tci
Posted by Thomas Huth 2 months, 1 week ago
On 16/09/2024 15.49, Peter Maydell wrote:
> In commit 1374ed49e1453c300 we forced the cross-i686-tci job to -j1 to
> see if this helped with test timeouts. It seems to help with that but
> on the other hand we now sometimes run into the overall 60 minute
> job timeout. Try -j2 instead.
> 
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> ---
>   .gitlab-ci.d/crossbuilds.yml | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/.gitlab-ci.d/crossbuilds.yml b/.gitlab-ci.d/crossbuilds.yml
> index 1e21d082aa4..95dfc392244 100644
> --- a/.gitlab-ci.d/crossbuilds.yml
> +++ b/.gitlab-ci.d/crossbuilds.yml
> @@ -62,11 +62,11 @@ cross-i686-tci:
>       IMAGE: debian-i686-cross
>       ACCEL: tcg-interpreter
>       EXTRA_CONFIGURE_OPTS: --target-list=i386-softmmu,i386-linux-user,aarch64-softmmu,aarch64-linux-user,ppc-softmmu,ppc-linux-user --disable-plugins --disable-kvm
> -    # Force tests to run in series, to see whether this
> +    # Force tests to run with reduced parallelism, to see whether this
>       # reduces the flakiness of this CI job. The CI
>       # environment by default shows us 8 CPUs and so we
>       # would otherwise be using a parallelism of 9.
> -    MAKE_CHECK_ARGS: check check-tcg -j1
> +    MAKE_CHECK_ARGS: check check-tcg -j2
>   
>   cross-mipsel-system:
>     extends: .cross_system_build_job

Reviewed-by: Thomas Huth <thuth@redhat.com>