From 322a83f321ccd503e51f91ad03fbfb483ac5bab8 Mon Sep 17 00:00:00 2001
From: Eric Engestrom <eric@igalia.com>
Date: Thu, 24 Oct 2024 20:10:33 +0200
Subject: [PATCH] ci/build: lower most job's timeout from 30 to 15 min

Out of 5 run during busy hours, all the jobs that once took 5+ min are:
build-for-tests / debian-arm64-asan      6 min
build-only      / debian-s390x           6 min
build-only      / debian-android         7 min
build-only      / debian-clang           8 min
build-for-tests / debian-arm32-asan      8 min
build-only      / debian-vulkan         11 min
build-for-tests / debian-testing        12 min
build-only      / debian-testing-msan   12 min
build-only      / debian-clang-release  13 min
build-only      / alpine-build-testing  14 min
build-for-tests / debian-testing-asan   21 min

The jobs at 10+ min are considered to take long enough that they might
risk crossing the 15 min mark, so let's keep these ones at 30 min and
lower the timeout for everyone else to 15 min.

It's worth pointing out that debian-testing-asan is a build-for-tests
job and as such it blocks build-only jobs from running until it's
finished, which can be a problem for a job that has been seen taking 20+
minutes. We should do something about that, but that's not the topic of
this MR.

Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/31846>
---
 .gitlab-ci/build/gitlab-ci.yml | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/.gitlab-ci/build/gitlab-ci.yml b/.gitlab-ci/build/gitlab-ci.yml
index 9d68add913a..36c3e1440cc 100644
--- a/.gitlab-ci/build/gitlab-ci.yml
+++ b/.gitlab-ci/build/gitlab-ci.yml
@@ -10,7 +10,7 @@
     # something has gone wrong and the job will never terminate, so we should
     # instead timeout so that the retry mechanism can kick in.
     # A few exception are made, see overrides in the rest of this file.
-    BUILD_JOB_TIMEOUT: 30m
+    BUILD_JOB_TIMEOUT: 15m
   timeout: 1h
   # We don't want to download any previous job's artifacts
   dependencies: []
@@ -96,6 +96,7 @@ debian-testing:
     - .ci-deqp-artifacts
   stage: build-for-tests
   variables:
+    BUILD_JOB_TIMEOUT: 30m
     UNWIND: "enabled"
     DRI_LOADERS: >
       -D glx=dri
@@ -129,6 +130,7 @@ debian-testing-asan:
     - debian-testing
   stage: build-for-tests
   variables:
+    BUILD_JOB_TIMEOUT: 30m
     C_ARGS: >
       -Wno-error=stringop-truncation
     EXTRA_OPTION: >
@@ -159,6 +161,7 @@ debian-testing-msan:
     - debian-clang
   # `needs:` inherited from debian-clang
   variables:
+    BUILD_JOB_TIMEOUT: 30m
     # l_undef is incompatible with msan
     EXTRA_OPTION:
       -D b_sanitize=memory
@@ -297,6 +300,7 @@ alpine-build-testing:
     - !reference [.use-alpine/x86_64_build, needs]
     - !reference [.build-for-tests-jobs]
   variables:
+    BUILD_JOB_TIMEOUT: 30m
     BUILDTYPE: "release"
     C_ARGS: >
       -Wno-error=cpp
@@ -672,6 +676,7 @@ debian-clang-release:
   extends: debian-clang
   # `needs:` inherited from debian-clang
   variables:
+    BUILD_JOB_TIMEOUT: 30m
     BUILDTYPE: "release"
     DRI_LOADERS: >
       -D glx=xlib
@@ -709,6 +714,7 @@ debian-vulkan:
     - !reference [.meson-build, needs]
     - !reference [.build-for-tests-jobs]
   variables:
+    BUILD_JOB_TIMEOUT: 30m
     BUILDTYPE: debug
     LLVM_VERSION: 15
     UNWIND: "disabled"