Andrew Wong has uploaded a new change for review. http://gerrit.cloudera.org:8080/7993
Change subject: Fix flakiness of ts_tablet_manager_itest TestFailedTabletsAreReplaced ...................................................................... Fix flakiness of ts_tablet_manager_itest TestFailedTabletsAreReplaced TestFailedTabletsAreReplaced manually fails the replica after only verifying that the tablet exists, with no regard for its state. This can cause the replica's bootstrap process to fail a check: F0907 00:05:46.153576 2697 tablet_replica.cc:173] Check failed: BOOTSTRAPPING == state_ (0 vs. 2) This is a test-only race where the replica successfully goes through the bootstrap process, the tablet is failed in test, and the TabletReplica::Start() is called on the replica, which requires its state to be BOOTSTRAPPING. This is not an issue seen in production, as bootstrapping is normally only run if the replica is not failed, but it did result in 6/1000 failures when run in release mode with --stress_cpu_thres=32. To fix this, the replica is failed only after the it is verified to be running. In doing so, the number of failures went from 6/1000 to 0/1000. Change-Id: I93b41c8196397ea5af42ed9e2aa47e967f7a520e --- M src/kudu/integration-tests/ts_tablet_manager-itest.cc 1 file changed, 14 insertions(+), 7 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/93/7993/1 -- To view, visit http://gerrit.cloudera.org:8080/7993 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I93b41c8196397ea5af42ed9e2aa47e967f7a520e Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
