-
Notifications
You must be signed in to change notification settings - Fork 366
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
[INDY-2324] Add random initial view no for sim pool #1455
[INDY-2324] Add random initial view no for sim pool #1455
Conversation
…m pool Signed-off-by: Andrew Nikitin <andrew.nikitin@dsr-corporation.com>
Signed-off-by: Andrew Nikitin <andrew.nikitin@dsr-corporation.com>
Signed-off-by: Andrew Nikitin <andrew.nikitin@dsr-corporation.com>
Signed-off-by: Andrew Nikitin <andrew.nikitin@dsr-corporation.com>
Signed-off-by: Andrew Nikitin <andrew.nikitin@dsr-corporation.com>
…m tests Signed-off-by: ashcherbakov <alexander.sherbakov@dsr-corporation.com>
Signed-off-by: Andrew Nikitin <andrew.nikitin@dsr-corporation.com>
…y-2324 Signed-off-by: Andrew Nikitin <andrew.nikitin@dsr-corporation.com>
Signed-off-by: ashcherbakov <alexander.sherbakov@dsr-corporation.com>
Signed-off-by: ashcherbakov <alexander.sherbakov@dsr-corporation.com>
Signed-off-by: ashcherbakov <alexander.sherbakov@dsr-corporation.com>
Signed-off-by: ashcherbakov <alexander.sherbakov@dsr-corporation.com>
Signed-off-by: Andrew Nikitin <andrew.nikitin@dsr-corporation.com>
…y-2324 Signed-off-by: Andrew Nikitin <andrew.nikitin@dsr-corporation.com>
Signed-off-by: Andrew Nikitin <andrew.nikitin@dsr-corporation.com>
@@ -104,6 +113,12 @@ def __init__(self, name: str, validators: List[str], primary_name: str, | |||
# ToDo: ugly way to understand node_reg changing | |||
self._previous_node_reg = self._write_manager.node_reg_handler.committed_node_reg | |||
|
|||
def setup_view_no(self, view_no=None): | |||
self._data.view_no = view_no | |||
self._data.primaries = self._primaries_selector.select_primaries(view_no) |
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.
We don't have primaries
field in ConsensusSharedData
anymore
def test_view_change_while_ordering_with_real_msgs_default_seed(seed): | ||
do_test(seed) | ||
def test_view_change_while_ordering_with_real_msgs_random_seed(random): | ||
do_test(random) | ||
|
||
|
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.
Why did we remove fixed seed here? I think we need to keep for regression test purposes.
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.
I removed only simple seed from 0 to 100.
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.
I returned back fixed seed.
Signed-off-by: Andrew Nikitin <andrew.nikitin@dsr-corporation.com>
This pull request introduces 1 alert when merging c4b8408 into ea38ac1 - view on LGTM.com new alerts:
|
Signed-off-by: Andrew Nikitin <andrew.nikitin@dsr-corporation.com>
Signed-off-by: Andrew Nikitin <andrew.nikitin@dsr-corporation.com>
This pull request introduces 1 alert when merging a08287d into ea38ac1 - view on LGTM.com new alerts:
|
Signed-off-by: Andrew Nikitin <andrew.nikitin@dsr-corporation.com>
This pull request introduces 1 alert when merging fbd6de1 into ea38ac1 - view on LGTM.com new alerts:
|
Signed-off-by: Andrew Nikitin <andrew.nikitin@dsr-corporation.com>
Signed-off-by: Andrew Nikitin <andrew.nikitin@dsr-corporation.com>
This pull request introduces 1 alert when merging e50ae8e into bdc8d05 - view on LGTM.com new alerts:
|
Signed-off-by: Andrew Nikitin <andrew.nikitin@dsr-corporation.com>
This pull request introduces 1 alert when merging f59b184 into bdc8d05 - view on LGTM.com new alerts:
|
No description provided.