Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14980 )
Change subject: test: deflake RaftConsensusNumLeadersMetricTest TestNumLeadersMetric ...................................................................... test: deflake RaftConsensusNumLeadersMetricTest TestNumLeadersMetric The test previously wouldn't wait for all tablets to be up and running on every tserver before doing stuff. This led to errors when attempting to start elections, since the StartElection RPC would be eaten and never retried, leaving some tablets with no leader. The fix is to wait for every tserver to have all running tablets. Before this fix, the test would fail in TSAN mode 6/1000 times. With it, it passed 1000/1000. Change-Id: I4ef4651e4562ac274982773ebb8c46e5137e38b5 Reviewed-on: http://gerrit.cloudera.org:8080/14980 Reviewed-by: Alexey Serbin <aser...@cloudera.com> Tested-by: Kudu Jenkins --- M src/kudu/integration-tests/raft_consensus_election-itest.cc 1 file changed, 4 insertions(+), 2 deletions(-) Approvals: Alexey Serbin: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/14980 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I4ef4651e4562ac274982773ebb8c46e5137e38b5 Gerrit-Change-Number: 14980 Gerrit-PatchSet: 4 Gerrit-Owner: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120)