From 518a215eec4812cd013ab0c4126530fe21692a6c Mon Sep 17 00:00:00 2001 From: Rich Ercolani Date: Sat, 22 Jan 2022 23:25:17 -0500 Subject: [PATCH] Add explicit timeout to test step If we die from timeout of the whole GH action run, we don't run the collect step afterward, which can make it hard to investigate the timeout. If we timeout first in the test action, though, it qualifies as failure, and collects appropriately. (330 minutes seems like an acceptable tradeoff between the 6h timeout by default on the action and the 4h and change "functional" usually takes.) Signed-off-by: Rich Ercolani --- .github/workflows/zfs-tests-functional.yml | 1 + .github/workflows/zfs-tests-sanity.yml | 1 + 2 files changed, 2 insertions(+) diff --git a/.github/workflows/zfs-tests-functional.yml b/.github/workflows/zfs-tests-functional.yml index 19d3f57baec9..40d03d25d4e5 100644 --- a/.github/workflows/zfs-tests-functional.yml +++ b/.github/workflows/zfs-tests-functional.yml @@ -64,6 +64,7 @@ jobs: - name: Tests run: | /usr/share/zfs/zfs-tests.sh -vR -s 3G + timeout-minutes: 330 - name: Prepare artifacts if: failure() run: | diff --git a/.github/workflows/zfs-tests-sanity.yml b/.github/workflows/zfs-tests-sanity.yml index 2b97fd61822f..d1dc0e88c173 100644 --- a/.github/workflows/zfs-tests-sanity.yml +++ b/.github/workflows/zfs-tests-sanity.yml @@ -60,6 +60,7 @@ jobs: - name: Tests run: | /usr/share/zfs/zfs-tests.sh -vR -s 3G -r sanity + timeout-minutes: 330 - name: Prepare artifacts if: failure() run: |