Skip to content
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

Bft bp deliveryclient censorshiptimeout 2 #4431

Conversation

MayRosenbaum
Copy link
Contributor

Type of change

  • Improvement (improvement to code)
  • Test update

Description

As described in issue #4401

Related issues

issue #4401

BFT header receivers that pull headers from the orderers, and keep the time and number of the last one.
The header receivers verify each block as it arrives.
The header receivers will receive (or ask for) full config blocks - in a later commit.
The header receivers will maintain their own private block verifier (bundle) - in a later commit.

A BFT censorship monitor which periodically compares the progress of headers relative to blocks.
The monitor triggers an alert if a header is ahead of the block stream for more than a certain time.

The BFTDeliverer is only a skeleton

Signed-off-by: Yoav Tock <tock@il.ibm.com>
Change-Id: I5180a98640073b87effb478fd86b6fa7d4df5dee
- A BFTDeliverer that fetches blocks and maintains a BFTCensorshipMonitor.
- Abstract the creation of a BFTCensorshipMonitor via an abstract factory, so that we can use a mock for it in testing.
- Add a "shuffledEndpoints" method to the connection source and test it.
- Unit testing of BFTDeliverer.

Signed-off-by: Yoav Tock <tock@il.ibm.com>
Change-Id: Ifead3f9e6c803c4d9fabc63acce11c6da472b88d
Integrate the BFT block deliverer with the deliver client.
If the consensus-type is BFT, the BFTDeliverer is created.

Signed-off-by: Yoav Tock <tock@il.ibm.com>
Change-Id: I94c4c5ef3060df4119fc079b2b27630e92015d98
@MayRosenbaum MayRosenbaum requested a review from a team as a code owner September 7, 2023 09:28
@MayRosenbaum MayRosenbaum force-pushed the bft-bp-deliveryclient-censorshiptimeout-2 branch 7 times, most recently from 4d6789e to 0c14f4f Compare September 14, 2023 08:08
@MayRosenbaum MayRosenbaum force-pushed the bft-bp-deliveryclient-censorshiptimeout-2 branch 10 times, most recently from 7b4f91f to 51483ec Compare September 21, 2023 13:54
@MayRosenbaum MayRosenbaum force-pushed the bft-bp-deliveryclient-censorshiptimeout-2 branch 7 times, most recently from f0dc535 to 75fb95d Compare October 2, 2023 08:48
Signed-off-by: May Rosenbaum <mayro1595@gmail.com>
@MayRosenbaum MayRosenbaum force-pushed the bft-bp-deliveryclient-censorshiptimeout-2 branch from 75fb95d to 9e0c3af Compare October 2, 2023 09:50
@MayRosenbaum
Copy link
Contributor Author

A new pull request was opened #4458

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants