Will Berkeley has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/9882 )

Change subject: [tools] ksck improvements [2/n]: KUDU-2306: ksck master health 
check
......................................................................

[tools] ksck improvements [2/n]: KUDU-2306: ksck master health check

This patch adds support for checking the health of each master passed to
ksck. Currently, ksck only checks that it can connect to the cluster by
connecting to a leader master.

Just like the tablet server health summary introduced recently, there's
now a master health summary:

$ build/latest/bin/kudu cluster ksck 
localhost:7051,localhost:7052,localhost:7053
WARNING: Unable to connect to master <unknown> (localhost:7052): Network error: 
could not get status from master: Client connection negotiation failed: client 
connection to 127.0.0.1:7052: connect: Connection refused (error 61)
Master Summary
               UUID               |    Address     |   Status
----------------------------------+----------------+-------------
 2096af644c1946e98628b7e9356f9ecf | localhost:7053 | HEALTHY
 99179f76035447ffaee00d15bfc89ebf | localhost:7051 | HEALTHY
 <unknown>                        | localhost:7052 | UNAVAILABLE

Change-Id: Ie0066468a0adc563979dcd2c61cd65d66978420c
Reviewed-on: http://gerrit.cloudera.org:8080/9882
Reviewed-by: Andrew Wong <[email protected]>
Tested-by: Will Berkeley <[email protected]>
---
M src/kudu/integration-tests/cluster_verifier.cc
M src/kudu/tools/ksck-test.cc
M src/kudu/tools/ksck.cc
M src/kudu/tools/ksck.h
M src/kudu/tools/ksck_remote-test.cc
M src/kudu/tools/ksck_remote.cc
M src/kudu/tools/ksck_remote.h
M src/kudu/tools/tool_action_cluster.cc
8 files changed, 404 insertions(+), 117 deletions(-)

Approvals:
  Andrew Wong: Looks good to me, approved
  Will Berkeley: Verified

--
To view, visit http://gerrit.cloudera.org:8080/9882
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie0066468a0adc563979dcd2c61cd65d66978420c
Gerrit-Change-Number: 9882
Gerrit-PatchSet: 6
Gerrit-Owner: Will Berkeley <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Tidy Bot
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Will Berkeley <[email protected]>

Reply via email to