Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/21996 )
Change subject: KUDU-3589 Wait until masters become voters ...................................................................... KUDU-3589 Wait until masters become voters In SecurityITest.TestNonDefaultPrincipalMultipleMaster test, when the leader was shut down, it was possible that some of the Raft members are still in learner mode. This makes it impossible for the cluster to recover and elect a new leader. To solve this test issue, there is an explicit wait until masters become voters in the Raft consensus. This was done by using the already existing VerifyVotersOnAllMasters function. The function was moved from dynamic_multi_master-test to external_mini_cluster to be usable from security-itest, too. Change-Id: I1ea46345680ce506206e9335a147b887fc173b50 Reviewed-on: http://gerrit.cloudera.org:8080/21636 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> (cherry picked from commit c5993dbf4f8e706784c690f3d5b38cad75276387) Reviewed-on: http://gerrit.cloudera.org:8080/21996 Reviewed-by: Abhishek Chennaka <[email protected]> --- M src/kudu/integration-tests/security-itest.cc M src/kudu/master/dynamic_multi_master-test.cc M src/kudu/mini-cluster/external_mini_cluster.cc M src/kudu/mini-cluster/external_mini_cluster.h 4 files changed, 46 insertions(+), 36 deletions(-) Approvals: Abhishek Chennaka: Looks good to me, approved Alexey Serbin: Verified -- To view, visit http://gerrit.cloudera.org:8080/21996 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.18.x Gerrit-MessageType: merged Gerrit-Change-Id: I1ea46345680ce506206e9335a147b887fc173b50 Gerrit-Change-Number: 21996 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: Ádám Bakai <[email protected]>
