-
Notifications
You must be signed in to change notification settings - Fork 177
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
[Dynamic Protocol State] Changing structure of participants in EpochSetup
#4726
Merged
durkmurder
merged 44 commits into
feature/dynamic-protocol-state
from
yurii/4649-todos-and-refactoring-part-1
Oct 23, 2023
Merged
Changes from 41 commits
Commits
Show all changes
44 commits
Select commit
Hold shift + click to select a range
f490707
Updated EpochSetup to contain IdentitySkeletonList instead of Identit…
durkmurder 9e98875
Introduced generic types for identity list, identity skeleton, orderi…
durkmurder 8641897
Updated godoc, cleanup of commented out code
durkmurder 59112b2
Merge branch 'yurii/4719-refactor-protocol-state-entry' of https://gi…
durkmurder 40fdaf5
Merge branch 'feature/dynamic-protocol-state' into yurii/4649-todos-a…
durkmurder 001d6a0
Updated how generic identities are handled in hotstuff committee
durkmurder c2f1123
Updated state updater and other usages of protocol state
durkmurder d9e78a6
Updated how static replicas are implemented
durkmurder e408bc1
Updated how static consensus committee is implemented. Updated identi…
durkmurder 4f68f38
Updated filter adapter
durkmurder 16c1347
Work in progress on fixing mutator tests
durkmurder 1ea0703
Fixed more compilation issues in tests
durkmurder 72349e0
Moved identity list to a separate file
durkmurder 420244f
Added proper encoding for identity and identity skeleton
durkmurder a9d7587
Fixed compilation issues regarding generic identity in main codebase
durkmurder 23cec18
Fixed more compilation issues regarding generic identity
durkmurder 38ffeee
Fixed compilation and implementation issues with tests
durkmurder 6e65018
Removed outdated tests. Fixed compilation for other tests
durkmurder ea3ef44
Fixed execution test
durkmurder 8f91562
Fixed last batch of broken tests
durkmurder 092927a
Linted
durkmurder a5ca17d
Updated godoc
durkmurder 5acf227
Fixed compilation in DKG tests
durkmurder d4cba51
Fixed compilation in integration tests
durkmurder 9230603
Merge branch 'yurii/4649-todos-and-refactoring-part-1' of https://git…
durkmurder fe04abc
Fixed test fixtures. Updated implementation of EqualTo
durkmurder 3dd3c4d
Updated identities ordering for some tests. Changed how root QC are v…
durkmurder d3ee95b
Updated integration test fixtures to work with BootstrapInfo instead …
durkmurder e4885eb
Linted
durkmurder 7e5d3c9
Fixed leftover compilation issues
durkmurder 6707537
Fixed insecure tests
durkmurder 99aff89
Linted
durkmurder 32dd1d2
Apply suggestions from code review
durkmurder 497bec2
Apply suggestions from PR review
durkmurder 636d3d1
Merge branch 'feature/dynamic-protocol-state' of https://github.com/o…
durkmurder 2a0b64b
removing dapper copyright notice.
4122b04
fixed typo
9feb09e
extended documentation of GenericIdentityList[T].Union
cf2469d
marginal goDoc update
c99c1ac
• extending documentation around cluster committee
739cbdf
Merge pull request #4832 from onflow/alex/4649-todos-and-refactoring-…
durkmurder b2c97b5
Apply suggestions from PR review
durkmurder 244af7b
Simplified bootstrap in cluster switchover test
durkmurder ad075bc
Apply suggestions from PR review
durkmurder 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
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
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
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.
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.
mockgen
doesn't support generics and it's deprecated I think we should replace it withmockery
but not as part of this PR. I've manually updated generated code to compile.