Skip to content

Commit

Permalink
chore(ci): Add a timeout to all CI jobs (#19895)
Browse files Browse the repository at this point in the history
* chore(ci): Add a timeout to all CI jobs

To override the default of 6 hours. This'll let us bump up the merge queue timeout.

Signed-off-by: Jesse Szwedko <[email protected]>

* Missed that e2e tests already had a timeout

Signed-off-by: Jesse Szwedko <[email protected]>

---------

Signed-off-by: Jesse Szwedko <[email protected]>
  • Loading branch information
jszwedko authored Feb 16, 2024
1 parent a798f68 commit 50a0c9b
Show file tree
Hide file tree
Showing 29 changed files with 69 additions and 1 deletion.
3 changes: 3 additions & 0 deletions .github/workflows/changes.yml
Original file line number Diff line number Diff line change
Expand Up @@ -128,6 +128,7 @@ jobs:
# Detects changes that are not specific to integration tests
source:
runs-on: ubuntu-20.04
timeout-minutes: 5
if: ${{ inputs.source }}
outputs:
source: ${{ steps.filter.outputs.source }}
Expand Down Expand Up @@ -195,6 +196,7 @@ jobs:
# Detects changes that are specific to integration tests
int_tests:
runs-on: ubuntu-latest
timeout-minutes: 5
if: ${{ inputs.int_tests }}
outputs:
all-tests: ${{ steps.filter.outputs.all-tests}}
Expand Down Expand Up @@ -252,6 +254,7 @@ jobs:
# Detects changes that are specific to e2e tests
e2e_tests:
runs-on: ubuntu-latest
timeout-minutes: 5
if: ${{ inputs.e2e_tests }}
outputs:
all-tests: ${{ steps.filter.outputs.all-tests}}
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/cli.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ on:
jobs:
test-cli:
runs-on: ubuntu-latest
timeout-minutes: 15
env:
CARGO_INCREMENTAL: 0
steps:
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/comment-trigger.yml
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,7 @@ jobs:
validate:
name: Validate comment
runs-on: ubuntu-latest
timeout-minutes: 5
if: |
github.event.issue.pull_request && ( contains(github.event.comment.body, '/ci-run-all')
|| contains(github.event.comment.body, '/ci-run-cli')
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/create_preview_sites.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ on:
jobs:
create_preview_site:
runs-on: ubuntu-latest
timeout-minutes: 5
steps:

# Get the artifacts with the PR number and branch name
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/cross.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ jobs:
cross-linux:
name: Cross - ${{ matrix.target }}
runs-on: ubuntu-latest
timeout-minutes: 30
env:
CARGO_INCREMENTAL: 0
strategy:
Expand Down Expand Up @@ -79,6 +80,7 @@ jobs:
update-pr-status:
name: (PR comment) Signal result to PR
runs-on: ubuntu-20.04
timeout-minutes: 5
needs: cross-linux
if: needs.cross-linux.result == 'success' && github.event_name == 'issue_comment'
steps:
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/deny.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ on:
jobs:
test-deny:
runs-on: ubuntu-latest
timeout-minutes: 15
env:
CARGO_INCREMENTAL: 0
steps:
Expand Down
3 changes: 2 additions & 1 deletion .github/workflows/e2e.yml
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,7 @@ jobs:
e2e-tests:
name: E2E Tests
runs-on: [linux, ubuntu-20.04-8core]
timeout-minutes: 45
needs: changes
if: always() && (
github.event_name == 'schedule' || (
Expand All @@ -58,7 +59,6 @@ jobs:
|| needs.changes.outputs.e2e-datadog-metrics == 'true'
)
)
timeout-minutes: 45
steps:
- uses: actions/checkout@v3
with:
Expand Down Expand Up @@ -101,6 +101,7 @@ jobs:
e2e-test-suite:
name: E2E Test Suite
runs-on: ubuntu-latest
timeout-minutes: 5
if: always()
needs: e2e-tests
env:
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/environment.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ env:
jobs:
publish-new-environment:
runs-on: ubuntu-20.04
timeout-minutes: 15
steps:
- name: (PR comment) Get PR branch
if: ${{ github.event_name == 'issue_comment' }}
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/gardener_issue_comment.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ jobs:
move-to-backlog:
name: Move issues back to Gardener project board Triage
runs-on: ubuntu-latest
timeout-minutes: 5
if: ${{ !github.event.issue.pull_request }}
steps:
- name: Generate authentication token
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/gardener_open_issue.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ jobs:
add-to-project:
name: Add issue to Gardener project board
runs-on: ubuntu-latest
timeout-minutes: 5
steps:
- uses: actions/[email protected]
with:
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/gardener_open_pr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ jobs:
add-contributor-to-project:
name: Add contributor PR to Gardener project board
runs-on: ubuntu-latest
timeout-minutes: 5
if: ${{ github.actor != 'dependabot[bot]' }}
steps:
- name: Generate authentication token
Expand All @@ -33,6 +34,7 @@ jobs:
add-dependabot-to-project:
name: Add dependabot PR to Gardener project board
runs-on: ubuntu-latest
timeout-minutes: 5
if: ${{ github.actor == 'dependabot[bot]' }}
steps:
- uses: actions/[email protected]
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/gardener_remove_waiting_author.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ on:
jobs:
remove_label:
runs-on: ubuntu-latest
timeout-minutes: 5
steps:
- uses: actions/checkout@v3
- uses: actions-ecosystem/action-remove-labels@v1
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/install-sh.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ jobs:

sync-install:
runs-on: ubuntu-20.04
timeout-minutes: 10
steps:
- name: (PR comment) Get PR branch
if: ${{ github.event_name == 'issue_comment' }}
Expand Down Expand Up @@ -51,6 +52,7 @@ jobs:
test-install:
needs: sync-install
runs-on: ubuntu-20.04
timeout-minutes: 5
steps:
- run: sudo apt-get install --yes curl bc
- run: curl --proto '=https' --tlsv1.2 -sSfL https://sh.vector.dev | bash -s -- -y
Expand Down
4 changes: 4 additions & 0 deletions .github/workflows/integration-comment.yml
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,7 @@ jobs:
prep-pr:
name: (PR comment) Signal pending to PR
runs-on: ubuntu-latest
timeout-minutes: 5
if: contains(github.event.comment.body, '/ci-run-integration') || contains(github.event.comment.body, '/ci-run-all')
steps:
- name: Generate authentication token
Expand Down Expand Up @@ -82,6 +83,7 @@ jobs:
integration-tests:
needs: prep-pr
runs-on: [linux, ubuntu-20.04-4core]
timeout-minutes: 90
steps:
- uses: actions/checkout@v3
with:
Expand Down Expand Up @@ -459,6 +461,7 @@ jobs:
e2e-tests:
needs: prep-pr
runs-on: [linux, ubuntu-20.04-8core]
timeout-minutes: 30
steps:
- uses: actions/checkout@v3
with:
Expand Down Expand Up @@ -490,6 +493,7 @@ jobs:
update-pr-status:
name: Signal result to PR
runs-on: ubuntu-latest
timeout-minutes: 5
needs:
- integration-tests
- e2e-tests
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/integration.yml
Original file line number Diff line number Diff line change
Expand Up @@ -414,6 +414,7 @@ jobs:
integration-test-suite:
name: Integration Test Suite
runs-on: ubuntu-latest
timeout-minutes: 5
if: always()
needs:
- changes
Expand Down
4 changes: 4 additions & 0 deletions .github/workflows/k8s_e2e.yml
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,7 @@ jobs:
build-x86_64-unknown-linux-gnu:
name: Build - x86_64-unknown-linux-gnu
runs-on: [linux, ubuntu-20.04-4core]
timeout-minutes: 45
needs: changes
# Run this job even if `changes` job is skipped (non- pull request trigger)
if: ${{ !failure() && !cancelled() && (github.event_name != 'pull_request' || needs.changes.outputs.k8s == 'true') }}
Expand Down Expand Up @@ -126,6 +127,7 @@ jobs:
compute-k8s-test-plan:
name: Compute K8s test plan
runs-on: ubuntu-latest
timeout-minutes: 5
needs: changes
# Run this job even if `changes` job is skipped
if: ${{ !failure() && !cancelled() && (github.event_name != 'pull_request' || needs.changes.outputs.k8s == 'true') }}
Expand Down Expand Up @@ -180,6 +182,7 @@ jobs:
test-e2e-kubernetes:
name: K8s ${{ matrix.kubernetes_version.version }} / ${{ matrix.container_runtime }} (${{ matrix.kubernetes_version.role }})
runs-on: [linux, ubuntu-20.04-4core]
timeout-minutes: 45
needs:
- build-x86_64-unknown-linux-gnu
- compute-k8s-test-plan
Expand Down Expand Up @@ -233,6 +236,7 @@ jobs:
final-result:
name: K8s E2E Suite
runs-on: ubuntu-latest
timeout-minutes: 5
needs:
- changes
- build-x86_64-unknown-linux-gnu
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/labeler.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ on:
jobs:
label:
runs-on: ubuntu-20.04
timeout-minutes: 5
permissions:
contents: read
pull-requests: write
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/master_merge_queue.yml
Original file line number Diff line number Diff line change
Expand Up @@ -105,6 +105,7 @@ jobs:
# Always run this so that pull_request triggers are marked as success.
if: always()
runs-on: ubuntu-20.04
timeout-minutes: 5
needs:
- changes
- test-cli
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/misc.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ on:
jobs:
test-misc:
runs-on: [linux, ubuntu-20.04-4core]
timeout-minutes: 45
env:
CARGO_INCREMENTAL: 0
steps:
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/msrv.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ env:
jobs:
check-msrv:
runs-on: ubuntu-latest
timeout-minutes: 20
steps:
- uses: actions/checkout@v3
- run: sudo -E bash scripts/environment/bootstrap-ubuntu-20.04.sh
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/preview_site_trigger.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ on:
jobs:
approval_check:
runs-on: ubuntu-latest
timeout-minutes: 5
if: ${{ ! contains(github.head_ref, 'dependabot*') && ! contains(github.head_ref, 'gh-readonly-queue*') }}
steps:
- name: Echo approval
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/protobuf.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ concurrency:
jobs:
validate-protos:
runs-on: ubuntu-latest
timeout-minutes: 5
steps:
# Run `git checkout`
- uses: actions/checkout@v3
Expand Down
Loading

0 comments on commit 50a0c9b

Please sign in to comment.