Skip to content

Conversation

@raulk
Copy link
Member

@raulk raulk commented May 25, 2025

This is a minimal changeset on top of the existing BPO hardforks spec to solve some loose ends.

Concretely, this adjusts the following at the consensus layer:

  • Assigns a fork_version to pure BPO forks.
  • Updates compute_fork_version() to take BPO forks into account.
  • Specifies ENR field mapping to account for upcoming BPO forks.
  • Notes that p2p topics will roll over.

Furthermore, this makes a broader adjustment to avoid future ambiguity:

  • We now require that all future blob parameter changes be performed through blob schedule configuration—even those introduced by regular (named) hardforks.

While thinking through this, I realized there may be value in formalizing the broader concept of "parametric forks" by defining how they interact with regular hard forks, establishing a process for bringing additional parameters under this management mechanism, and iterating on client config schemas (https://github.com/eth-clients) to introduce time-based triggers as a top-level organizational axis. Perhaps the subject of a future proposal.

@raulk raulk requested a review from eth-bot as a code owner May 25, 2025 23:16
@github-actions github-actions bot added c-update Modifies an existing proposal s-draft This EIP is a Draft t-informational labels May 25, 2025
@eth-bot
Copy link
Collaborator

eth-bot commented May 25, 2025

File EIPS/eip-7892.md

Requires 1 more reviewers from @ethDreamer

@eth-bot eth-bot added the a-review Waiting on author to review label May 25, 2025
@raulk raulk force-pushed the raulk/harden-bpo branch from a1c8fb6 to 172ea41 Compare May 25, 2025 23:17
@raulk raulk changed the title Update EIP-7892: harden BPO fork spec with p2p details. Update EIP-7892: harden BPO forks spec. May 25, 2025
@raulk raulk force-pushed the raulk/harden-bpo branch from 172ea41 to 0a1fdbd Compare May 25, 2025 23:20
@raulk raulk force-pushed the raulk/harden-bpo branch from 94b9d2d to 9c36137 Compare May 26, 2025 14:59
@raulk
Copy link
Member Author

raulk commented May 28, 2025

Following further discussion on Discord, I posted a synthesis of current consensus/opinions and recommendation here: ethereum/consensus-specs#4331 (comment)

EIPS/eip-7892.md Outdated
```
BLOB_SCHEDULE:
- EPOCH: 348618
A new `BLOB_PARAMETER_SCHEDULE` field is added to consensus layer configuration, containing a sequence of entries representing blob parameter changes after `ELECTRA_FORK_EPOCH`. This replaces the deprecated `MAX_BLOB_PER_BLOCK{_FORKNAME}` convention for future forks, while preserving existing entries for previous forks.

Choose a reason for hiding this comment

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

why are we renaming the blob schedule? we can just do this under blob_schedule still unless we're worried about old parsers breaking or something... The blob_schedule already resulted in objects and we're just adding an attribute imo..

Copy link
Member Author

Choose a reason for hiding this comment

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

This is a leftover from a previous version that backported Deneb and Electra. Fixing.

@raulk raulk force-pushed the raulk/harden-bpo branch from 9df070d to 68a5bd2 Compare May 29, 2025 14:18
@raulk raulk force-pushed the raulk/harden-bpo branch from 68a5bd2 to 34b7d0f Compare May 29, 2025 14:19
@raulk
Copy link
Member Author

raulk commented May 30, 2025

Superseded by #9840.

@raulk raulk closed this May 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a-review Waiting on author to review c-update Modifies an existing proposal s-draft This EIP is a Draft t-informational

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants