Skip to content

consolidation by moving balance upon exit#14

Merged
mkalinin merged 7 commits into
michaelneuder:maxeb-pyspec-min-viablefrom
fradamt:consolidate-through-exit
Jan 24, 2024
Merged

consolidation by moving balance upon exit#14
mkalinin merged 7 commits into
michaelneuder:maxeb-pyspec-min-viablefrom
fradamt:consolidate-through-exit

Conversation

@fradamt
Copy link
Copy Markdown
Collaborator

@fradamt fradamt commented Jan 16, 2024

First attempt at a simpler consolidation design, suggested by @mkalinin, where the Consolidation simply exits the validator and then moves the balance upon exit. No slashing across source and target because churn is applied to the exit, and the moving balance earns rewards throughout, by the source until the exit epoch and then by the target

Comment thread specs/_features/maxeb_increase/capella.py Outdated
Comment thread specs/_features/maxeb_increase/capella.py
Comment thread specs/_features/maxeb_increase/capella.py
Comment thread specs/_features/maxeb_increase/capella.py Outdated
@yokem55
Copy link
Copy Markdown

yokem55 commented Jan 17, 2024

It seems to me that the way to make this work is to restrict exit_transfer exits to a 7002 based exit (a message from the withdrawal address on el), or a message signed by with a 0x00 credential on the beacon chain. The validator key would still be able to do traditional exit messages that simply withdraw to the 0x01 credential, but would not be able to do the transfer side of things.

Comment thread specs/_features/maxeb_increase/capella.py Outdated
Comment thread specs/_features/maxeb_increase/capella.py
Co-authored-by: fradamt <104826920+fradamt@users.noreply.github.com>
Comment thread specs/_features/maxeb_increase/capella.py
mkalinin and others added 3 commits January 23, 2024 19:32
Co-authored-by: fradamt <104826920+fradamt@users.noreply.github.com>
@mkalinin mkalinin merged commit 53582d0 into michaelneuder:maxeb-pyspec-min-viable Jan 24, 2024
fradamt added a commit to fradamt/consensus-specs that referenced this pull request Jan 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants