Repository navigation
chore(deps): update Lighthouse to unstable 8b927ced - #1327
Merged
shane-moore merged 1 commit intoSep 29, 2026
Merged
Conversation
Move all 18 Lighthouse dependencies from 73e4dc12 to 8b927cedd. sigp/lighthouse#9675 switched head events to a broadcast channel and lets the sync committee service sign on head events. Subscribe the attestation, sync committee and metadata services directly before the head monitor starts, and drop the Anchor fan-out relay with its drop metric and dashboard panel. The metadata service now uses Lighthouse's head_event_or_deadline instead of its own copy. Committees with a sync committee member no longer hold their attestation partials until the sync message due, which is the Gloas attestation deadline.
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## epbs #1327 +/- ##
=======================================
Coverage ? 80.90%
=======================================
Files ? 179
Lines ? 41238
Branches ? 0
=======================================
Hits ? 33364
Misses ? 7874
Partials ? 0
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
shane-moore
commented
Sep 29, 2026
shane-moore
left a comment
Member
Author
There was a problem hiding this comment.
Review from Claude Opus 5.5 (high effort) at 3ec099e4: no blocking findings.
Checked:
- Wiring matches Lighthouse's own VC at
8b927ced. All three subscriptions happen beforestart_fallback_updater_servicespawns the monitor, so it never sends on a channel with no receivers. - The earlier sync trigger can't sign ahead of consensus. Anchor signs
decided.block_rootafterget_voting_context(slot), not the head-event root Lighthouse now passes in. - The motivation holds statically.
voting_message_count_for_committeecounts sync validators, and the collector only flushes a full batch, so attestation partials wait for the sync partial. - The gain relies on the SlotTime start-gate removal (#1136), which is
epbs-only.unstablestill sleeps to the attestation due before round 1, so it would see little benefit from this bump alone. - The other four Lighthouse commits don't reach Anchor.
GetGraffitiResponsehas no users here, and Lighthouse's rootCargo.toml/Cargo.lockare unchanged between the revs.
Not checked: the ssv-mini timing measurement the PR already lists as owed.
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
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.
Problem, Evidence, and Context (Required)
epbspins Lighthouse73e4dc12, 5 commits behindunstableHEAD8b927cedd.Change Overview (Required)
8b927cedd. The lockfile diff is only those crates' source lines.head_event_or_deadlinein place of its own copy.Risks, Trade-offs, and Mitigations (Required)
BeaconNodeFallback.Cargo.tomlandCargo.lockare unchanged between the two revs, so the mirrored[patch.crates-io]entries stay correct.Validation (Required)
make cargo-fmt-check,make lint,cargo nextest run --release --workspace --no-fail-fast: 994 passed.Rollback (Required for behavior or runtime changes; optional otherwise)
Revert the commit to return to
73e4dc12and the fan-out relay. No config or data impact. Theanchor_head_event_fanout_drops_totalmetric goes away.🤖 Generated with Claude Code