Skip to content

fix(consensus): wait for execution layer to be ready before driving it - #7230

Merged
SuperFluffy merged 15 commits into
mainfrom
centaur/gate-consensus-executor-checkpoints-1787146458
Aug 28, 2026
Merged

fix(consensus): wait for execution layer to be ready before driving it#7230
SuperFluffy merged 15 commits into
mainfrom
centaur/gate-consensus-executor-checkpoints-1787146458

Conversation

@decofe

@decofe decofe commented Aug 19, 2026

Copy link
Copy Markdown
Member

Waits for reth to be ready. When starting from a fresh snapshot, reth will rebuild it indices. The cleanest way to check for readiness is to issue a non-destructive FCU using the latest reported EL state. Once it returns VALID, the node can start interacting with it.

This removes the need to retry finalized blocks because we might have caught reth while it was still rebuilding its indices, and as a consequence returned SYNCING.

@decofe decofe changed the title fix(consensus): gate executor on pipeline sync fix(consensus): gate executor on index rebuild Aug 19, 2026
@decofe
decofe force-pushed the centaur/gate-consensus-executor-checkpoints-1787146458 branch 2 times, most recently from c3bfa01 to 57c0e9e Compare August 25, 2026 13:56
@decofe
decofe changed the base branch from main to centaur/remove-executor-pacing-1787665736 August 25, 2026 13:56
@decofe
decofe force-pushed the centaur/remove-executor-pacing-1787665736 branch from e24ce0e to 6d34fd7 Compare August 25, 2026 14:04
@decofe
decofe force-pushed the centaur/gate-consensus-executor-checkpoints-1787146458 branch from 57c0e9e to 3bae36f Compare August 25, 2026 14:07
@decofe
decofe force-pushed the centaur/remove-executor-pacing-1787665736 branch from 6d34fd7 to c90f992 Compare August 25, 2026 14:16
@decofe
decofe force-pushed the centaur/gate-consensus-executor-checkpoints-1787146458 branch from 3bae36f to 76341fc Compare August 25, 2026 14:18
@SuperFluffy
SuperFluffy force-pushed the centaur/remove-executor-pacing-1787665736 branch 2 times, most recently from 62bae04 to bd09ca3 Compare August 27, 2026 13:04
@decofe
decofe force-pushed the centaur/gate-consensus-executor-checkpoints-1787146458 branch 2 times, most recently from 90d955d to aecf550 Compare August 27, 2026 14:07
Base automatically changed from centaur/remove-executor-pacing-1787665736 to main August 27, 2026 14:13
@decofe
decofe force-pushed the centaur/gate-consensus-executor-checkpoints-1787146458 branch 3 times, most recently from e46b1fe to 44428f7 Compare August 27, 2026 15:00
SuperFluffy and others added 6 commits August 27, 2026 15:34
Co-authored-by: Derek Cofausper <256792747+decofe@users.noreply.github.com>
Co-authored-by: Derek Cofausper <256792747+decofe@users.noreply.github.com>
Co-authored-by: Derek Cofausper <256792747+decofe@users.noreply.github.com>
Co-authored-by: Derek Cofausper <256792747+decofe@users.noreply.github.com>
Co-authored-by: Derek Cofausper <256792747+decofe@users.noreply.github.com>
Co-authored-by: Derek Cofausper <256792747+decofe@users.noreply.github.com>
@decofe
decofe force-pushed the centaur/gate-consensus-executor-checkpoints-1787146458 branch from 44428f7 to 580731c Compare August 27, 2026 15:37
Comment thread crates/consensus/src/executor/actor.rs Outdated
Comment thread crates/consensus/src/executor/mod.rs Outdated
Comment thread crates/consensus/src/executor/actor.rs Outdated
Comment thread crates/consensus/src/executor/actor.rs
@decofe
decofe force-pushed the centaur/gate-consensus-executor-checkpoints-1787146458 branch 2 times, most recently from 5e45b62 to a4cb90d Compare August 27, 2026 15:48
@SuperFluffy SuperFluffy changed the title fix(consensus): gate executor on index rebuild fix(consensus): wait for execution layer to be ready before driving it Aug 28, 2026
@SuperFluffy

Copy link
Copy Markdown
Contributor

cyclops audit fast

@tempoxyz-bot tempoxyz-bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👁️ Cyclops Review — No actionable findings.

@SuperFluffy
SuperFluffy added this pull request to the merge queue Aug 28, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 28, 2026
@SuperFluffy
SuperFluffy added this pull request to the merge queue Aug 28, 2026
Merged via the queue into main with commit d5c6b38 Aug 28, 2026
55 checks passed
@SuperFluffy
SuperFluffy deleted the centaur/gate-consensus-executor-checkpoints-1787146458 branch August 28, 2026 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants