-
Notifications
You must be signed in to change notification settings - Fork 995
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
Implement Stream Duties Functionality #5685
Merged
Merged
Changes from all commits
Commits
Show all changes
56 commits
Select commit
Hold shift + click to select a range
29f4148
add test for stream duties
rauljordan ed807b0
rem gomock
rauljordan 4efab4e
Merge branch 'master' into stream-duties
rauljordan 99a848d
context cancelation test
rauljordan 08f37d5
use interface for epoch ticker in duties
rauljordan bb5e60d
fix build
rauljordan 3be7452
compute duties on first call
rauljordan aa9ea93
pass tests for streams
rauljordan d893fd7
gaz
rauljordan a04d738
ss stream duties
rauljordan 6a3392d
Merge branch 'master' into stream-duties
rauljordan 31d0bc1
ensure only stream across epoch-wide reorgs
rauljordan ac55332
regen mock for validator
rauljordan 3784dc2
Update beacon-chain/blockchain/head.go
rauljordan 89cdb94
Merge refs/heads/master into stream-duties
prylabs-bulldozer[bot] 6247c91
Merge refs/heads/master into stream-duties
prylabs-bulldozer[bot] 9a26622
Update beacon-chain/rpc/validator/server.go
rauljordan 467233d
Merge branch 'master' into stream-duties
rauljordan 86ab8a2
update workspac
rauljordan 21262d1
Merge refs/heads/master into stream-duties
prylabs-bulldozer[bot] 1585c14
Merge refs/heads/master into stream-duties
prylabs-bulldozer[bot] 78bc89f
Merge refs/heads/master into stream-duties
prylabs-bulldozer[bot] 645fdfd
Merge refs/heads/master into stream-duties
prylabs-bulldozer[bot] f616897
Merge refs/heads/master into stream-duties
prylabs-bulldozer[bot] 6161be4
Merge refs/heads/master into stream-duties
prylabs-bulldozer[bot] 33a3bc1
Merge refs/heads/master into stream-duties
prylabs-bulldozer[bot] 069c307
pregenesis check
rauljordan 7a40233
Merge refs/heads/master into stream-duties
prylabs-bulldozer[bot] 9029d63
tests pass
rauljordan 04f35d0
build fix
rauljordan a8358e7
Revert "tests pass"
rauljordan 9f7b659
ethereumapis
rauljordan 947c34a
gaz
rauljordan 419e08b
Merge refs/heads/master into stream-duties
prylabs-bulldozer[bot] d791233
Merge refs/heads/master into stream-duties
prylabs-bulldozer[bot] 90337c8
Merge refs/heads/master into stream-duties
prylabs-bulldozer[bot] 70e91ed
Merge refs/heads/master into stream-duties
prylabs-bulldozer[bot] 320c257
Merge refs/heads/master into stream-duties
prylabs-bulldozer[bot] a8082d7
Merge refs/heads/master into stream-duties
prylabs-bulldozer[bot] 90e9074
Merge refs/heads/master into stream-duties
prylabs-bulldozer[bot] 4cd6783
Merge refs/heads/master into stream-duties
prylabs-bulldozer[bot] 58e688e
Merge refs/heads/master into stream-duties
prylabs-bulldozer[bot] 8929e3e
Merge refs/heads/master into stream-duties
prylabs-bulldozer[bot] e03aa1d
Merge refs/heads/master into stream-duties
prylabs-bulldozer[bot] dfc7e36
Update beacon-chain/rpc/validator/assignments.go
prestonvanloon daeddb8
Merge refs/heads/master into stream-duties
prylabs-bulldozer[bot] c1dd9c4
Merge refs/heads/master into stream-duties
prylabs-bulldozer[bot] 23c7cf1
Merge refs/heads/master into stream-duties
prylabs-bulldozer[bot] 670dde7
fix confs
rauljordan 21305e7
Merge refs/heads/master into stream-duties
prylabs-bulldozer[bot] 0ca6512
Merge refs/heads/master into stream-duties
prylabs-bulldozer[bot] 1b8cf21
Merge refs/heads/master into stream-duties
prylabs-bulldozer[bot] f21c9a5
test for reorg
rauljordan 06121d7
Merge branch 'stream-duties' of github.com:prysmaticlabs/prysm into s…
rauljordan 92cd704
fix up required tests
rauljordan 68a9e33
fmt
rauljordan 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 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 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 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 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 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.
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.
Is this correct ? shouldn't it be new block root instead
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.
This is fine.
ReorgData
is just to notify downstream service and based on downstream service's need. CurrentlyOldRoot
andNewRoot
are not needed by the downstream services. They can be added later if its necessary