-
Notifications
You must be signed in to change notification settings - Fork 145
Aggregator, Sync Committee Contribution - Committee Runner #2503
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
155 commits
Select commit
Hold shift + click to select a range
8080ed4
draft
olegshmuelov 33f9a51
fix build
nkryuchkov c840f12
WIP on setting up aggregator committee runner
nkryuchkov e9c6e82
Merge branch 'stage' into aggregator-committee
nkryuchkov af127f6
implement reverted beacon client changes
nkryuchkov b6f495f
run aggregator committee duty
nkryuchkov 2bd98f9
add aggregator committee checks to message validation
nkryuchkov 1e2e89c
add basic fork handling to handlers
nkryuchkov 0107fe6
add submission logs for aggregator committee
nkryuchkov 5654a12
fix missed RoleAggregatorCommittee handling
nkryuchkov f875cc4
fix message validation aggregator committee issues
nkryuchkov 61a95a0
fix some message validation issues
nkryuchkov dc5fb2a
Merge branch 'stage' into aggregator-committee
nkryuchkov 7c2e239
WIP on issues after merging
nkryuchkov cf8d7bf
fix issues after merging
nkryuchkov be274d5
fix some bugs
nkryuchkov f9126ff
fix aggregator committee post-consensus message validation
nkryuchkov 9ce6743
fix on timeout logic for aggregator committee
nkryuchkov d661f22
fix aggregator committee value check
nkryuchkov 82c5b9f
fix aggregator committee post consensus message validation
nkryuchkov 518c69a
filter messages matching the runner role from the queue
nkryuchkov 0a835d9
fix expected aggregator committee duty type
nkryuchkov 54c5944
pass error on failure to decide
nkryuchkov 40bf179
fix ProcessConsensus
nkryuchkov 8c5630b
Merge branch 'stage' into aggregator-committee
nkryuchkov bbd1e09
fix merging leftovers
nkryuchkov 8ed4fd1
Merge branch 'stage' into aggregator-committee
nkryuchkov 2b0747b
fix queue bugs
nkryuchkov f8a7709
fix post-consensus bug
nkryuchkov 33b8a84
fix assertion bug
nkryuchkov 6aeb03e
missing metrics
nkryuchkov a88f6e5
set fork epoch in all configs
nkryuchkov 4c27b84
fork epoch 0 in test/local networks
nkryuchkov 71b07b4
fetch attester and sync committee duties
nkryuchkov 107a862
fix log text
nkryuchkov 16e76ec
update spec
nkryuchkov f9c884c
add fulu to constructSignedAggregateAndProof
nkryuchkov 539666b
fix a bug in constructSignedAggregateAndProof
nkryuchkov e0e75d7
fix a bug in constructSignedAggregateAndProof [2]
nkryuchkov 422a89a
update spec version
nkryuchkov 386a143
update spec version
nkryuchkov a36ce6d
update spec version
nkryuchkov 804fbfc
Merge branch 'stage' into aggregator-committee
nkryuchkov 7f8b7d9
fix linter
nkryuchkov 9a7b1ea
fix some tests
nkryuchkov 06e2692
fix a data race in tests
nkryuchkov a14b36c
Merge branch 'stage' into aggregator-committee
nkryuchkov 09a6835
fix using wrong cache
nkryuchkov 194586e
fix root caching
nkryuchkov 8bc8237
Merge branch 'stage' into aggregator-committee
nkryuchkov 9ef2204
add missing AggregatorCommittee in tests
nkryuchkov e74f880
fix duty scheduler unit tests
nkryuchkov 77e8a8d
add a TODO
nkryuchkov fcfcc36
partially fix ssv mapping tests
nkryuchkov a0cee07
leftovers for AggregatorRunners in ssv mapping
nkryuchkov fb477ce
fix spec tests
nkryuchkov 82e62ab
Merge branch 'stage' into aggregator-committee
nkryuchkov c3289c5
fix linter
nkryuchkov beb4ca5
simplify GetProcessMessageF
nkryuchkov 8a2737e
further simplification
nkryuchkov 82da430
delete ExecuteAggregatorCommitteeDuty
nkryuchkov d37f64c
delete CreateAggregatorRunnerFn
nkryuchkov 96f000c
code review
nkryuchkov 432d6a9
simplify validator committee
nkryuchkov ea29da2
delete confusing comment
nkryuchkov 0ac2c78
delete hardcoded consts
nkryuchkov 771752a
check PostDutyCommitteeRoot for aggregator committee
nkryuchkov 62ca1cd
fix aggregator duty submission bug
nkryuchkov 7a0291c
delete unnecessary normalizeAggregatorDecidedValues
nkryuchkov 30cc104
update spec version
nkryuchkov 8391389
cleanup some leftovers
nkryuchkov 014acc0
Merge branch 'stage' into aggregator-committee
nkryuchkov 341e6ff
Revert "update spec version"
nkryuchkov fecca4e
Merge branch 'stage' into aggregator-committee
nkryuchkov ef11ccc
code review suggestions
nkryuchkov 71b74b0
Merge branch 'stage' into aggregator-committee
nkryuchkov 8942757
fix spec tests data race
nkryuchkov 02851a0
get rid of the aggregator committee handler
nkryuchkov 53fbef1
wait only for committee role in duty scheduler
nkryuchkov 002eea4
add waiting in aggregator committee runner
nkryuchkov a8c4558
aggregator committee runner: improve formatting
nkryuchkov 86087ab
rename AggregatorCommittee fork to Boole; delete Alan fork from config
nkryuchkov 81da547
attempt to fix TestScheduler_Committee_Indices_Changed_Attester_Only
nkryuchkov 3d6f388
align to the latest spec changes
nkryuchkov 29527e5
improve the comment
nkryuchkov 1539de9
use trace.SpanFromContext
nkryuchkov dd93297
align to the latest spec
nkryuchkov 7971c7f
Merge branch 'stage' into aggregator-committee
nkryuchkov 8ae7ccd
align to https://github.com/ssvlabs/ssv/pull/2629
nkryuchkov cf2323e
code review comments
nkryuchkov e49c3ea
code review comments
nkryuchkov 2e0f8a5
Merge branch 'stage' into aggregator-committee
nkryuchkov eec1a74
message/validation: update rules according to https://github.com/ssvl…
nkryuchkov 7e0539a
fix variable name
nkryuchkov 3b895e3
align with the latest spec
nkryuchkov 7b3f7a7
align with the latest spec
nkryuchkov e73d41b
Merge branch 'stage' into aggregator-committee
nkryuchkov 90b6b1a
fix compilation issue
nkryuchkov 44a4e5f
fix tests
nkryuchkov 9ed705e
get message size limits from spec
nkryuchkov 86f431a
add nil share checks
nkryuchkov 10211c2
use spec without jsons
nkryuchkov e666322
update spec version for ssvsigner
nkryuchkov b372872
fix linter
nkryuchkov 188bbe5
fix missing roles in the updated spec
nkryuchkov 63305a5
fix remaining usages of duty.RunnerRole()
nkryuchkov 73c4358
fix pre-fork duty runner choice
nkryuchkov 9e43711
fix a committeeDuty bug
nkryuchkov 4ad0e5b
fix loop iteration
nkryuchkov a6a28f1
Merge branch 'stage' into aggregator-committee
nkryuchkov 8769885
fix issues after merging
nkryuchkov a1e1c4f
apply changes from https://github.com/ssvlabs/ssv/pull/2658 to the ag…
nkryuchkov be4ab2a
fix TestFieldPreservation
nkryuchkov 74f9c54
fix code review comments
nkryuchkov 8bbec89
use spec without some tests
nkryuchkov baf3f4c
fix some spec tests
nkryuchkov 8de1071
fix deduplication in agg comm runner
nkryuchkov 4cd8084
fix value checker in spec tests
nkryuchkov c21a701
remove redundant spec tests code
nkryuchkov 3a177ae
simplify ssv spec test
nkryuchkov 2f43aec
refactor ssv mapping test
nkryuchkov c5760b2
simplify fixCommitteeForRun
nkryuchkov a2ae5fa
fix linter
nkryuchkov e83d299
improve comments
nkryuchkov fbe56a6
reject Boole roles during Alan and Alan roles during Boole
nkryuchkov a3fe784
optimize the role check
nkryuchkov 61dd89c
add 'slot' to error log
nkryuchkov e133dfd
fix a bug with validator consensus data value check during Alan
nkryuchkov 8e35a08
use the correct root(s) when calculating SyncCommitteeSubnetID
nkryuchkov fcd5dc4
fix a panic on nil interface
nkryuchkov 6ce575e
Merge branch 'boole-fork' into aggregator-committee
nkryuchkov 217f53d
fix issues after merging
nkryuchkov cfe7d1a
deduplicate messages if validator has already been seen for a subnet
nkryuchkov 2aa6aa2
debug logs on errors in loops
nkryuchkov d74fa1e
cache aggregator committee roots
nkryuchkov 82f64a8
fix a typo
nkryuchkov 19ec3ba
end consensus on failure in decide()
nkryuchkov 60c46b2
iterate over consensus data instead of duty
nkryuchkov bc80e2f
add slot in returned error
nkryuchkov 4cf41f1
pass logger to where it's missing
nkryuchkov b48d61d
struct tags for SSVForks
nkryuchkov 37efb44
use correct type for sync committee index
nkryuchkov ece1a85
implement missing checks for aggregator committee
nkryuchkov 8abb87e
attempt to fix unit tests
nkryuchkov a4207dc
Revert "implement missing checks for aggregator committee"
nkryuchkov 8a3af6b
Revert "deduplicate messages if validator has already been seen for a…
nkryuchkov 714ce37
Merge branch 'boole-fork' into aggregator-committee
nkryuchkov c4d4909
fix issues after merging
nkryuchkov 46eb76d
fix linter
nkryuchkov b757f32
Merge branch 'boole-fork' into aggregator-committee
nkryuchkov 8596dbb
attempt to fix linter
nkryuchkov aa62a2c
fix unit tests after pulling changes from the base branch
nkryuchkov c4acabc
Revert "attempt to fix linter"
nkryuchkov c5fa18c
Merge branch 'boole-fork' into aggregator-committee
nkryuchkov 8894049
fix tests
nkryuchkov 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
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.
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.
Uh oh!
There was an error while loading. Please reload this page.