From dd694ac6e59419fb3289855c9298bf8dda045fcf Mon Sep 17 00:00:00 2001 From: Eric Engestrom Date: Fri, 16 Aug 2024 18:03:38 +0200 Subject: [PATCH] broadcom/ci: set default TEST_PHASE_TIMEOUT_MINUTES so that jobs never timeout without uploading results The CI-wide default is 20min, so if we set a 20min job timeout here, we can't get the results of our jobs when they timeout. Instead of setting the test timeout to 15min, which would be too short for some jobs, leave it at 20min (but be explicit about it and protect against a future change of that default), and bump the job timeout by 5min to allow for results to be uploaded. Part-of: --- src/broadcom/ci/gitlab-ci-inc.yml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/broadcom/ci/gitlab-ci-inc.yml b/src/broadcom/ci/gitlab-ci-inc.yml index 46257f8e37a..23cf83d3772 100644 --- a/src/broadcom/ci/gitlab-ci-inc.yml +++ b/src/broadcom/ci/gitlab-ci-inc.yml @@ -152,10 +152,11 @@ - ./install/bare-metal/poe-powered.sh variables: BOOT_PHASE_TIMEOUT_SECONDS: 90 + TEST_PHASE_TIMEOUT_MINUTES: 20 # when overriding, keep this 5min less than the job's `timeout:` HWCI_TEST_SCRIPT: "/install/deqp-runner.sh" FLAKES_CHANNEL: "#videocore-ci" FARM: igalia - timeout: 20m + timeout: 25m .broadcom-test:arm64: extends: