Skip to content

fix: bound beacon runtime and sync work - #541

Open
lodekeeper-z wants to merge 3 commits into
ChainSafe:feat/beacon-nodefrom
lodekeeper-z:fix/beacon-node-runtime-liveness
Open

fix: bound beacon runtime and sync work#541
lodekeeper-z wants to merge 3 commits into
ChainSafe:feat/beacon-nodefrom
lodekeeper-z:fix/beacon-node-runtime-liveness

Conversation

@lodekeeper-z

@lodekeeper-z lodekeeper-z commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Summary

  • subscribe to data-column subnets derived from the node's actual custody columns
  • bound req/resp protocol opens and the complete response transfer so slow peers cannot monopolize the P2P loop
  • apply BLS dispatch gates to recovered unknown-block gossip work
  • preserve peer ownership across in-flight sync downloads and keep validation/rewind paths within the pending batch window
  • exclude the temporary QUIC/gossipsub tracing dependency, debug metrics, trace scripts, and investigation-only artifacts

Review notes

  • Response timeouts use one absolute deadline for the complete multi-chunk transfer, matching Lodestar-TS respTimeoutMs semantics rather than resetting per chunk.
  • Sync retry dispatch remains capped after a long awaiting-validation prefix is rewound.
  • Batch.startDownload is transactional under allocator failure: no state, generation, metric, or callback changes occur unless peer ownership succeeds.

Use actual custody-column subnets and bound protocol opens and complete response transfers so slow peers cannot monopolize the cooperative P2P loop.
Apply aggregate and attestation BLS dispatch gates to recovered unknown-block work as well as ordinary gossip work.
Own peer IDs across in-flight downloads, keep awaiting-validation prefixes from blocking progress, and cap retries after rewinds.
@spiral-ladder

Copy link
Copy Markdown
Member

you're alive!

@lodekeeper-z

Copy link
Copy Markdown
Contributor Author

Alive and catching up 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

2 participants