You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.

510 lines
16 KiB

7 years ago
7 years ago
7 years ago
7 years ago
lint: Enable Golint (#4212) * Fix many golint errors * Fix golint errors in the 'lite' package * Don't export Pool.store * Fix typo * Revert unwanted changes * Fix errors in counter package * Fix linter errors in kvstore package * Fix linter error in example package * Fix error in tests package * Fix linter errors in v2 package * Fix linter errors in consensus package * Fix linter errors in evidence package * Fix linter error in fail package * Fix linter errors in query package * Fix linter errors in core package * Fix linter errors in node package * Fix linter errors in mempool package * Fix linter error in conn package * Fix linter errors in pex package * Rename PEXReactor export to Reactor * Fix linter errors in trust package * Fix linter errors in upnp package * Fix linter errors in p2p package * Fix linter errors in proxy package * Fix linter errors in mock_test package * Fix linter error in client_test package * Fix linter errors in coretypes package * Fix linter errors in coregrpc package * Fix linter errors in rpcserver package * Fix linter errors in rpctypes package * Fix linter errors in rpctest package * Fix linter error in json2wal script * Fix linter error in wal2json script * Fix linter errors in kv package * Fix linter error in state package * Fix linter error in grpc_client * Fix linter errors in types package * Fix linter error in version package * Fix remaining errors * Address review comments * Fix broken tests * Reconcile package coregrpc * Fix golangci bot error * Fix new golint errors * Fix broken reference * Enable golint linter * minor changes to bring golint into line * fix failing test * fix pex reactor naming * address PR comments
5 years ago
7 years ago
8 years ago
pubsub 2.0 (#3227) * green pubsub tests :OK: * get rid of clientToQueryMap * Subscribe and SubscribeUnbuffered * start adapting other pkgs to new pubsub * nope * rename MsgAndTags to Message * remove TagMap it does not bring any additional benefits * bring back EventSubscriber * fix test * fix data race in TestStartNextHeightCorrectly ``` Write at 0x00c0001c7418 by goroutine 796: github.com/tendermint/tendermint/consensus.TestStartNextHeightCorrectly() /go/src/github.com/tendermint/tendermint/consensus/state_test.go:1296 +0xad testing.tRunner() /usr/local/go/src/testing/testing.go:827 +0x162 Previous read at 0x00c0001c7418 by goroutine 858: github.com/tendermint/tendermint/consensus.(*ConsensusState).addVote() /go/src/github.com/tendermint/tendermint/consensus/state.go:1631 +0x1366 github.com/tendermint/tendermint/consensus.(*ConsensusState).tryAddVote() /go/src/github.com/tendermint/tendermint/consensus/state.go:1476 +0x8f github.com/tendermint/tendermint/consensus.(*ConsensusState).handleMsg() /go/src/github.com/tendermint/tendermint/consensus/state.go:667 +0xa1e github.com/tendermint/tendermint/consensus.(*ConsensusState).receiveRoutine() /go/src/github.com/tendermint/tendermint/consensus/state.go:628 +0x794 Goroutine 796 (running) created at: testing.(*T).Run() /usr/local/go/src/testing/testing.go:878 +0x659 testing.runTests.func1() /usr/local/go/src/testing/testing.go:1119 +0xa8 testing.tRunner() /usr/local/go/src/testing/testing.go:827 +0x162 testing.runTests() /usr/local/go/src/testing/testing.go:1117 +0x4ee testing.(*M).Run() /usr/local/go/src/testing/testing.go:1034 +0x2ee main.main() _testmain.go:214 +0x332 Goroutine 858 (running) created at: github.com/tendermint/tendermint/consensus.(*ConsensusState).startRoutines() /go/src/github.com/tendermint/tendermint/consensus/state.go:334 +0x221 github.com/tendermint/tendermint/consensus.startTestRound() /go/src/github.com/tendermint/tendermint/consensus/common_test.go:122 +0x63 github.com/tendermint/tendermint/consensus.TestStateFullRound1() /go/src/github.com/tendermint/tendermint/consensus/state_test.go:255 +0x397 testing.tRunner() /usr/local/go/src/testing/testing.go:827 +0x162 ``` * fixes after my own review * fix formatting * wait 100ms before kicking a subscriber out + a test for indexer_service * fixes after my second review * no timeout * add changelog entries * fix merge conflicts * fix typos after Thane's review Co-Authored-By: melekes <anton.kalyaev@gmail.com> * reformat code * rewrite indexer service in the attempt to fix failing test https://github.com/tendermint/tendermint/pull/3227/#issuecomment-462316527 * Revert "rewrite indexer service in the attempt to fix failing test" This reverts commit 0d9107a098230de7138abb1c201877c246e89ed1. * another attempt to fix indexer * fixes after Ethan's review * use unbuffered channel when indexing transactions Refs https://github.com/tendermint/tendermint/pull/3227#discussion_r258786716 * add a comment for EventBus#SubscribeUnbuffered * format code
6 years ago
pubsub 2.0 (#3227) * green pubsub tests :OK: * get rid of clientToQueryMap * Subscribe and SubscribeUnbuffered * start adapting other pkgs to new pubsub * nope * rename MsgAndTags to Message * remove TagMap it does not bring any additional benefits * bring back EventSubscriber * fix test * fix data race in TestStartNextHeightCorrectly ``` Write at 0x00c0001c7418 by goroutine 796: github.com/tendermint/tendermint/consensus.TestStartNextHeightCorrectly() /go/src/github.com/tendermint/tendermint/consensus/state_test.go:1296 +0xad testing.tRunner() /usr/local/go/src/testing/testing.go:827 +0x162 Previous read at 0x00c0001c7418 by goroutine 858: github.com/tendermint/tendermint/consensus.(*ConsensusState).addVote() /go/src/github.com/tendermint/tendermint/consensus/state.go:1631 +0x1366 github.com/tendermint/tendermint/consensus.(*ConsensusState).tryAddVote() /go/src/github.com/tendermint/tendermint/consensus/state.go:1476 +0x8f github.com/tendermint/tendermint/consensus.(*ConsensusState).handleMsg() /go/src/github.com/tendermint/tendermint/consensus/state.go:667 +0xa1e github.com/tendermint/tendermint/consensus.(*ConsensusState).receiveRoutine() /go/src/github.com/tendermint/tendermint/consensus/state.go:628 +0x794 Goroutine 796 (running) created at: testing.(*T).Run() /usr/local/go/src/testing/testing.go:878 +0x659 testing.runTests.func1() /usr/local/go/src/testing/testing.go:1119 +0xa8 testing.tRunner() /usr/local/go/src/testing/testing.go:827 +0x162 testing.runTests() /usr/local/go/src/testing/testing.go:1117 +0x4ee testing.(*M).Run() /usr/local/go/src/testing/testing.go:1034 +0x2ee main.main() _testmain.go:214 +0x332 Goroutine 858 (running) created at: github.com/tendermint/tendermint/consensus.(*ConsensusState).startRoutines() /go/src/github.com/tendermint/tendermint/consensus/state.go:334 +0x221 github.com/tendermint/tendermint/consensus.startTestRound() /go/src/github.com/tendermint/tendermint/consensus/common_test.go:122 +0x63 github.com/tendermint/tendermint/consensus.TestStateFullRound1() /go/src/github.com/tendermint/tendermint/consensus/state_test.go:255 +0x397 testing.tRunner() /usr/local/go/src/testing/testing.go:827 +0x162 ``` * fixes after my own review * fix formatting * wait 100ms before kicking a subscriber out + a test for indexer_service * fixes after my second review * no timeout * add changelog entries * fix merge conflicts * fix typos after Thane's review Co-Authored-By: melekes <anton.kalyaev@gmail.com> * reformat code * rewrite indexer service in the attempt to fix failing test https://github.com/tendermint/tendermint/pull/3227/#issuecomment-462316527 * Revert "rewrite indexer service in the attempt to fix failing test" This reverts commit 0d9107a098230de7138abb1c201877c246e89ed1. * another attempt to fix indexer * fixes after Ethan's review * use unbuffered channel when indexing transactions Refs https://github.com/tendermint/tendermint/pull/3227#discussion_r258786716 * add a comment for EventBus#SubscribeUnbuffered * format code
6 years ago
pubsub 2.0 (#3227) * green pubsub tests :OK: * get rid of clientToQueryMap * Subscribe and SubscribeUnbuffered * start adapting other pkgs to new pubsub * nope * rename MsgAndTags to Message * remove TagMap it does not bring any additional benefits * bring back EventSubscriber * fix test * fix data race in TestStartNextHeightCorrectly ``` Write at 0x00c0001c7418 by goroutine 796: github.com/tendermint/tendermint/consensus.TestStartNextHeightCorrectly() /go/src/github.com/tendermint/tendermint/consensus/state_test.go:1296 +0xad testing.tRunner() /usr/local/go/src/testing/testing.go:827 +0x162 Previous read at 0x00c0001c7418 by goroutine 858: github.com/tendermint/tendermint/consensus.(*ConsensusState).addVote() /go/src/github.com/tendermint/tendermint/consensus/state.go:1631 +0x1366 github.com/tendermint/tendermint/consensus.(*ConsensusState).tryAddVote() /go/src/github.com/tendermint/tendermint/consensus/state.go:1476 +0x8f github.com/tendermint/tendermint/consensus.(*ConsensusState).handleMsg() /go/src/github.com/tendermint/tendermint/consensus/state.go:667 +0xa1e github.com/tendermint/tendermint/consensus.(*ConsensusState).receiveRoutine() /go/src/github.com/tendermint/tendermint/consensus/state.go:628 +0x794 Goroutine 796 (running) created at: testing.(*T).Run() /usr/local/go/src/testing/testing.go:878 +0x659 testing.runTests.func1() /usr/local/go/src/testing/testing.go:1119 +0xa8 testing.tRunner() /usr/local/go/src/testing/testing.go:827 +0x162 testing.runTests() /usr/local/go/src/testing/testing.go:1117 +0x4ee testing.(*M).Run() /usr/local/go/src/testing/testing.go:1034 +0x2ee main.main() _testmain.go:214 +0x332 Goroutine 858 (running) created at: github.com/tendermint/tendermint/consensus.(*ConsensusState).startRoutines() /go/src/github.com/tendermint/tendermint/consensus/state.go:334 +0x221 github.com/tendermint/tendermint/consensus.startTestRound() /go/src/github.com/tendermint/tendermint/consensus/common_test.go:122 +0x63 github.com/tendermint/tendermint/consensus.TestStateFullRound1() /go/src/github.com/tendermint/tendermint/consensus/state_test.go:255 +0x397 testing.tRunner() /usr/local/go/src/testing/testing.go:827 +0x162 ``` * fixes after my own review * fix formatting * wait 100ms before kicking a subscriber out + a test for indexer_service * fixes after my second review * no timeout * add changelog entries * fix merge conflicts * fix typos after Thane's review Co-Authored-By: melekes <anton.kalyaev@gmail.com> * reformat code * rewrite indexer service in the attempt to fix failing test https://github.com/tendermint/tendermint/pull/3227/#issuecomment-462316527 * Revert "rewrite indexer service in the attempt to fix failing test" This reverts commit 0d9107a098230de7138abb1c201877c246e89ed1. * another attempt to fix indexer * fixes after Ethan's review * use unbuffered channel when indexing transactions Refs https://github.com/tendermint/tendermint/pull/3227#discussion_r258786716 * add a comment for EventBus#SubscribeUnbuffered * format code
6 years ago
8 years ago
8 years ago
8 years ago
7 years ago
lint: Enable Golint (#4212) * Fix many golint errors * Fix golint errors in the 'lite' package * Don't export Pool.store * Fix typo * Revert unwanted changes * Fix errors in counter package * Fix linter errors in kvstore package * Fix linter error in example package * Fix error in tests package * Fix linter errors in v2 package * Fix linter errors in consensus package * Fix linter errors in evidence package * Fix linter error in fail package * Fix linter errors in query package * Fix linter errors in core package * Fix linter errors in node package * Fix linter errors in mempool package * Fix linter error in conn package * Fix linter errors in pex package * Rename PEXReactor export to Reactor * Fix linter errors in trust package * Fix linter errors in upnp package * Fix linter errors in p2p package * Fix linter errors in proxy package * Fix linter errors in mock_test package * Fix linter error in client_test package * Fix linter errors in coretypes package * Fix linter errors in coregrpc package * Fix linter errors in rpcserver package * Fix linter errors in rpctypes package * Fix linter errors in rpctest package * Fix linter error in json2wal script * Fix linter error in wal2json script * Fix linter errors in kv package * Fix linter error in state package * Fix linter error in grpc_client * Fix linter errors in types package * Fix linter error in version package * Fix remaining errors * Address review comments * Fix broken tests * Reconcile package coregrpc * Fix golangci bot error * Fix new golint errors * Fix broken reference * Enable golint linter * minor changes to bring golint into line * fix failing test * fix pex reactor naming * address PR comments
5 years ago
7 years ago
7 years ago
lint: Enable Golint (#4212) * Fix many golint errors * Fix golint errors in the 'lite' package * Don't export Pool.store * Fix typo * Revert unwanted changes * Fix errors in counter package * Fix linter errors in kvstore package * Fix linter error in example package * Fix error in tests package * Fix linter errors in v2 package * Fix linter errors in consensus package * Fix linter errors in evidence package * Fix linter error in fail package * Fix linter errors in query package * Fix linter errors in core package * Fix linter errors in node package * Fix linter errors in mempool package * Fix linter error in conn package * Fix linter errors in pex package * Rename PEXReactor export to Reactor * Fix linter errors in trust package * Fix linter errors in upnp package * Fix linter errors in p2p package * Fix linter errors in proxy package * Fix linter errors in mock_test package * Fix linter error in client_test package * Fix linter errors in coretypes package * Fix linter errors in coregrpc package * Fix linter errors in rpcserver package * Fix linter errors in rpctypes package * Fix linter errors in rpctest package * Fix linter error in json2wal script * Fix linter error in wal2json script * Fix linter errors in kv package * Fix linter error in state package * Fix linter error in grpc_client * Fix linter errors in types package * Fix linter error in version package * Fix remaining errors * Address review comments * Fix broken tests * Reconcile package coregrpc * Fix golangci bot error * Fix new golint errors * Fix broken reference * Enable golint linter * minor changes to bring golint into line * fix failing test * fix pex reactor naming * address PR comments
5 years ago
7 years ago
8 years ago
8 years ago
7 years ago
8 years ago
8 years ago
WAL: better errors and new fail point (#3246) * privval: more info in errors * wal: change Debug logs to Info * wal: log and return error on corrupted wal instead of panicing * fail: Exit right away instead of sending interupt * consensus: FAIL before handling our own vote allows to replicate #3089: - run using `FAIL_TEST_INDEX=0` - delete some bytes from the end of the WAL - start normally Results in logs like: ``` I[2019-02-03|18:12:58.225] Searching for height module=consensus wal=/Users/ethanbuchman/.tendermint/data/cs.wal/wal height=1 min=0 max=0 E[2019-02-03|18:12:58.225] Error on catchup replay. Proceeding to start ConsensusState anyway module=consensus err="failed to read data: EOF" I[2019-02-03|18:12:58.225] Started node module=main nodeInfo="{ProtocolVersion:{P2P:6 Block:9 App:1} ID_:35e87e93f2e31f305b65a5517fd2102331b56002 ListenAddr:tcp://0.0.0.0:26656 Network:test-chain-J8JvJH Version:0.29.1 Channels:4020212223303800 Moniker:Ethans-MacBook-Pro.local Other:{TxIndex:on RPCAddress:tcp://0.0.0.0:26657}}" E[2019-02-03|18:12:58.226] Couldn't connect to any seeds module=p2p I[2019-02-03|18:12:59.229] Timed out module=consensus dur=998.568ms height=1 round=0 step=RoundStepNewHeight I[2019-02-03|18:12:59.230] enterNewRound(1/0). Current: 1/0/RoundStepNewHeight module=consensus height=1 round=0 I[2019-02-03|18:12:59.230] enterPropose(1/0). Current: 1/0/RoundStepNewRound module=consensus height=1 round=0 I[2019-02-03|18:12:59.230] enterPropose: Our turn to propose module=consensus height=1 round=0 proposer=AD278B7767B05D7FBEB76207024C650988FA77D5 privValidator="PrivValidator{AD278B7767B05D7FBEB76207024C650988FA77D5 LH:1, LR:0, LS:2}" E[2019-02-03|18:12:59.230] enterPropose: Error signing proposal module=consensus height=1 round=0 err="Error signing proposal: Step regression at height 1 round 0. Got 1, last step 2" I[2019-02-03|18:13:02.233] Timed out module=consensus dur=3s height=1 round=0 step=RoundStepPropose I[2019-02-03|18:13:02.233] enterPrevote(1/0). Current: 1/0/RoundStepPropose module=consensus I[2019-02-03|18:13:02.233] enterPrevote: ProposalBlock is nil module=consensus height=1 round=0 E[2019-02-03|18:13:02.234] Error signing vote module=consensus height=1 round=0 vote="Vote{0:AD278B7767B0 1/00/1(Prevote) 000000000000 000000000000 @ 2019-02-04T02:13:02.233897Z}" err="Error signing vote: Conflicting data" ``` Notice the EOF, the step regression, and the conflicting data. * wal: change errors to be DataCorruptionError * exit on corrupt WAL * fix log * fix new line
6 years ago
8 years ago
7 years ago
7 years ago
7 years ago
7 years ago
7 years ago
7 years ago
8 years ago
7 years ago
cs/replay: execCommitBlock should not read from state.lastValidators (#3067) * execCommitBlock should not read from state.lastValidators * fix height 1 * fix blockchain/reactor_test * fix consensus/mempool_test * fix consensus/reactor_test * fix consensus/replay_test * add CHANGELOG * fix consensus/reactor_test * fix consensus/replay_test * add a test for replay validators change * fix mem_pool test * fix byzantine test * remove a redundant code * reduce validator change blocks to 6 * fix * return peer0 config * seperate testName * seperate testName 1 * seperate testName 2 * seperate app db path * seperate app db path 1 * add a lock before startNet * move the lock to reactor_test * simulate just once * try to find problem * handshake only saveState when app version changed * update gometalinter to 3.0.0 (#3233) in the attempt to fix https://circleci.com/gh/tendermint/tendermint/43165 also code is simplified by running gofmt -s . remove unused vars enable linters we're currently passing remove deprecated linters (cherry picked from commit d47094550315c094512a242445e0dde24b5a03f5) * gofmt code * goimport code * change the bool name to testValidatorsChange * adjust receive kvstore.ProtocolVersion * adjust receive kvstore.ProtocolVersion 1 * adjust receive kvstore.ProtocolVersion 3 * fix merge execution.go * fix merge develop * fix merge develop 1 * fix run cleanupFunc * adjust code according to reviewers' opinion * modify the func name match the convention * simplify simulate a chain containing some validator change txs 1 * test CI error * Merge remote-tracking branch 'upstream/develop' into fixReplay 1 * fix pubsub_test * subscribeUnbuffered vote channel
6 years ago
7 years ago
7 years ago
7 years ago
add support for block pruning via ABCI Commit response (#4588) * Added BlockStore.DeleteBlock() * Added initial block pruner prototype * wip * Added BlockStore.PruneBlocks() * Added consensus setting for block pruning * Added BlockStore base * Error on replay if base does not have blocks * Handle missing blocks when sending VoteSetMaj23Message * Error message tweak * Properly update blockstore state * Error message fix again * blockchain: ignore peer missing blocks * Added FIXME * Added test for block replay with truncated history * Handle peer base in blockchain reactor * Improved replay error handling * Added tests for Store.PruneBlocks() * Fix non-RPC handling of truncated block history * Panic on missing block meta in needProofBlock() * Updated changelog * Handle truncated block history in RPC layer * Added info about earliest block in /status RPC * Reorder height and base in blockchain reactor messages * Updated changelog * Fix tests * Appease linter * Minor review fixes * Non-empty BlockStores should always have base > 0 * Update code to assume base > 0 invariant * Added blockstore tests for pruning to 0 * Make sure we don't prune below the current base * Added BlockStore.Size() * config: added retain_blocks recommendations * Update v1 blockchain reactor to handle blockstore base * Added state database pruning * Propagate errors on missing validator sets * Comment tweaks * Improved error message Co-Authored-By: Anton Kaliaev <anton.kalyaev@gmail.com> * use ABCI field ResponseCommit.retain_height instead of retain-blocks config option * remove State.RetainHeight, return value instead * fix minor issues * rename pruneHeights() to pruneBlocks() * noop to fix GitHub borkage Co-authored-by: Anton Kaliaev <anton.kalyaev@gmail.com>
5 years ago
7 years ago
add support for block pruning via ABCI Commit response (#4588) * Added BlockStore.DeleteBlock() * Added initial block pruner prototype * wip * Added BlockStore.PruneBlocks() * Added consensus setting for block pruning * Added BlockStore base * Error on replay if base does not have blocks * Handle missing blocks when sending VoteSetMaj23Message * Error message tweak * Properly update blockstore state * Error message fix again * blockchain: ignore peer missing blocks * Added FIXME * Added test for block replay with truncated history * Handle peer base in blockchain reactor * Improved replay error handling * Added tests for Store.PruneBlocks() * Fix non-RPC handling of truncated block history * Panic on missing block meta in needProofBlock() * Updated changelog * Handle truncated block history in RPC layer * Added info about earliest block in /status RPC * Reorder height and base in blockchain reactor messages * Updated changelog * Fix tests * Appease linter * Minor review fixes * Non-empty BlockStores should always have base > 0 * Update code to assume base > 0 invariant * Added blockstore tests for pruning to 0 * Make sure we don't prune below the current base * Added BlockStore.Size() * config: added retain_blocks recommendations * Update v1 blockchain reactor to handle blockstore base * Added state database pruning * Propagate errors on missing validator sets * Comment tweaks * Improved error message Co-Authored-By: Anton Kaliaev <anton.kalyaev@gmail.com> * use ABCI field ResponseCommit.retain_height instead of retain-blocks config option * remove State.RetainHeight, return value instead * fix minor issues * rename pruneHeights() to pruneBlocks() * noop to fix GitHub borkage Co-authored-by: Anton Kaliaev <anton.kalyaev@gmail.com>
5 years ago
add support for block pruning via ABCI Commit response (#4588) * Added BlockStore.DeleteBlock() * Added initial block pruner prototype * wip * Added BlockStore.PruneBlocks() * Added consensus setting for block pruning * Added BlockStore base * Error on replay if base does not have blocks * Handle missing blocks when sending VoteSetMaj23Message * Error message tweak * Properly update blockstore state * Error message fix again * blockchain: ignore peer missing blocks * Added FIXME * Added test for block replay with truncated history * Handle peer base in blockchain reactor * Improved replay error handling * Added tests for Store.PruneBlocks() * Fix non-RPC handling of truncated block history * Panic on missing block meta in needProofBlock() * Updated changelog * Handle truncated block history in RPC layer * Added info about earliest block in /status RPC * Reorder height and base in blockchain reactor messages * Updated changelog * Fix tests * Appease linter * Minor review fixes * Non-empty BlockStores should always have base > 0 * Update code to assume base > 0 invariant * Added blockstore tests for pruning to 0 * Make sure we don't prune below the current base * Added BlockStore.Size() * config: added retain_blocks recommendations * Update v1 blockchain reactor to handle blockstore base * Added state database pruning * Propagate errors on missing validator sets * Comment tweaks * Improved error message Co-Authored-By: Anton Kaliaev <anton.kalyaev@gmail.com> * use ABCI field ResponseCommit.retain_height instead of retain-blocks config option * remove State.RetainHeight, return value instead * fix minor issues * rename pruneHeights() to pruneBlocks() * noop to fix GitHub borkage Co-authored-by: Anton Kaliaev <anton.kalyaev@gmail.com>
5 years ago
7 years ago
8 years ago
7 years ago
8 years ago
8 years ago
8 years ago
cs/replay: execCommitBlock should not read from state.lastValidators (#3067) * execCommitBlock should not read from state.lastValidators * fix height 1 * fix blockchain/reactor_test * fix consensus/mempool_test * fix consensus/reactor_test * fix consensus/replay_test * add CHANGELOG * fix consensus/reactor_test * fix consensus/replay_test * add a test for replay validators change * fix mem_pool test * fix byzantine test * remove a redundant code * reduce validator change blocks to 6 * fix * return peer0 config * seperate testName * seperate testName 1 * seperate testName 2 * seperate app db path * seperate app db path 1 * add a lock before startNet * move the lock to reactor_test * simulate just once * try to find problem * handshake only saveState when app version changed * update gometalinter to 3.0.0 (#3233) in the attempt to fix https://circleci.com/gh/tendermint/tendermint/43165 also code is simplified by running gofmt -s . remove unused vars enable linters we're currently passing remove deprecated linters (cherry picked from commit d47094550315c094512a242445e0dde24b5a03f5) * gofmt code * goimport code * change the bool name to testValidatorsChange * adjust receive kvstore.ProtocolVersion * adjust receive kvstore.ProtocolVersion 1 * adjust receive kvstore.ProtocolVersion 3 * fix merge execution.go * fix merge develop * fix merge develop 1 * fix run cleanupFunc * adjust code according to reviewers' opinion * modify the func name match the convention * simplify simulate a chain containing some validator change txs 1 * test CI error * Merge remote-tracking branch 'upstream/develop' into fixReplay 1 * fix pubsub_test * subscribeUnbuffered vote channel
6 years ago
add support for block pruning via ABCI Commit response (#4588) * Added BlockStore.DeleteBlock() * Added initial block pruner prototype * wip * Added BlockStore.PruneBlocks() * Added consensus setting for block pruning * Added BlockStore base * Error on replay if base does not have blocks * Handle missing blocks when sending VoteSetMaj23Message * Error message tweak * Properly update blockstore state * Error message fix again * blockchain: ignore peer missing blocks * Added FIXME * Added test for block replay with truncated history * Handle peer base in blockchain reactor * Improved replay error handling * Added tests for Store.PruneBlocks() * Fix non-RPC handling of truncated block history * Panic on missing block meta in needProofBlock() * Updated changelog * Handle truncated block history in RPC layer * Added info about earliest block in /status RPC * Reorder height and base in blockchain reactor messages * Updated changelog * Fix tests * Appease linter * Minor review fixes * Non-empty BlockStores should always have base > 0 * Update code to assume base > 0 invariant * Added blockstore tests for pruning to 0 * Make sure we don't prune below the current base * Added BlockStore.Size() * config: added retain_blocks recommendations * Update v1 blockchain reactor to handle blockstore base * Added state database pruning * Propagate errors on missing validator sets * Comment tweaks * Improved error message Co-Authored-By: Anton Kaliaev <anton.kalyaev@gmail.com> * use ABCI field ResponseCommit.retain_height instead of retain-blocks config option * remove State.RetainHeight, return value instead * fix minor issues * rename pruneHeights() to pruneBlocks() * noop to fix GitHub borkage Co-authored-by: Anton Kaliaev <anton.kalyaev@gmail.com>
5 years ago
  1. package consensus
  2. import (
  3. "bytes"
  4. "fmt"
  5. "hash/crc32"
  6. "io"
  7. "reflect"
  8. //"strconv"
  9. //"strings"
  10. "time"
  11. abci "github.com/tendermint/tendermint/abci/types"
  12. //auto "github.com/tendermint/tendermint/libs/autofile"
  13. dbm "github.com/tendermint/tm-db"
  14. "github.com/tendermint/tendermint/libs/log"
  15. "github.com/tendermint/tendermint/proxy"
  16. sm "github.com/tendermint/tendermint/state"
  17. "github.com/tendermint/tendermint/types"
  18. )
  19. var crc32c = crc32.MakeTable(crc32.Castagnoli)
  20. // Functionality to replay blocks and messages on recovery from a crash.
  21. // There are two general failure scenarios:
  22. //
  23. // 1. failure during consensus
  24. // 2. failure while applying the block
  25. //
  26. // The former is handled by the WAL, the latter by the proxyApp Handshake on
  27. // restart, which ultimately hands off the work to the WAL.
  28. //-----------------------------------------
  29. // 1. Recover from failure during consensus
  30. // (by replaying messages from the WAL)
  31. //-----------------------------------------
  32. // Unmarshal and apply a single message to the consensus state as if it were
  33. // received in receiveRoutine. Lines that start with "#" are ignored.
  34. // NOTE: receiveRoutine should not be running.
  35. func (cs *State) readReplayMessage(msg *TimedWALMessage, newStepSub types.Subscription) error {
  36. // Skip meta messages which exist for demarcating boundaries.
  37. if _, ok := msg.Msg.(EndHeightMessage); ok {
  38. return nil
  39. }
  40. // for logging
  41. switch m := msg.Msg.(type) {
  42. case types.EventDataRoundState:
  43. cs.Logger.Info("Replay: New Step", "height", m.Height, "round", m.Round, "step", m.Step)
  44. // these are playback checks
  45. ticker := time.After(time.Second * 2)
  46. if newStepSub != nil {
  47. select {
  48. case stepMsg := <-newStepSub.Out():
  49. m2 := stepMsg.Data().(types.EventDataRoundState)
  50. if m.Height != m2.Height || m.Round != m2.Round || m.Step != m2.Step {
  51. return fmt.Errorf("roundState mismatch. Got %v; Expected %v", m2, m)
  52. }
  53. case <-newStepSub.Cancelled():
  54. return fmt.Errorf("failed to read off newStepSub.Out(). newStepSub was cancelled")
  55. case <-ticker:
  56. return fmt.Errorf("failed to read off newStepSub.Out()")
  57. }
  58. }
  59. case msgInfo:
  60. peerID := m.PeerID
  61. if peerID == "" {
  62. peerID = "local"
  63. }
  64. switch msg := m.Msg.(type) {
  65. case *ProposalMessage:
  66. p := msg.Proposal
  67. cs.Logger.Info("Replay: Proposal", "height", p.Height, "round", p.Round, "header",
  68. p.BlockID.PartsHeader, "pol", p.POLRound, "peer", peerID)
  69. case *BlockPartMessage:
  70. cs.Logger.Info("Replay: BlockPart", "height", msg.Height, "round", msg.Round, "peer", peerID)
  71. case *VoteMessage:
  72. v := msg.Vote
  73. cs.Logger.Info("Replay: Vote", "height", v.Height, "round", v.Round, "type", v.Type,
  74. "blockID", v.BlockID, "peer", peerID)
  75. }
  76. cs.handleMsg(m)
  77. case timeoutInfo:
  78. cs.Logger.Info("Replay: Timeout", "height", m.Height, "round", m.Round, "step", m.Step, "dur", m.Duration)
  79. cs.handleTimeout(m, cs.RoundState)
  80. default:
  81. return fmt.Errorf("replay: Unknown TimedWALMessage type: %v", reflect.TypeOf(msg.Msg))
  82. }
  83. return nil
  84. }
  85. // Replay only those messages since the last block. `timeoutRoutine` should
  86. // run concurrently to read off tickChan.
  87. func (cs *State) catchupReplay(csHeight int64) error {
  88. // Set replayMode to true so we don't log signing errors.
  89. cs.replayMode = true
  90. defer func() { cs.replayMode = false }()
  91. // Ensure that #ENDHEIGHT for this height doesn't exist.
  92. // NOTE: This is just a sanity check. As far as we know things work fine
  93. // without it, and Handshake could reuse State if it weren't for
  94. // this check (since we can crash after writing #ENDHEIGHT).
  95. //
  96. // Ignore data corruption errors since this is a sanity check.
  97. gr, found, err := cs.wal.SearchForEndHeight(csHeight, &WALSearchOptions{IgnoreDataCorruptionErrors: true})
  98. if err != nil {
  99. return err
  100. }
  101. if gr != nil {
  102. if err := gr.Close(); err != nil {
  103. return err
  104. }
  105. }
  106. if found {
  107. return fmt.Errorf("wal should not contain #ENDHEIGHT %d", csHeight)
  108. }
  109. // Search for last height marker.
  110. //
  111. // Ignore data corruption errors in previous heights because we only care about last height
  112. gr, found, err = cs.wal.SearchForEndHeight(csHeight-1, &WALSearchOptions{IgnoreDataCorruptionErrors: true})
  113. if err == io.EOF {
  114. cs.Logger.Error("Replay: wal.group.Search returned EOF", "#ENDHEIGHT", csHeight-1)
  115. } else if err != nil {
  116. return err
  117. }
  118. if !found {
  119. return fmt.Errorf("cannot replay height %d. WAL does not contain #ENDHEIGHT for %d", csHeight, csHeight-1)
  120. }
  121. defer gr.Close() // nolint: errcheck
  122. cs.Logger.Info("Catchup by replaying consensus messages", "height", csHeight)
  123. var msg *TimedWALMessage
  124. dec := WALDecoder{gr}
  125. LOOP:
  126. for {
  127. msg, err = dec.Decode()
  128. switch {
  129. case err == io.EOF:
  130. break LOOP
  131. case IsDataCorruptionError(err):
  132. cs.Logger.Error("data has been corrupted in last height of consensus WAL", "err", err, "height", csHeight)
  133. return err
  134. case err != nil:
  135. return err
  136. }
  137. // NOTE: since the priv key is set when the msgs are received
  138. // it will attempt to eg double sign but we can just ignore it
  139. // since the votes will be replayed and we'll get to the next step
  140. if err := cs.readReplayMessage(msg, nil); err != nil {
  141. return err
  142. }
  143. }
  144. cs.Logger.Info("Replay: Done")
  145. return nil
  146. }
  147. //--------------------------------------------------------------------------------
  148. // Parses marker lines of the form:
  149. // #ENDHEIGHT: 12345
  150. /*
  151. func makeHeightSearchFunc(height int64) auto.SearchFunc {
  152. return func(line string) (int, error) {
  153. line = strings.TrimRight(line, "\n")
  154. parts := strings.Split(line, " ")
  155. if len(parts) != 2 {
  156. return -1, errors.New("line did not have 2 parts")
  157. }
  158. i, err := strconv.Atoi(parts[1])
  159. if err != nil {
  160. return -1, errors.New("failed to parse INFO: " + err.Error())
  161. }
  162. if height < i {
  163. return 1, nil
  164. } else if height == i {
  165. return 0, nil
  166. } else {
  167. return -1, nil
  168. }
  169. }
  170. }*/
  171. //---------------------------------------------------
  172. // 2. Recover from failure while applying the block.
  173. // (by handshaking with the app to figure out where
  174. // we were last, and using the WAL to recover there.)
  175. //---------------------------------------------------
  176. type Handshaker struct {
  177. stateDB dbm.DB
  178. initialState sm.State
  179. store sm.BlockStore
  180. eventBus types.BlockEventPublisher
  181. genDoc *types.GenesisDoc
  182. logger log.Logger
  183. nBlocks int // number of blocks applied to the state
  184. }
  185. func NewHandshaker(stateDB dbm.DB, state sm.State,
  186. store sm.BlockStore, genDoc *types.GenesisDoc) *Handshaker {
  187. return &Handshaker{
  188. stateDB: stateDB,
  189. initialState: state,
  190. store: store,
  191. eventBus: types.NopEventBus{},
  192. genDoc: genDoc,
  193. logger: log.NewNopLogger(),
  194. nBlocks: 0,
  195. }
  196. }
  197. func (h *Handshaker) SetLogger(l log.Logger) {
  198. h.logger = l
  199. }
  200. // SetEventBus - sets the event bus for publishing block related events.
  201. // If not called, it defaults to types.NopEventBus.
  202. func (h *Handshaker) SetEventBus(eventBus types.BlockEventPublisher) {
  203. h.eventBus = eventBus
  204. }
  205. // NBlocks returns the number of blocks applied to the state.
  206. func (h *Handshaker) NBlocks() int {
  207. return h.nBlocks
  208. }
  209. // TODO: retry the handshake/replay if it fails ?
  210. func (h *Handshaker) Handshake(proxyApp proxy.AppConns) error {
  211. // Handshake is done via ABCI Info on the query conn.
  212. res, err := proxyApp.Query().InfoSync(proxy.RequestInfo)
  213. if err != nil {
  214. return fmt.Errorf("error calling Info: %v", err)
  215. }
  216. blockHeight := res.LastBlockHeight
  217. if blockHeight < 0 {
  218. return fmt.Errorf("got a negative last block height (%d) from the app", blockHeight)
  219. }
  220. appHash := res.LastBlockAppHash
  221. h.logger.Info("ABCI Handshake App Info",
  222. "height", blockHeight,
  223. "hash", fmt.Sprintf("%X", appHash),
  224. "software-version", res.Version,
  225. "protocol-version", res.AppVersion,
  226. )
  227. // Set AppVersion on the state.
  228. if h.initialState.Version.Consensus.App != res.AppVersion {
  229. h.initialState.Version.Consensus.App = res.AppVersion
  230. sm.SaveState(h.stateDB, h.initialState)
  231. }
  232. // Replay blocks up to the latest in the blockstore.
  233. _, err = h.ReplayBlocks(h.initialState, appHash, blockHeight, proxyApp)
  234. if err != nil {
  235. return fmt.Errorf("error on replay: %v", err)
  236. }
  237. h.logger.Info("Completed ABCI Handshake - Tendermint and App are synced",
  238. "appHeight", blockHeight, "appHash", fmt.Sprintf("%X", appHash))
  239. // TODO: (on restart) replay mempool
  240. return nil
  241. }
  242. // ReplayBlocks replays all blocks since appBlockHeight and ensures the result
  243. // matches the current state.
  244. // Returns the final AppHash or an error.
  245. func (h *Handshaker) ReplayBlocks(
  246. state sm.State,
  247. appHash []byte,
  248. appBlockHeight int64,
  249. proxyApp proxy.AppConns,
  250. ) ([]byte, error) {
  251. storeBlockBase := h.store.Base()
  252. storeBlockHeight := h.store.Height()
  253. stateBlockHeight := state.LastBlockHeight
  254. h.logger.Info(
  255. "ABCI Replay Blocks",
  256. "appHeight",
  257. appBlockHeight,
  258. "storeHeight",
  259. storeBlockHeight,
  260. "stateHeight",
  261. stateBlockHeight)
  262. // If appBlockHeight == 0 it means that we are at genesis and hence should send InitChain.
  263. if appBlockHeight == 0 {
  264. validators := make([]*types.Validator, len(h.genDoc.Validators))
  265. for i, val := range h.genDoc.Validators {
  266. validators[i] = types.NewValidator(val.PubKey, val.Power)
  267. }
  268. validatorSet := types.NewValidatorSet(validators)
  269. nextVals := types.TM2PB.ValidatorUpdates(validatorSet)
  270. csParams := types.TM2PB.ConsensusParams(h.genDoc.ConsensusParams)
  271. req := abci.RequestInitChain{
  272. Time: h.genDoc.GenesisTime,
  273. ChainId: h.genDoc.ChainID,
  274. ConsensusParams: csParams,
  275. Validators: nextVals,
  276. AppStateBytes: h.genDoc.AppState,
  277. }
  278. res, err := proxyApp.Consensus().InitChainSync(req)
  279. if err != nil {
  280. return nil, err
  281. }
  282. if stateBlockHeight == 0 { //we only update state when we are in initial state
  283. // If the app returned validators or consensus params, update the state.
  284. if len(res.Validators) > 0 {
  285. vals, err := types.PB2TM.ValidatorUpdates(res.Validators)
  286. if err != nil {
  287. return nil, err
  288. }
  289. state.Validators = types.NewValidatorSet(vals)
  290. state.NextValidators = types.NewValidatorSet(vals)
  291. } else if len(h.genDoc.Validators) == 0 {
  292. // If validator set is not set in genesis and still empty after InitChain, exit.
  293. return nil, fmt.Errorf("validator set is nil in genesis and still empty after InitChain")
  294. }
  295. if res.ConsensusParams != nil {
  296. state.ConsensusParams = types.UpdateConsensusParams(state.ConsensusParams, res.ConsensusParams)
  297. }
  298. sm.SaveState(h.stateDB, state)
  299. }
  300. }
  301. // First handle edge cases and constraints on the storeBlockHeight and storeBlockBase.
  302. switch {
  303. case storeBlockHeight == 0:
  304. assertAppHashEqualsOneFromState(appHash, state)
  305. return appHash, nil
  306. case appBlockHeight < storeBlockBase-1:
  307. // the app is too far behind truncated store (can be 1 behind since we replay the next)
  308. return appHash, sm.ErrAppBlockHeightTooLow{AppHeight: appBlockHeight, StoreBase: storeBlockBase}
  309. case storeBlockHeight < appBlockHeight:
  310. // the app should never be ahead of the store (but this is under app's control)
  311. return appHash, sm.ErrAppBlockHeightTooHigh{CoreHeight: storeBlockHeight, AppHeight: appBlockHeight}
  312. case storeBlockHeight < stateBlockHeight:
  313. // the state should never be ahead of the store (this is under tendermint's control)
  314. panic(fmt.Sprintf("StateBlockHeight (%d) > StoreBlockHeight (%d)", stateBlockHeight, storeBlockHeight))
  315. case storeBlockHeight > stateBlockHeight+1:
  316. // store should be at most one ahead of the state (this is under tendermint's control)
  317. panic(fmt.Sprintf("StoreBlockHeight (%d) > StateBlockHeight + 1 (%d)", storeBlockHeight, stateBlockHeight+1))
  318. }
  319. var err error
  320. // Now either store is equal to state, or one ahead.
  321. // For each, consider all cases of where the app could be, given app <= store
  322. if storeBlockHeight == stateBlockHeight {
  323. // Tendermint ran Commit and saved the state.
  324. // Either the app is asking for replay, or we're all synced up.
  325. if appBlockHeight < storeBlockHeight {
  326. // the app is behind, so replay blocks, but no need to go through WAL (state is already synced to store)
  327. return h.replayBlocks(state, proxyApp, appBlockHeight, storeBlockHeight, false)
  328. } else if appBlockHeight == storeBlockHeight {
  329. // We're good!
  330. assertAppHashEqualsOneFromState(appHash, state)
  331. return appHash, nil
  332. }
  333. } else if storeBlockHeight == stateBlockHeight+1 {
  334. // We saved the block in the store but haven't updated the state,
  335. // so we'll need to replay a block using the WAL.
  336. switch {
  337. case appBlockHeight < stateBlockHeight:
  338. // the app is further behind than it should be, so replay blocks
  339. // but leave the last block to go through the WAL
  340. return h.replayBlocks(state, proxyApp, appBlockHeight, storeBlockHeight, true)
  341. case appBlockHeight == stateBlockHeight:
  342. // We haven't run Commit (both the state and app are one block behind),
  343. // so replayBlock with the real app.
  344. // NOTE: We could instead use the cs.WAL on cs.Start,
  345. // but we'd have to allow the WAL to replay a block that wrote it's #ENDHEIGHT
  346. h.logger.Info("Replay last block using real app")
  347. state, err = h.replayBlock(state, storeBlockHeight, proxyApp.Consensus())
  348. return state.AppHash, err
  349. case appBlockHeight == storeBlockHeight:
  350. // We ran Commit, but didn't save the state, so replayBlock with mock app.
  351. abciResponses, err := sm.LoadABCIResponses(h.stateDB, storeBlockHeight)
  352. if err != nil {
  353. return nil, err
  354. }
  355. mockApp := newMockProxyApp(appHash, abciResponses)
  356. h.logger.Info("Replay last block using mock app")
  357. state, err = h.replayBlock(state, storeBlockHeight, mockApp)
  358. return state.AppHash, err
  359. }
  360. }
  361. panic(fmt.Sprintf("uncovered case! appHeight: %d, storeHeight: %d, stateHeight: %d",
  362. appBlockHeight, storeBlockHeight, stateBlockHeight))
  363. }
  364. func (h *Handshaker) replayBlocks(
  365. state sm.State,
  366. proxyApp proxy.AppConns,
  367. appBlockHeight,
  368. storeBlockHeight int64,
  369. mutateState bool) ([]byte, error) {
  370. // App is further behind than it should be, so we need to replay blocks.
  371. // We replay all blocks from appBlockHeight+1.
  372. //
  373. // Note that we don't have an old version of the state,
  374. // so we by-pass state validation/mutation using sm.ExecCommitBlock.
  375. // This also means we won't be saving validator sets if they change during this period.
  376. // TODO: Load the historical information to fix this and just use state.ApplyBlock
  377. //
  378. // If mutateState == true, the final block is replayed with h.replayBlock()
  379. var appHash []byte
  380. var err error
  381. finalBlock := storeBlockHeight
  382. if mutateState {
  383. finalBlock--
  384. }
  385. for i := appBlockHeight + 1; i <= finalBlock; i++ {
  386. h.logger.Info("Applying block", "height", i)
  387. block := h.store.LoadBlock(i)
  388. // Extra check to ensure the app was not changed in a way it shouldn't have.
  389. if len(appHash) > 0 {
  390. assertAppHashEqualsOneFromBlock(appHash, block)
  391. }
  392. appHash, err = sm.ExecCommitBlock(proxyApp.Consensus(), block, h.logger, h.stateDB)
  393. if err != nil {
  394. return nil, err
  395. }
  396. h.nBlocks++
  397. }
  398. if mutateState {
  399. // sync the final block
  400. state, err = h.replayBlock(state, storeBlockHeight, proxyApp.Consensus())
  401. if err != nil {
  402. return nil, err
  403. }
  404. appHash = state.AppHash
  405. }
  406. assertAppHashEqualsOneFromState(appHash, state)
  407. return appHash, nil
  408. }
  409. // ApplyBlock on the proxyApp with the last block.
  410. func (h *Handshaker) replayBlock(state sm.State, height int64, proxyApp proxy.AppConnConsensus) (sm.State, error) {
  411. block := h.store.LoadBlock(height)
  412. meta := h.store.LoadBlockMeta(height)
  413. // Use stubs for both mempool and evidence pool since no transactions nor
  414. // evidence are needed here - block already exists.
  415. blockExec := sm.NewBlockExecutor(h.stateDB, h.logger, proxyApp, emptyMempool{}, emptyEvidencePool{})
  416. blockExec.SetEventBus(h.eventBus)
  417. var err error
  418. state, _, err = blockExec.ApplyBlock(state, meta.BlockID, block)
  419. if err != nil {
  420. return sm.State{}, err
  421. }
  422. h.nBlocks++
  423. return state, nil
  424. }
  425. func assertAppHashEqualsOneFromBlock(appHash []byte, block *types.Block) {
  426. if !bytes.Equal(appHash, block.AppHash) {
  427. panic(fmt.Sprintf(`block.AppHash does not match AppHash after replay. Got %X, expected %X.
  428. Block: %v
  429. `,
  430. appHash, block.AppHash, block))
  431. }
  432. }
  433. func assertAppHashEqualsOneFromState(appHash []byte, state sm.State) {
  434. if !bytes.Equal(appHash, state.AppHash) {
  435. panic(fmt.Sprintf(`state.AppHash does not match AppHash after replay. Got
  436. %X, expected %X.
  437. State: %v
  438. Did you reset Tendermint without resetting your application's data?`,
  439. appHash, state.AppHash, state))
  440. }
  441. }