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

EIP-7251: Increase the MAX_EFFECTIVE_BALANCE #502

Closed
emhane opened this issue Jan 15, 2025 · 2 comments · Fixed by #554
Closed

EIP-7251: Increase the MAX_EFFECTIVE_BALANCE #502

emhane opened this issue Jan 15, 2025 · 2 comments · Fixed by #554
Assignees
@refcell
Labels
A-execution Area: execution layer H-isthmus Hardfork: change is planned for Isthmus upgrade H-pectra Hardfork: change planned for Pectra (L1) upgrade U-node Upgrade: involving changes to node component (cl, el, etc.) U-noop Upgrade: L1 feature not used on L2 requires noop spec to disable

Comments

@emhane
Copy link
Member

emhane commented Jan 15, 2025 •
edited by refcell
Loading

https://eips.ethereum.org/EIPS/eip-7251

References
@emhane emhane added H-pectra Hardfork: change planned for Pectra (L1) upgrade S-wontfix Status: issue won't be fixed as result of deliberate design decision and removed S-wontfix Status: issue won't be fixed as result of deliberate design decision labels Jan 15, 2025
@emhane
Copy link
Member Author

emhane commented Jan 15, 2025

@protolambda is there anything to do here?

@protolambda
Copy link
Contributor

See comment on #505, very similar reasoning applies here. Let's change the block-sealing code to not add requests of type CONSOLIDATION_REQUEST_TYPE (and for other EIPs, see their respective request type), if the optimism config is active. (Doesn't necessarily need a specific fork check, since the request shouldn't exist pre-pectra).

@emhane emhane mentioned this issue Jan 17, 2025
Closed
@emhane emhane added H-isthmus Hardfork: change is planned for Isthmus upgrade A-execution Area: execution layer labels Jan 17, 2025
@emhane emhane mentioned this issue Jan 17, 2025
Open
@emhane emhane added U-node Upgrade: involving changes to node component (cl, el, etc.) U-noop Upgrade: L1 feature not used on L2 requires noop spec to disable labels Jan 17, 2025
@refcell refcell self-assigned this Jan 27, 2025
@refcell refcell mentioned this issue Jan 27, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@refcell refcell

Labels
A-execution Area: execution layer H-isthmus Hardfork: change is planned for Isthmus upgrade H-pectra Hardfork: change planned for Pectra (L1) upgrade U-node Upgrade: involving changes to node component (cl, el, etc.) U-noop Upgrade: L1 feature not used on L2 requires noop spec to disable
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Isthmus: EIP-7251 ethereum-optimism/specs
3 participants
@protolambda @refcell @emhane