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.) Reviewed-by: George Melikov <mail@gmelikov.ru> Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov> Signed-off-by: Rich Ercolani <rincebrain@gmail.com> Closes #12999
This commit is contained in:
parent
4372e96f4b
commit
cd26b217dc
1
.github/workflows/zfs-tests-functional.yml
vendored
1
.github/workflows/zfs-tests-functional.yml
vendored
@ -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: |
|
||||
|
1
.github/workflows/zfs-tests-sanity.yml
vendored
1
.github/workflows/zfs-tests-sanity.yml
vendored
@ -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: |
|
||||
|
Loading…
Reference in New Issue
Block a user