-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Refactored large network test to use finality to assert success #978
Conversation
cc1ac20
to
dbc2858
Compare
Also increased the node count to 200. Wondering if any checks should be stricter... |
tests/multinode.rs
Outdated
|
||
let now = Instant::now(); | ||
let mut finality = client.get_finality(); | ||
while finality == last_finality { |
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.
while duration_as_ms(now.elapsed()) < client.get_finality() {
sleep(Duration::from_millis(100));
}
right? if finality is longer then T since confirmation we wait. If its shorter that means 2/3rds of the validators have moved passed T.
tests/multinode.rs
Outdated
let mut retain_me = true; | ||
let mut client = mk_client(&server.0); | ||
trace!("{:x} get_balance start", server.0.debug_id()); | ||
let getbal = retry_get_balance(&mut client, &bob_pubkey, Some(expected_balance)); |
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 would change this to just confirm the last sig in all the validators instead of retry get balance.
dbc2858
to
ecf120f
Compare
@aeyakovenko , the latest patch addresses all the comments. |
…#978) Bumps [typescript](https://github.com/Microsoft/TypeScript) from 4.1.2 to 4.1.3. - [Release notes](https://github.com/Microsoft/TypeScript/releases) - [Commits](microsoft/TypeScript@v4.1.2...v4.1.3) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…864) (#978) limits number of nodes per IP address in Turbine (#864) (cherry picked from commit c87b830) Co-authored-by: behzad nouri <[email protected]>
…olana-labs#864) (solana-labs#978) limits number of nodes per IP address in Turbine (solana-labs#864) (cherry picked from commit c87b830) Co-authored-by: behzad nouri <[email protected]>
No description provided.