Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/20701 )
Change subject: [WIP] Leader Master shut down test ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/20701/1/src/kudu/master/dynamic_multi_master-test.cc File src/kudu/master/dynamic_multi_master-test.cc: http://gerrit.cloudera.org:8080/#/c/20701/1/src/kudu/master/dynamic_multi_master-test.cc@1566 PS1, Line 1566: cluster_->master(0)->Shutdown(); : SleepFor(MonoDelta::FromSeconds(3)); According to the documented procedure at https://kudu.apache.org/docs/administration.html#migrate_to_multi_master (items 6 & 7), it's also necessary to restart all the tablet servers in the cluster with the --tserver_master_addrs flag updated correspondingly (i.e. list the addresses of all the existing Kudu masters in the cluster). You could check how it's done in ExternalMiniCluster::AddMaster(), and then used in the ExternalMiniClusterTest.TestAddMaster scenario. -- To view, visit http://gerrit.cloudera.org:8080/20701 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I038b3ea0ff0465dca7e8c5c1c2504d789fb05557 Gerrit-Change-Number: 20701 Gerrit-PatchSet: 1 Gerrit-Owner: Ádám Bakai <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Mon, 13 Nov 2023 18:39:30 +0000 Gerrit-HasComments: Yes
