-
Notifications
You must be signed in to change notification settings - Fork 28
Support chain-supplied validator voluntary exits #300
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
Merged
Merged
Changes from all commits
Commits
Show all changes
40 commits
Select commit
Hold shift + click to select a range
3352b0c
Initial implementation
diegomrsantos 847ace9
add signature collection
diegomrsantos d6ac318
Merge branch 'unstable' into voluntary-exit
diegomrsantos fddb299
fix get_validator_index
diegomrsantos 9b264ba
check the owner for all validators; process exit only for our validators
diegomrsantos ce1964d
fix is_our_validator
diegomrsantos 3109ab7
Merge branch 'unstable' into voluntary-exit
diegomrsantos 89c2cea
add docs to verify_validator_owner
diegomrsantos 0f74f32
Merge branch 'unstable' into voluntary-exit
diegomrsantos b459206
add doc for get_validator_index
diegomrsantos 92f81c7
parse block_timestamp as secs
diegomrsantos 7120a81
Merge branch 'unstable' into voluntary-exit
diegomrsantos 6929260
fix formatting
diegomrsantos 6eec9ff
process exit at the right slot
diegomrsantos b7fe568
Merge branch 'unstable' into voluntary-exit
diegomrsantos d0da421
get block ts from exec client if not available
diegomrsantos cf67a16
Merge branch 'unstable' into voluntary-exit
diegomrsantos 807060d
remove unused dep
diegomrsantos 3e29869
Merge branch 'unstable' into voluntary-exit
diegomrsantos b0b7337
Merge branch 'unstable' into voluntary-exit
diegomrsantos fb03944
fix domain computation
diegomrsantos ecddf0b
Merge branch 'unstable' into voluntary-exit
diegomrsantos d0fb2c0
simplify match
diegomrsantos 36b78b0
use workspace edition and add authors
diegomrsantos c0d5ae6
make code more idiomatic
diegomrsantos e38a411
renaming to collect_voluntary_exit_partial_signatures
diegomrsantos 6c6a289
Merge branch 'unstable' into voluntary-exit
diegomrsantos ef660fb
remove retry
diegomrsantos 6d792c4
use duration_to_next_slot instead of calculate_sleep_duration
diegomrsantos 61e7737
lint
diegomrsantos d458461
remove last_processed_slot
diegomrsantos 57d7a64
remove unused code
diegomrsantos 34fa7f1
Update anchor/voluntary_exit/src/voluntary_exit_processor.rs
diegomrsantos e53362c
use voluntary_exit.get_domain
diegomrsantos fab92a4
return error when cluster is liquidated
diegomrsantos 1736e8f
Merge branch 'unstable' into voluntary-exit
diegomrsantos bd8c8a3
simplify is_our_operator
diegomrsantos 7980067
Update anchor/common/ssv_types/src/share.rs
diegomrsantos f4d7877
update ShareMultiIndexMap docs
diegomrsantos cebe4e8
Merge branch 'unstable' into voluntary-exit
diegomrsantos File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dknopik maybe we should bound this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You know my liberal policy about
unbounded_channelthat have a natural boundary due to on-chain limitations ;DStill, if you want you can estimate how many exits can be burst in a single block, add some safety margin on top and apply that as limit.