-
Notifications
You must be signed in to change notification settings - Fork 7
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 multi-hop routing of consensus messages #472
Merged
Changes from all commits
Commits
Show all changes
44 commits
Select commit
Hold shift + click to select a range
cc46805
Add multi-hop routing implementation. This has some basic unit tests,…
swatanabe b5163db
Support local sockets for outgoing p2p connections
swatanabe 40cd347
Handle absolute-form request targets
swatanabe 9b131e1
Add unix socket support for psibase
swatanabe 6f4f39e
Add queries to expose all the consensus state
swatanabe 7a1941f
Make sure that match_index is set up when building a block that sets …
swatanabe 4ed7da6
Use shortest_path_routing in psinode
swatanabe fae8455
Add initial psinode integration test
swatanabe eea3ae8
Add test for changing routes
swatanabe 81fc74a
Fix test failures
swatanabe ea57fd5
Restrict routing to active producers. Prevents blowing up the routing…
swatanabe 01591d4
Fix test program name
swatanabe 3c5a50c
Update docker images
swatanabe 4d7cc39
Temporarily switch to image-builder PR merge commit
swatanabe 6099c9d
Fix typo
swatanabe d0eee6d
Update clang paths
swatanabe a896096
Ensure that log file is closed
swatanabe 56f107c
Increase log level
swatanabe de742a2
Fix use-after-free when disconnect removes the last feasible route
swatanabe 285e61c
Fix uninitialized variable
swatanabe 6c6213b
Some refactoring of tests
swatanabe 06675f8
Set --database-cache-size
swatanabe 1944989
Add test for routing with BFT
swatanabe a211c34
When we rate-limit seqno requests, delay rather than dropping the mos…
swatanabe e17ada8
Delay consensus messages until we're sure that the peer has the block
swatanabe c68e957
Fix compile error
swatanabe 5f828dc
Rework consensus to ensure convergence
swatanabe 2e35749
Fix compilation
swatanabe 6ecfd66
Restrict prepares and commits
swatanabe f688897
Improve log messages on consensus change
swatanabe 2bc6815
Merge remote-tracking branch 'origin/main' into routing
swatanabe b0f26da
Fix compile error on old gcc
swatanabe 5ed7ebf
Another instance of duration streaming
swatanabe 1400ae3
Merge remote-tracking branch 'origin/main' into routing
swatanabe 5c9b63a
Fix uninitialized variable
swatanabe d792e37
Include joint producers when searching for the best view change
swatanabe a19d25f
Fix another io_context halt
swatanabe 436f2c9
Update consensus docs
swatanabe 887003e
Bad timing can cause the tests to fail. Account for this by ignoring …
swatanabe 21e810c
Fix close on timeout. We should not depend on the client closing thei…
swatanabe 1eba2b2
Merge branch 'main' into routing
swatanabe 76c3192
Add tests with stopping and restarting nodes
swatanabe 55ed56f
Fix cmake
swatanabe 91864be
Implement new conditions on prepare and commit
swatanabe 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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -24,3 +24,4 @@ vite.config.ts.timestamp-*.mjs | |
psibase.127.0.0.1.sslip.io*.pem | ||
psinode_db | ||
psinode_db_secure | ||
__pycache__ |
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.
Doesn't the existence of X and Y already imply that at least f+1 producers have violated the protocol?
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.
It does. This list is the breakdown of exactly how they violated the protocol.