Skip to content

Reduce min epochs for block requests in Gloas - #5680

Merged
jtraglia merged 9 commits into
ethereum:masterfrom
nflaig:nflaig/gloas-min-epochs-for-block-requests
Sep 28, 2026
Merged

jtraglia merged 9 commits into
ethereum:masterfrom
nflaig:nflaig/gloas-min-epochs-for-block-requests

Conversation

@nflaig

@nflaig nflaig commented Sep 25, 2026

Copy link
Copy Markdown
Member

compute_min_epochs_for_block_requests() is derived from compute_weak_subjectivity_period for a very large validator set at maximal safety decay (#2131). EIP-8061 changed the churn limits and updated compute_weak_subjectivity_period (#5061), which roughly halves the weak subjectivity period (#5629), but compute_min_epochs_for_block_requests() was not updated and still returns 33024 epochs based on the Phase 0 CHURN_LIMIT_QUOTIENT.

This applies the same change, which gives 14299 epochs (~2 months) on mainnet and 262 on minimal. It is a relaxation, clients can keep serving a longer range.

Comment thread specs/gloas/p2p-interface.md Outdated
Co-authored-by: Justin Traglia <95511699+jtraglia@users.noreply.github.com>

@jihoonsong jihoonsong left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

IMO, this can be merged in and clients can implement this at their preferences, e.g. before or after Gloas.

Comment thread specs/gloas/p2p-interface.md Outdated
Co-authored-by: Jihoon Song <jihoonsong@users.noreply.github.com>
@jtraglia

Copy link
Copy Markdown
Member

At ACDT today, we agreed to merge this PR today. As Jihoon said above, clients do not need to make this change immediately. Since this just lowers the minimum, clients remain spec-compliant without any changes.

@jtraglia
jtraglia enabled auto-merge (squash) September 28, 2026 20:29
@jtraglia
jtraglia merged commit e321975 into ethereum:master Sep 28, 2026
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gloas testing CI, actions, tests, testing infra

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants