Skip to content

Commit

Permalink
Set shorter workflow timeouts
Browse files Browse the repository at this point in the history
The default timeout is 6 hours, which just causes unnecessary hangs,
delays, and blocked self-hosted runners if anything goes wrong (such as
pylint-dev/pylint#3899).

A normal run takes about 10 minutes, so set a timeout of 30 minutes.
Use one hour for the full container rebuild+validation+upload to be on
the safe side (it usually takes between 15 and 18 minutes).
  • Loading branch information
martinpitt committed Nov 24, 2020
1 parent cef9b96 commit db4c400
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 0 deletions.
1 change: 1 addition & 0 deletions .github/workflows/container-autoupdate.yml
Expand Up @@ -15,6 +15,7 @@ jobs:
matrix:
branch:
- master
timeout-minutes: 60
steps:
- name: Checkout anaconda repository
uses: actions/checkout@v2
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/validate-rhel-8.yml
Expand Up @@ -45,6 +45,7 @@ jobs:
needs: pr-info
if: needs.pr-info.outputs.base_ref == 'rhel-8' && needs.pr-info.outputs.allowed_user == 'true'
runs-on: [self-hosted, ci-tasks, rhel-8]
timeout-minutes: 30
steps:
# we post statuses manually as this does not run from a pull_request event
# https://developer.github.com/v3/repos/statuses/#create-a-status
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/validate.yml
Expand Up @@ -13,6 +13,7 @@ jobs:
env:
CI_TAG: '${{ matrix.release }}'
CONTAINER_BUILD_ARGS: '${{ matrix.build-args }}'
timeout-minutes: 30

steps:
- name: Clone repository
Expand Down Expand Up @@ -58,6 +59,7 @@ jobs:
image: docker.io/fedora:rawhide
# HACK: bash's builtin `test -r` fails due to incompatible seccomp profile
options: --security-opt=seccomp=unconfined
timeout-minutes: 30
steps:
- name: Clone repository
uses: actions/checkout@v2
Expand Down

0 comments on commit db4c400

Please sign in to comment.