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.

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

EIP-7002: EL Triggerable Exits #5129

Closed
macladson opened this issue Jan 26, 2024 · 3 comments
Closed

EIP-7002: EL Triggerable Exits #5129

macladson opened this issue Jan 26, 2024 · 3 comments
Labels
electra Required for the Electra/Prague fork spec_change A change related to the Eth2 spec

Comments

@macladson
Copy link
Member

Description

EIP -> https://eips.ethereum.org/EIPS/eip-7002

Was confirmed for Electra in latest ACD consensus call.
This will require some coordination from EL teams in order to do testing, etc.

Implementation

As per the EIP, the flow is:

  • New operation ExecutionLayerExit
  • Will show up in ExecutionPayload as an SSZ List bound by length MAX_EXITS_PER_BLOCK
  • New function in process_execution_layer_exit that has similar functionality to process_voluntary_exit but that can fail validations (e.g. validator is already exited) without the block failing (similar to deposit coming from EL)
  • process_execution_layer_exit called in process_operations for each ExecutionLayerExit found in the ExecutionPayload
@macladson macladson added spec_change A change related to the Eth2 spec electra Required for the Electra/Prague fork labels Jan 26, 2024
@dapplion
Copy link
Collaborator

There was strong agreement for inclusion? At this point is okay to spend dev resources on it for sure, but I would refrain from merging it into unstable/electra branch for at least 2-3 ACDCs

@macladson
Copy link
Member Author

There was strong agreement for inclusion? At this point is okay to spend dev resources on it for sure, but I would refrain from merging it into unstable/electra branch for at least 2-3 ACDCs

For now just going off of this PR/commit. My aim is to keep these issues updated as things change

@realbigsean
Copy link
Member

realbigsean commented Jan 26, 2024

It sounded like EL teams had already OK'd 6110 and 7002 on the last ACDE

but I would refrain from merging it into unstable/electra branch for at least 2-3 ACDCs

Seems reasonable!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
electra Required for the Electra/Prague fork spec_change A change related to the Eth2 spec
Projects
None yet
Development

No branches or pull requests

3 participants