-
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
add validator catchup to multi-node test #450
Conversation
Can you move that to a separate test? And rename the original to something more precise. Monolithic catch-all tests are the worst. |
tests/multinode.rs
Outdated
if let Ok(bal) = client.poll_get_balance(&bob_pubkey) { | ||
info!("validator balance {}", bal); | ||
if bal == leader_balance { | ||
success += 1; |
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 think you need to reset this to 0 if you are gong to rerun the loop
thanks, Stephen. my approach was to move leader creation farther up. this test is better, I think |
eb3d491
to
e605ae0
Compare
leader.sockets.respond, | ||
leader.sockets.gossip, | ||
exit.clone(), | ||
sink(), |
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 think we need to sink to tmp file, and let the validators restart from there.
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.
This is a confusing comment, since it's on the sink()
line. When you say "sink to a tmp file", do you mean "sync to a tmp file"?
e605ae0
to
7597d80
Compare
7597d80
to
360b41e
Compare
Bumps [eslint](https://github.com/eslint/eslint) from 7.8.1 to 7.9.0. - [Release notes](https://github.com/eslint/eslint/releases) - [Changelog](https://github.com/eslint/eslint/blob/master/CHANGELOG.md) - [Commits](eslint/eslint@v7.8.1...v7.9.0) Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Use cluster info functions for tpu (solana-labs#345) (solana-labs#347) Use git rev-parse for git sha Remove blacklisted tx from message_hash_to_transaction (Backport solana-labs#374) (solana-labs#376) Updates scripts for easy local setup. (solana-labs#383) Backports sim_bundle improvements (solana-labs#407) backports clone derivation 416 (solana-labs#417) Backport solana-labs#419: add upsert to AccountOverrides (solana-labs#420) backports solana-labs#430 v1.16: update jito-programs sha (solana-labs#431) [JIT-1661] Faster Autosnapshot (solana-labs#406) Fix Buildkite warnings (solana-labs#437) Backport solana-labs#446 to v1.16 (solana-labs#447) backport 428, runtime plugin (solana-labs#429) v1.16: Backport solana-labs#449 (solana-labs#450)
* runtime: Stakes::new: avoid extra loops over stakes.stake_delegations * runtime: parallelize Stakes::new with rayon * Address review comments * Address more comments
* runtime: Stakes::new: avoid extra loops over stakes.stake_delegations * runtime: parallelize Stakes::new with rayon * Address review comments * Address more comments
No description provided.