Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/22516 )
Change subject: KUDU-3641 fix flaky TestNewLeaderCantResolvePeers (take 2) ...................................................................... KUDU-3641 fix flaky TestNewLeaderCantResolvePeers (take 2) I noticed that even after [1] the TestNewLeaderCantResolvePeers was still failing in about once in 60 runs [2], so I took a closer look. It turns out StartElection() doesn't trigger a re-election if it arriving at the current Raft leader. To address that, this patch replaces StartElection() back with LeaderStepDown() but with new target leader being the tablet replica at the third tablet server. This is a follow-up to [1]. [1] https://github.com/apache/kudu/commit/6c77ec875 [2] http://dist-test.cloudera.org:8080/test_drilldown?test_name=raft_consensus_election-itest Change-Id: I3bee924353079f7c8bfab6d0d5a6367bd1ee243e Reviewed-on: http://gerrit.cloudera.org:8080/22516 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Yifan Zhang <[email protected]> --- M src/kudu/integration-tests/raft_consensus_election-itest.cc 1 file changed, 5 insertions(+), 2 deletions(-) Approvals: Alexey Serbin: Verified Yifan Zhang: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/22516 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I3bee924353079f7c8bfab6d0d5a6367bd1ee243e Gerrit-Change-Number: 22516 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yifan Zhang <[email protected]>
