Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add EIP: Add blob schedule to EL config files #9129

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Add EIP: Add blob schedule to EL config files #9129

wants to merge 6 commits into from
+70 −0

Conversation

lightclient
Copy link
Member

Adds a new object to the EL genesis chain config:

of the shape:

"blobSchedule": {
  "prague": { "target": 6, "max": 9 },
  "osaka":   { "target": 12, "max": 16},
  ...
}

@lightclient lightclient requested a review from eth-bot as a code owner December 12, 2024 15:17
@github-actions github-actions bot added c-new Creates a brand new proposal s-draft This EIP is a Draft t-core labels Dec 12, 2024
@eth-bot
Copy link
Collaborator

eth-bot commented Dec 12, 2024 •
edited
Loading

File EIPS/eip-7840.md

Requires 1 more reviewers from @g11tech, @SamWilsn

@eth-bot eth-bot added the e-consensus Waiting on editor consensus label Dec 12, 2024
@github-actions github-actions bot added the w-ci Waiting on CI to pass label Dec 12, 2024
abcoathup
abcoathup reviewed Dec 12, 2024
View reviewed changes
EIPS/blob-config.md Outdated Show resolved Hide resolved
abcoathup
abcoathup reviewed Dec 13, 2024
View reviewed changes
EIPS/blob-config.md Outdated Show resolved Hide resolved
@timbeiko timbeiko mentioned this pull request Dec 13, 2024
Open
@eth-bot eth-bot added the e-review Waiting on editor to review label Dec 13, 2024
@eth-bot eth-bot changed the title Add blob config eip Add EIP: Add blob schedule to execution client configuration files Dec 13, 2024
@github-actions github-actions bot added w-ci Waiting on CI to pass and removed w-ci Waiting on CI to pass labels Dec 13, 2024
@eth-bot eth-bot changed the title Add EIP: Add blob schedule to execution client configuration files Add EIP: Add blob schedule to EL client configuration files Dec 13, 2024
@github-actions github-actions bot removed the w-ci Waiting on CI to pass label Dec 13, 2024
@github-actions GitHub Actions
Copy link

The commit 16a6427 (as a parent of 6586687) contains errors.
Please inspect the Run Summary for details.

@github-actions github-actions bot added the w-ci Waiting on CI to pass label Dec 13, 2024
@jflo jflo mentioned this pull request Dec 13, 2024
Open
@eth-bot eth-bot changed the title Add EIP: Add blob schedule to EL client configuration files Add EIP: Add blob schedule to EL config files Dec 13, 2024
@github-actions github-actions bot removed the w-ci Waiting on CI to pass label Dec 13, 2024
g11tech
g11tech reviewed Dec 16, 2024
View reviewed changes
EIPS/eip-7840.md Outdated Show resolved Hide resolved
g11tech
g11tech reviewed Dec 16, 2024
View reviewed changes
EIPS/eip-7840.md

## Test Cases

TODO
Copy link
Contributor

Choose a reason for hiding this comment

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

<-- TODO --> tag for tracking via bot

g11tech
g11tech reviewed Dec 16, 2024
View reviewed changes
EIPS/eip-7840.md

## Security Considerations

Needs discussion.
Copy link
Contributor

Choose a reason for hiding this comment

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

<-- TODO --> tag

jochem-brouwer
jochem-brouwer reviewed Dec 16, 2024
View reviewed changes
EIPS/eip-7840.md Outdated Show resolved Hide resolved
MarekM25
MarekM25 reviewed Dec 16, 2024
View reviewed changes
Copy link
Contributor

@MarekM25 MarekM25 left a comment

Choose a reason for hiding this comment

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

Nethermind team is okay with this change. Signaling support 👍

@siladu siladu mentioned this pull request Dec 17, 2024
Open
8 tasks
@siladu
Copy link

siladu commented Dec 17, 2024

Besu is ok with this change to get us through Prague and have a prototype implementation ready hyperledger/besu#8042

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

@g11tech g11tech g11tech left review comments

@jochem-brouwer jochem-brouwer jochem-brouwer left review comments

@abcoathup abcoathup abcoathup left review comments

@MarekM25 MarekM25 MarekM25 left review comments

@eth-bot eth-bot Awaiting requested review from eth-bot eth-bot is a code owner

At least 0 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
c-new Creates a brand new proposal e-consensus Waiting on editor consensus e-review Waiting on editor to review s-draft This EIP is a Draft t-core
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@lightclient @eth-bot @siladu @MarekM25 @abcoathup @jochem-brouwer @g11tech