Skip to content

Commit

Permalink
Add more test job timeouts (#41162)
Browse files Browse the repository at this point in the history
This adds some upper bound time limits on our test jobs to ensure we
aren't slipping on test times or letting CI job stall and waste
concurrency.
  • Loading branch information
ijjk committed Oct 4, 2022
1 parent 0d5886f commit 5af1a93
Showing 1 changed file with 12 additions and 0 deletions.
12 changes: 12 additions & 0 deletions .github/workflows/build_test_deploy.yml
Expand Up @@ -208,6 +208,7 @@ jobs:
name: Test Unit
runs-on: ubuntu-latest
needs: [build, build-native-test]
timeout-minutes: 5
env:
NEXT_TELEMETRY_DISABLED: 1
NEXT_TEST_JOB: 1
Expand Down Expand Up @@ -242,6 +243,7 @@ jobs:
name: Test Development
runs-on: ubuntu-latest
needs: [build, build-native-test]
timeout-minutes: 20
env:
NEXT_TELEMETRY_DISABLED: 1
NEXT_TEST_JOB: 1
Expand Down Expand Up @@ -362,6 +364,7 @@ jobs:
name: Test Development (E2E)
runs-on: ubuntu-latest
needs: [build, build-native-test]
timeout-minutes: 20
env:
NEXT_TELEMETRY_DISABLED: 1
NEXT_TEST_JOB: 1
Expand Down Expand Up @@ -502,6 +505,7 @@ jobs:
name: Test Production
runs-on: ubuntu-latest
needs: [build, build-native-test]
timeout-minutes: 15
env:
NEXT_TELEMETRY_DISABLED: 1
NEXT_TEST_JOB: 1
Expand Down Expand Up @@ -602,6 +606,7 @@ jobs:
name: Test Production (E2E)
runs-on: ubuntu-latest
needs: [build, build-native-test]
timeout-minutes: 25
env:
NEXT_TELEMETRY_DISABLED: 1
NEXT_TEST_JOB: 1
Expand Down Expand Up @@ -704,6 +709,7 @@ jobs:
name: Test Integration
runs-on: ubuntu-latest
needs: [build, build-native-test]
timeout-minutes: 20
env:
NEXT_TELEMETRY_DISABLED: 1
NEXT_TEST_JOB: 1
Expand Down Expand Up @@ -785,6 +791,7 @@ jobs:
name: Test Electron
runs-on: ubuntu-latest
needs: [build, build-native-test]
timeout-minutes: 5
env:
NEXT_TELEMETRY_DISABLED: 1
NEXT_TEST_JOB: 1
Expand Down Expand Up @@ -840,6 +847,7 @@ jobs:
name: Test Firefox (production)
runs-on: ubuntu-latest
needs: [build, build-native-test]
timeout-minutes: 5
env:
BROWSER_NAME: 'firefox'
NEXT_TELEMETRY_DISABLED: 1
Expand Down Expand Up @@ -874,6 +882,7 @@ jobs:
name: Test Safari (production)
runs-on: ubuntu-latest
needs: [build, build-native-test]
timeout-minutes: 10
env:
BROWSER_NAME: 'safari'
NEXT_TEST_MODE: 'start'
Expand Down Expand Up @@ -919,6 +928,7 @@ jobs:
name: Test Safari 10.1 (nav)
runs-on: ubuntu-latest
needs: [build, build-native-test]
timeout-minutes: 5
env:
BROWSERSTACK: true
LEGACY_SAFARI: true
Expand Down Expand Up @@ -965,6 +975,7 @@ jobs:
name: Test Firefox Node.js 18
runs-on: ubuntu-latest
needs: [build, testFirefox, build-native-test]
timeout-minutes: 5
env:
BROWSER_NAME: 'firefox'
NEXT_TELEMETRY_DISABLED: 1
Expand Down Expand Up @@ -1237,6 +1248,7 @@ jobs:
test-wasm:
name: Test the wasm build
runs-on: ubuntu-latest
timeout-minutes: 10
needs: [build, build-native-test, build-wasm-dev]

steps:
Expand Down

0 comments on commit 5af1a93

Please sign in to comment.