Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14222 )
Change subject: KUDU-2069 p4: stop replication from failed servers in maintenance mode ...................................................................... KUDU-2069 p4: stop replication from failed servers in maintenance mode When determining whether a replica needs to be added, we may now consider a set of of UUIDs that are allowed to be in a "bad" state while not counting towards being under-replicated. Since the goal of maintenance mode is to cope with unexpected failures, "healthy" movement, e.g. through tooling that uses REPLACE and PROMOTE tagging, is still allowed to and from tservers in maintenance mode. Testing: - a unit test is added to exercise the new quorum logic to ignore certain failed UUIDs, taking into account REPLACE and PROMOTE replicas - integration tests are added to test: - behavior with RF=3 through restarts of the master - behavior when running move_replica tooling - behavior with RF=5 with background failures Change-Id: I9a63b55011d16900c0d27eac0eb75880074204db Reviewed-on: http://gerrit.cloudera.org:8080/14222 Reviewed-by: Adar Dembo <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> Tested-by: Andrew Wong <[email protected]> --- M src/kudu/consensus/quorum_util-test.cc M src/kudu/consensus/quorum_util.cc M src/kudu/consensus/quorum_util.h M src/kudu/integration-tests/CMakeLists.txt A src/kudu/integration-tests/maintenance_mode-itest.cc M src/kudu/master/catalog_manager.cc M src/kudu/master/ts_manager.cc M src/kudu/master/ts_manager.h 8 files changed, 657 insertions(+), 24 deletions(-) Approvals: Adar Dembo: Looks good to me, but someone else must approve Alexey Serbin: Looks good to me, approved Andrew Wong: Verified -- To view, visit http://gerrit.cloudera.org:8080/14222 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I9a63b55011d16900c0d27eac0eb75880074204db Gerrit-Change-Number: 14222 Gerrit-PatchSet: 7 Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241)
