Feat/eip8025 networking improvements - #21
Merged
frisitano merged 12 commits intoMar 30, 2026
Merged
Conversation
Fixes a runtime panic where clap stored proof-types values as String but get_many::<u8> expected u8, causing a TypeId mismatch. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Add `map_client_config` to `TestNetworkFixtureBuilder` as a generic escape hatch for transforming the `ClientConfig` before nodes are created, following the existing `map_spec`/`map_network_params` pattern. Use it in `base_builder()` to set `proof_sync_activation_slots = 0`. The default of 10 slots is intended for production (to let the beacon processor drain after range sync), but on a loaded 2-CPU CI runner each effective slot takes ~5-6 s, so the countdown alone consumes 50-60 s of the 60 s test timeout before the first proof request fires. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Proof engine tests spawn multiple beacon nodes and are sensitive to slot timing. Running them inside the parallel release-tests-ubuntu job on a 2-CPU runner causes CPU starvation that makes effective slot time 5-6× slower than real time, exhausting test timeouts. Changes: - Exclude proof_engine_test from test-release (parallel job) - Add test-proof-engine Makefile target with --test-threads 1 - Add proof-engine-tests CI job using a lighter dedicated runner - Add proof-engine-tests to test-suite-success gate Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Issue Addressed
Which issue # does this PR address?
Proposed Changes
Please list or describe the changes introduced by this PR.
Additional Info
Please provide any additional information. For example, future considerations
or information useful for reviewers.