From 24f11a1136c09ec46e0e86b377b6479f886c7087 Mon Sep 17 00:00:00 2001 From: Marko Atanasievski Date: Wed, 2 Oct 2024 13:34:03 +0200 Subject: [PATCH] fix: add timeouts to ci tests (#687) --- .github/workflows/audit.yml | 1 + .github/workflows/book.yml | 1 + .github/workflows/ci.yml | 14 +++++++------- .github/workflows/docker_build.yml | 1 + .github/workflows/docker_build_push.yml | 1 + .github/workflows/jerigon-native.yml | 2 +- .github/workflows/jerigon-zero.yml | 2 +- .github/workflows/labeler.yml | 1 + .github/workflows/lint.yml | 5 +++++ 9 files changed, 19 insertions(+), 9 deletions(-) diff --git a/.github/workflows/audit.yml b/.github/workflows/audit.yml index d8866a675..ee1716de7 100644 --- a/.github/workflows/audit.yml +++ b/.github/workflows/audit.yml @@ -6,6 +6,7 @@ on: jobs: security_audit: runs-on: ubuntu-latest + timeout-minutes: 10 steps: - uses: actions/checkout@v3 - uses: rustsec/audit-check@v1.4.1 diff --git a/.github/workflows/book.yml b/.github/workflows/book.yml index aef229b3f..c5df23222 100644 --- a/.github/workflows/book.yml +++ b/.github/workflows/book.yml @@ -10,6 +10,7 @@ on: jobs: build: runs-on: ubuntu-latest + timeout-minutes: 10 steps: - uses: actions/checkout@v3 - uses: actions-rs/toolchain@v1 diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index b14656fa0..09280fae2 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -45,7 +45,7 @@ jobs: test_trace_decoder: name: Test trace_decoder runs-on: zero-ci - timeout-minutes: 30 + timeout-minutes: 10 env: RUST_LOG: info CARGO_INCREMENTAL: 1 @@ -72,7 +72,7 @@ jobs: test_evm_arithmetization: name: Test evm_arithmetization runs-on: ubuntu-latest - timeout-minutes: 30 + timeout-minutes: 20 if: ${{ ! contains(toJSON(github.event.commits.*.message), '[skip-ci]') }} steps: - name: Checkout sources @@ -96,7 +96,7 @@ jobs: test_zero_bin: name: Test zero_bin runs-on: ubuntu-latest - timeout-minutes: 30 + timeout-minutes: 5 if: ${{ ! contains(toJSON(github.event.commits.*.message), '[skip-ci]') }} steps: - name: Checkout sources @@ -122,7 +122,7 @@ jobs: test_zk_evm_proc_macro: name: Test zk_evm_proc_macro runs-on: ubuntu-latest - timeout-minutes: 30 + timeout-minutes: 5 if: ${{ ! contains(toJSON(github.event.commits.*.message), '[skip-ci]') }} steps: - name: Checkout sources @@ -146,7 +146,7 @@ jobs: simple_proof_regular: name: Execute bash script to generate and verify a proof for a small block. runs-on: zero-ci - + timeout-minutes: 15 steps: - name: Checkout code uses: actions/checkout@v3 @@ -157,7 +157,7 @@ jobs: simple_proof_witness_only: name: Execute bash script to generate the proof witness for a small block. runs-on: zero-ci - + timeout-minutes: 5 steps: - name: Checkout code uses: actions/checkout@v3 @@ -168,7 +168,7 @@ jobs: multi_blocks_proof_regular: name: Execute bash script to generate and verify a proof for multiple blocks using parallel proving. runs-on: zero-ci - + timeout-minutes: 10 steps: - name: Checkout code uses: actions/checkout@v3 diff --git a/.github/workflows/docker_build.yml b/.github/workflows/docker_build.yml index c907bbf39..1d24cc8b7 100644 --- a/.github/workflows/docker_build.yml +++ b/.github/workflows/docker_build.yml @@ -14,6 +14,7 @@ jobs: docker: name: Regression test docker images runs-on: ubuntu-latest + timeout-minutes: 10 steps: - name: Checkout uses: actions/checkout@v4 diff --git a/.github/workflows/docker_build_push.yml b/.github/workflows/docker_build_push.yml index 0c64efecd..37b89fa92 100644 --- a/.github/workflows/docker_build_push.yml +++ b/.github/workflows/docker_build_push.yml @@ -13,6 +13,7 @@ jobs: docker: name: Build and push leader and worker docker images to GitHub Container Registry runs-on: ubuntu-latest + timeout-minutes: 30 permissions: packages: write steps: diff --git a/.github/workflows/jerigon-native.yml b/.github/workflows/jerigon-native.yml index abb9b2a2c..29a380c3a 100644 --- a/.github/workflows/jerigon-native.yml +++ b/.github/workflows/jerigon-native.yml @@ -18,7 +18,7 @@ jobs: test_jerigon_native_tracer_proving: name: Native tracer proof generation runs-on: zero-ci - timeout-minutes: 40 + timeout-minutes: 30 if: "! contains(toJSON(github.event.commits.*.message), '[skip-ci]')" steps: - name: Checkout sources diff --git a/.github/workflows/jerigon-zero.yml b/.github/workflows/jerigon-zero.yml index c9e23b7bb..216b32f8c 100644 --- a/.github/workflows/jerigon-zero.yml +++ b/.github/workflows/jerigon-zero.yml @@ -19,7 +19,7 @@ jobs: test_jerigon_zero_tracer_proving: name: Zero tracer proof generation runs-on: zero-ci - timeout-minutes: 40 + timeout-minutes: 30 if: "! contains(toJSON(github.event.commits.*.message), '[skip-ci]')" steps: - name: Checkout sources diff --git a/.github/workflows/labeler.yml b/.github/workflows/labeler.yml index 4ad279fec..ee79f0f56 100644 --- a/.github/workflows/labeler.yml +++ b/.github/workflows/labeler.yml @@ -8,6 +8,7 @@ jobs: contents: read pull-requests: write runs-on: ubuntu-latest + timeout-minutes: 10 steps: - uses: actions/labeler@v5 with: diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml index 3cdee27ed..446d6a314 100644 --- a/.github/workflows/lint.yml +++ b/.github/workflows/lint.yml @@ -19,6 +19,7 @@ env: jobs: clippy: runs-on: ubuntu-latest + timeout-minutes: 10 steps: - uses: actions/checkout@v3 - uses: ./.github/actions/rust @@ -26,6 +27,7 @@ jobs: - run: cargo clippy --all-targets --no-default-features --features cdk_erigon -- -D warnings udeps: runs-on: ubuntu-latest + timeout-minutes: 5 steps: - uses: actions/checkout@v3 - uses: ./.github/actions/rust @@ -36,18 +38,21 @@ jobs: - run: cargo udeps --quiet --tests --benches --examples rustdoc: runs-on: ubuntu-latest + timeout-minutes: 5 steps: - uses: actions/checkout@v3 - uses: ./.github/actions/rust - run: cargo doc --all --no-deps cargo-fmt: runs-on: ubuntu-latest + timeout-minutes: 5 steps: - uses: actions/checkout@v3 - uses: ./.github/actions/rust - run: cargo fmt --check taplo-fmt: runs-on: ubuntu-latest + timeout-minutes: 5 steps: - uses: actions/checkout@v3 - uses: ./.github/actions/rust