Skip to content

Commit

Permalink
chore(workflows) Give all of the CI jobs sensible timeouts (#4685)
Browse files Browse the repository at this point in the history
  • Loading branch information
Rheeseyb authored Jan 4, 2024
1 parent 87d26fd commit 5f4ade7
Show file tree
Hide file tree
Showing 18 changed files with 27 additions and 0 deletions.
1 change: 1 addition & 0 deletions .github/workflows/cache-pnpm-install.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ on:
jobs:
cache-pnpm-store:
name: Install everything with PNPM and cache it
timeout-minutes: 15
runs-on: ubuntu-latest
env:
UTOPIA_SHA: ${{ github.sha }}
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/discord-discussion-comments.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ permissions:
jobs:
post-to-discord:
name: Post Comment To Discord
timeout-minutes: 5
runs-on: ubuntu-latest
steps:
- name: Build Embeds JSON
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/discord-discussion-opened.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ permissions:
jobs:
post-to-discord:
name: Post Notification To Discord
timeout-minutes: 5
runs-on: ubuntu-latest
steps:
- name: Build Embeds JSON
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/discord-issue-closed.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ permissions:
jobs:
post-to-discord:
name: Post Notification To Discord
timeout-minutes: 5
runs-on: ubuntu-latest
steps:
- name: Build Embeds JSON
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/discord-issue-comments.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ permissions:
jobs:
post-to-discord:
name: Post Comment To Discord
timeout-minutes: 5
runs-on: ubuntu-latest
if: ${{ !github.event.issue.pull_request }}
steps:
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/discord-issue-opened.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ permissions:
jobs:
post-to-discord:
name: Post Notification To Discord
timeout-minutes: 5
runs-on: ubuntu-latest
steps:
- name: Build Embeds JSON
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/discord-pr-closed.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ permissions:
jobs:
post-to-discord:
name: Post Notification To Discord
timeout-minutes: 5
runs-on: ubuntu-latest
steps:
- name: Build Merged Args
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/discord-pr-comments.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@ permissions:
jobs:
post-to-discord:
name: Post Notification To Discord
timeout-minutes: 5
runs-on: ubuntu-latest
if: ${{ github.event.pull_request || github.event.issue.pull_request }}
steps:
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/discord-pr-opened.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ permissions:
jobs:
post-to-discord:
name: Post Notification To Discord
timeout-minutes: 5
runs-on: ubuntu-latest
if: ${{ !github.event.pull_request.draft }}
steps:
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/discord-pr-review.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ permissions:
jobs:
post-to-discord:
name: Post Notification To Discord
timeout-minutes: 5
runs-on: ubuntu-latest
if: ${{ github.event.review.state == 'approved' || github.event.review.state == 'changes_requested' }}
steps:
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/editor-sharded-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ on:
jobs:
test-editor-karma-shard:
name: Test Editor PR – Karma tests (Shard ${{ inputs.shard_number }})
timeout-minutes: 15
runs-on: ubuntu-latest
env:
UTOPIA_SHA: ${{ github.sha }}
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/google-webfonts-list-updater.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ on:
jobs:
google-web-fonts-updater:
name: Google Web Fonts Updater
timeout-minutes: 5
runs-on: ubuntu-20.04
steps:
- name: Checkout code
Expand Down
5 changes: 5 additions & 0 deletions .github/workflows/master-pushes.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ jobs:

test-editor-code:
name: Test Editor - TypeScript, ESLint, dependency-cruiser
timeout-minutes: 15
runs-on: ubuntu-latest
needs: [cache-pnpm-store]
env:
Expand Down Expand Up @@ -44,6 +45,7 @@ jobs:

test-editor-jest:
name: Test Editor - Jest tests
timeout-minutes: 15
runs-on: ubuntu-latest
needs: [cache-pnpm-store]
env:
Expand Down Expand Up @@ -97,6 +99,7 @@ jobs:

test-server:
name: Test Server
timeout-minutes: 15
runs-on: ubuntu-latest
env:
UTOPIA_SHA: ${{ github.sha }}
Expand Down Expand Up @@ -137,6 +140,7 @@ jobs:

trigger-deploy:
name: Trigger Deploy
timeout-minutes: 5
needs:
[
test-editor-code,
Expand All @@ -156,6 +160,7 @@ jobs:

post-failure-to-discord:
name: Post Failure Message To Discord
timeout-minutes: 5
runs-on: ubuntu-latest
if: failure()
needs: [trigger-deploy]
Expand Down
6 changes: 6 additions & 0 deletions .github/workflows/pull-requests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ jobs:

test-editor-code:
name: Test Editor PR – TypeScript, ESLint, dependency-cruiser
timeout-minutes: 15
runs-on: ubuntu-latest
needs: [cache-pnpm-store]
env:
Expand Down Expand Up @@ -41,8 +42,10 @@ jobs:
- name: Run tsc, eslint, depdendency-cruiser and the website tests
if: steps.cache-editor-tests.outputs.cache-hit != 'true'
run: nix-shell --arg includeServerBuildSupport false --arg includeRunLocallySupport false --run check-editor-code-ci

test-editor-jest:
name: Test Editor PR – Jest tests
timeout-minutes: 15
runs-on: ubuntu-latest
needs: [cache-pnpm-store]
env:
Expand Down Expand Up @@ -96,6 +99,7 @@ jobs:

deploy-staging:
name: Deploy Staging Editor
timeout-minutes: 15
runs-on: ubuntu-latest
needs: [cache-pnpm-store]
env:
Expand Down Expand Up @@ -212,6 +216,7 @@ jobs:
performance-test:
name: Run Performance Tests
timeout-minutes: 15
runs-on: self-hosted
needs: [deploy-staging, cache-pnpm-store]
env:
Expand Down Expand Up @@ -317,6 +322,7 @@ jobs:

system-test:
name: Run System Tests
timeout-minutes: 15
runs-on: ubuntu-latest
needs: [deploy-staging, cache-pnpm-store]
env:
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/screenshot-production.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ jobs:

take-screenshot:
name: Take Screenshot
timeout-minutes: 15
runs-on: ubuntu-latest
needs: [cache-pnpm-store]
steps:
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/screenshot-staging.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ jobs:

take-screenshot:
name: Take Screenshot
timeout-minutes: 15
runs-on: ubuntu-latest
needs: [cache-pnpm-store]
steps:
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/tag-release-automated.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ jobs:
system-test:
# For automated releases we want to check it is safe first
name: Run System Tests
timeout-minutes: 15
runs-on: ubuntu-latest
needs: [cache-pnpm-store]
env:
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/tag-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ on:
jobs:
release:
name: Create Release
timeout-minutes: 15
runs-on: ubuntu-latest
steps:
- name: Checkout code
Expand Down

0 comments on commit 5f4ade7

Please sign in to comment.