gitlab-ci: increase clang-user timeout

[PATCH v2] gitlab-ci: increase clang-user timeout

Stefan Hajnoczi posted 1 patch 1 year, 6 months ago

Download mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20221104113659.427690-1-stefanha@redhat.com

Maintainers: "Alex Bennée" <alex.bennee@linaro.org>, "Philippe Mathieu-Daudé" <philmd@linaro.org>, Thomas Huth <thuth@redhat.com>, Wainer dos Santos Moschetta <wainersm@redhat.com>, Beraldo Leal <bleal@redhat.com>

.gitlab-ci.d/buildtest.yml | 1 +
1 file changed, 1 insertion(+)

Expand all Fold all

[PATCH v2] gitlab-ci: increase clang-user timeout

Posted by Stefan Hajnoczi 1 year, 6 months ago

The clang-user test exceeds the 1 hour timeout occassionally.

Philippe Mathieu-Daudé has pointed out that the number of tcg tests has
increased since QEMU 7.1. The execution time therefore probably reflects
a legitimate increase in tests rather than a performance regression.

Bump the timeout to prevent CI failures.

Suggested-by: Thomas Huth <thuth@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
---
v2:
- Increase the timeout instead of splitting tests [Thomas]
---
 .gitlab-ci.d/buildtest.yml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/.gitlab-ci.d/buildtest.yml b/.gitlab-ci.d/buildtest.yml
index 6c05c46397..7173749c52 100644
--- a/.gitlab-ci.d/buildtest.yml
+++ b/.gitlab-ci.d/buildtest.yml
@@ -327,6 +327,7 @@ clang-user:
   extends: .native_build_job_template
   needs:
     job: amd64-debian-user-cross-container
+  timeout: 70m
   variables:
     IMAGE: debian-all-test-cross
     CONFIGURE_ARGS: --cc=clang --cxx=clang++ --disable-system
-- 
2.38.1

Re: [PATCH v2] gitlab-ci: increase clang-user timeout

Posted by Thomas Huth 1 year, 6 months ago

On 04/11/2022 12.36, Stefan Hajnoczi wrote:
> The clang-user test exceeds the 1 hour timeout occassionally.
> 
> Philippe Mathieu-Daudé has pointed out that the number of tcg tests has
> increased since QEMU 7.1. The execution time therefore probably reflects
> a legitimate increase in tests rather than a performance regression.
> 
> Bump the timeout to prevent CI failures.
> 
> Suggested-by: Thomas Huth <thuth@redhat.com>
> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
> ---
> v2:
> - Increase the timeout instead of splitting tests [Thomas]
> ---
>   .gitlab-ci.d/buildtest.yml | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/.gitlab-ci.d/buildtest.yml b/.gitlab-ci.d/buildtest.yml
> index 6c05c46397..7173749c52 100644
> --- a/.gitlab-ci.d/buildtest.yml
> +++ b/.gitlab-ci.d/buildtest.yml
> @@ -327,6 +327,7 @@ clang-user:
>     extends: .native_build_job_template
>     needs:
>       job: amd64-debian-user-cross-container
> +  timeout: 70m
>     variables:
>       IMAGE: debian-all-test-cross
>       CONFIGURE_ARGS: --cc=clang --cxx=clang++ --disable-system

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