Skip to content

Commit f816262

Browse files
michaelsproulWoodpile37
authored andcommitted
Fix exec integration tests for Geth v1.11.0 (sigp#3982)
* Bump Go from 1.17 to 1.20. The latest Geth release v1.11.0 requires 1.18 minimum. * Prevent a cache miss during payload building by using the right fee recipient. This prevents Geth v1.11.0 from building a block with 0 transactions. The payload building mechanism is overhauled in the new Geth to improve the payload every 2s, and the tests were failing because we were falling back on a `getPayload` call with no lookahead due to `get_payload_id` cache miss caused by the mismatched fee recipient. Alternatively we could hack the tests to send `proposer_preparation_data`, but I think the static fee recipient is simpler for now. * Add support for optionally enabling Lighthouse logs in the integration tests. Enable using `cargo run --release --features logging/test_logger`. This was very useful for debugging.
1 parent 16ecda8 commit f816262

2 files changed

Lines changed: 2 additions & 1 deletion

File tree

.github/workflows/test-suite.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -280,7 +280,7 @@ jobs:
280280
- uses: actions/checkout@v3
281281
- uses: actions/setup-go@v3
282282
with:
283-
go-version: '1.17'
283+
go-version: '1.20'
284284
- uses: actions/setup-dotnet@v3
285285
with:
286286
dotnet-version: '6.0.201'

Cargo.lock

Lines changed: 1 addition & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)