Skip to content

feat: prepare for epoch boundary reorg - #9915

Merged
nflaig merged 1 commit into
unstablefrom
te/prepare_epoch_boundary_reorg
Aug 26, 2026
Merged

nflaig merged 1 commit into
unstablefrom
te/prepare_epoch_boundary_reorg

Conversation

@twoeths

@twoeths twoeths commented Aug 26, 2026

Copy link
Copy Markdown
Member

Motivation

  • starting Jun, we see more epoch transition when processing epoch boundary block, which correlate to lighthouse's release of v8.2.0, where it enabled epoch-boundary reorg by default
  • we also did that via feat: allow proposer reorgs at epoch boundaries #9769, Prysm enabled it but it was not in their release yet

Description

  • prepare for epoch-boundary reorg if we find weak head at the last slot of an epoch

Closes #9843

AI Assistance Disclosure

  • created with the help of Claude

@github-actions

This comment was marked as resolved.

@twoeths
twoeths marked this pull request as ready for review August 26, 2026 06:31
@twoeths
twoeths requested a review from a team as a code owner August 26, 2026 06:31
@codecov

codecov Bot commented Aug 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 52.49%. Comparing base (9572bcc) to head (5e8b560).

Additional details and impacted files
@@            Coverage Diff            @@
##           unstable    #9915   +/-   ##
=========================================
  Coverage     52.49%   52.49%           
=========================================
  Files           848      848           
  Lines         59573    59571    -2     
  Branches       4366     4365    -1     
=========================================
- Hits          31274    31273    -1     
+ Misses        28240    28239    -1     
  Partials         59       59           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@nflaig
nflaig merged commit aa8303e into unstable Aug 26, 2026
22 of 24 checks passed
@nflaig
nflaig deleted the te/prepare_epoch_boundary_reorg branch August 26, 2026 08:25
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Lodestar Team Coordination Aug 26, 2026
nflaig pushed a commit that referenced this pull request Aug 31, 2026
**Motivation**

- previously, we didn't allow reorg bid over epoch transition in #9756,
with the assumption that we don't do epoch boundary reorg
- now we switch to allow proposer boost reorg over epoch boundary, we
even prepare for it at epoch boundary in #9915
- see
#9741 (comment)

**Description**

- change bid gossip validation logic of `isBidCompatibleWithHead()` to
allow epoch boundary reorg if it's the same as what we predict
- with this, we still do 1 epoch transition per epoch if what we predict
is correct
@wemeetagain

Copy link
Copy Markdown
Member

🎉 This PR is included in v1.47.0 🎉

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

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Epoch boundary reorg preparation

3 participants