Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/22517 )
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]> (cherry picked from commit 8ce91854a3ae749ea02c45096dfff4b877050a82) Reviewed-on: http://gerrit.cloudera.org:8080/22517 --- M src/kudu/integration-tests/raft_consensus_election-itest.cc 1 file changed, 5 insertions(+), 2 deletions(-) Approvals: Yifan Zhang: Looks good to me, approved Alexey Serbin: Verified -- To view, visit http://gerrit.cloudera.org:8080/22517 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.18.x Gerrit-MessageType: merged Gerrit-Change-Id: I3bee924353079f7c8bfab6d0d5a6367bd1ee243e Gerrit-Change-Number: 22517 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yifan Zhang <[email protected]>
