-
Notifications
You must be signed in to change notification settings - Fork 293
Broadcast vote power. #4748
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
Broadcast vote power. #4748
Conversation
|
Hey @Frozen, during the testing I found a bug, more details below. Result: signPower wasn't updating after consensus crash, last successful value is return How to reproduce:
|
9cac668 to
050c2dc
Compare
|
Thanks for the fix, @Frozen I've tested the following scenario:
|
050c2dc to
1715ff5
Compare
|
rn, blockchain couldn't spin up in RPC/Rosseta tests due to: {
"level": "error",
"port": "9000",
"ip": "127.0.0.1",
"error": "cannot finalize block: Timeout waiting for commit sigs for reward calculation",
"caller": "/go/src/github.com/harmony-one/harmony/consensus/consensus_block_proposing.go:236",
"time": "2024-09-06T09:40:39.249311887Z",
"message": "[ProposeNewBlock] Failed finalizing the new block"
}
|
|
@Frozen you latest rebase/conflict fix broke consensus on localnet. Please check |
Rebase on latest dev broke consensus. Fixed |
Rechecked the same scenario as mentioned above on both shards with external validator:
Result: PASSED ✔️ |
|
@Frozen please fix rebase and fix conflict with latest dev |
5ad7789 to
7c0a850
Compare
|
Rebased |
|
just finish testing this one with the latest rebase. All good for me. Good job @Frozen |
Leader broadcasts vote power, all others receive numbers and store. It available to receive throw api method
GetLastSigningPower.