From 2f14efac245f8fd6e6d4ba71836156fc778a420f Mon Sep 17 00:00:00 2001 From: teor Date: Thu, 19 Oct 2023 22:30:53 +1000 Subject: [PATCH] fix(ci): Add missing timeouts to CI and CD tests (#7730) * Add missing timeouts to CI and CD tests * Remove timeout keys that are not allowed by the syntax --- .github/workflows/ci-build-crates.yml | 1 + .github/workflows/ci-unit-tests-docker.yml | 4 ++++ 2 files changed, 5 insertions(+) diff --git a/.github/workflows/ci-build-crates.yml b/.github/workflows/ci-build-crates.yml index e12f6031..8adf9e08 100644 --- a/.github/workflows/ci-build-crates.yml +++ b/.github/workflows/ci-build-crates.yml @@ -99,6 +99,7 @@ jobs: build: name: Build ${{ matrix.crate }} crate + timeout-minutes: 90 needs: [ matrix, check-matrix ] runs-on: ubuntu-latest strategy: diff --git a/.github/workflows/ci-unit-tests-docker.yml b/.github/workflows/ci-unit-tests-docker.yml index fcd8e74d..bf15ce55 100644 --- a/.github/workflows/ci-unit-tests-docker.yml +++ b/.github/workflows/ci-unit-tests-docker.yml @@ -91,6 +91,7 @@ jobs: # TODO: turn this test and the getblocktemplate test into a matrix, so the jobs use exactly the same diagnostics settings test-all: name: Test all + timeout-minutes: 180 runs-on: ubuntu-latest-xl needs: build steps: @@ -144,6 +145,7 @@ jobs: # (The gRPC feature is a zebrad feature, so it isn't needed here.) test-fake-activation-heights: name: Test with fake activation heights + timeout-minutes: 60 runs-on: ubuntu-latest needs: build steps: @@ -167,6 +169,7 @@ jobs: # (We activate the gRPC feature to avoid recompiling `zebrad`, but we don't actually run any gRPC tests.) test-empty-sync: name: Test checkpoint sync from empty state + timeout-minutes: 60 runs-on: ubuntu-latest needs: build steps: @@ -189,6 +192,7 @@ jobs: # (We activate the gRPC feature to avoid recompiling `zebrad`, but we don't actually run any gRPC tests.) test-lightwalletd-integration: name: Test integration with lightwalletd + timeout-minutes: 60 runs-on: ubuntu-latest needs: build steps: