From a3d5058b1d1c902b48d0064133edc1fff4376c1e Mon Sep 17 00:00:00 2001 From: LJ <81748770+elle-j@users.noreply.github.com> Date: Wed, 27 May 2026 16:15:21 +0200 Subject: [PATCH] Configure concurrency for e2e tests to fix flakiness (#530) We experienced flakiness in the e2e tests after bumping the revive-differential-tests submodule which included changes to the global RPC concurrency limiter and batching layer. This PR attempts to fix the flakiness by lowering the number of concurrent tasks spawned. --- .github/workflows/differential-tests.yml | 3 +++ revive-differential-tests | 2 +- 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/.github/workflows/differential-tests.yml b/.github/workflows/differential-tests.yml index 9a82f72..d7cfbc9 100644 --- a/.github/workflows/differential-tests.yml +++ b/.github/workflows/differential-tests.yml @@ -164,6 +164,9 @@ jobs: resolc-path: ./resolc-bin/resolc-x86_64-unknown-linux-musl solc-version: ${{ env.SOLC_VERSION }} expectations-file-path: ./revive/.github/assets/revive-dev-node-polkavm-resolc.json + # Using a higher concurrency, such as 100, currently overloads the dev node's RPC + # bridge, causing intermittent `eth_getTransactionReceipt` timeouts during code upload. + concurrency-number-of-concurrent-tasks: 50 # Sentinel job that depends on the jobs we want to allow completing (instead of # being canceled once the other job chains complete). Thus, run-e2e-tests diff --git a/revive-differential-tests b/revive-differential-tests index c0094e6..e544a18 160000 --- a/revive-differential-tests +++ b/revive-differential-tests @@ -1 +1 @@ -Subproject commit c0094e62ddcde0115dbaf157bd1588665c22890f +Subproject commit e544a189da379f5c60858f2eee46947c916629d8