try to fix deadlock/timeout in logpoller tests#22337
Conversation
|
I see you updated files related to
|
|
✅ No conflicts with other open PRs targeting |
272723f to
95e0ddb
Compare
There was a problem hiding this comment.
Pull request overview
Risk Rating: MEDIUM
This PR updates the repository’s github.com/smartcontractkit/chainlink-evm dependency to pick up an upstream fix intended to reduce logpoller-related test deadlocks/timeouts, and regenerates generated artifacts impacted by that bump (notably config docs).
Changes:
- Bump
github.com/smartcontractkit/chainlink-evmtov0.3.4-0.20260507171202-46e6a397da2dacross root and submodulego.modfiles (and correspondinggo.sumupdates). - Regenerate
docs/CONFIG.md, which (notably) changes Mantle chain defaultGasEstimator.LimitDefault/LimitMaxvalues.
Scrupulous human review needed:
- Verify the
chainlink-evmbump actually contains the intended goroutine shutdown / deadlock fix (since the core logic change is upstream and not visible in this diff). - Confirm the Mantle Mainnet/Sepolia default gas estimator limit changes in
docs/CONFIG.mdare intentional and acceptable, and update the PR description accordingly (see comments).
Reviewed changes
Copilot reviewed 8 out of 15 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| system-tests/tests/go.mod | Updates chainlink-evm version for system test module. |
| system-tests/tests/go.sum | Updates checksums for new chainlink-evm version. |
| system-tests/lib/go.mod | Updates chainlink-evm version for system test library module. |
| system-tests/lib/go.sum | Updates checksums for new chainlink-evm version. |
| integration-tests/load/go.mod | Updates chainlink-evm version for load test module. |
| integration-tests/load/go.sum | Updates checksums for new chainlink-evm version. |
| integration-tests/go.mod | Updates chainlink-evm version for integration test module. |
| integration-tests/go.sum | Updates checksums for new chainlink-evm version. |
| deployment/go.mod | Updates chainlink-evm version for deployment module. |
| deployment/go.sum | Updates checksums for new chainlink-evm version. |
| core/scripts/go.mod | Updates chainlink-evm version for scripts module. |
| core/scripts/go.sum | Updates checksums for new chainlink-evm version. |
| go.mod | Updates root module chainlink-evm requirement. |
| go.sum | Updates root module checksums for new chainlink-evm version. |
| docs/CONFIG.md | Regenerated config docs; changes Mantle chain gas estimator limit defaults. |
|





15 successful runs: https://github.com/smartcontractkit/chainlink/actions/runs/25488891367 ✅
Requires: smartcontractkit/chainlink-evm#463 (merged ✅)
Made with AI and explained as:
Hoping to solve deadlocks like this one, which are endemic in
chainlinkrepo:or