Will Berkeley has submitted this change and it was merged. (
http://gerrit.cloudera.org:8080/10579 )
Change subject: [tools] Add version info and check to ksck
......................................................................
[tools] Add version info and check to ksck
This adds a new version summary table, similar to the one on the
master web ui for tablet servers, that summarizes the versions of Kudu
running in the cluster (both masters and tablet servers). ksck will
issue a warning if there are multiple versions. The version info is
available as part of the server health summaries in the KsckResults, so
clients of the Ksck class can use this information. Version info is
plumbed into the JSON output as well.
An example table and warning:
Version Summary
Version | Servers
----------------+-----------------------------------------------------------------------------------------------------
1.6.0 | [email protected], [email protected],
[email protected], and 1 other server(s)
1.7.0 | [email protected]
1.8.0-SNAPSHOT | [email protected]
==================
Warnings:
==================
version check error: not all servers are running the same version: 3 different
versions were seen
Change-Id: I4a2bc7138f074ab8d74f21ace2eb2b8018c53f50
Reviewed-on: http://gerrit.cloudera.org:8080/10579
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin <[email protected]>
---
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.cc
M src/kudu/tools/ksck_results.cc
M src/kudu/tools/ksck_results.h
M src/kudu/tools/tool.proto
7 files changed, 118 insertions(+), 9 deletions(-)
Approvals:
Kudu Jenkins: Verified
Alexey Serbin: Looks good to me, approved
--
To view, visit http://gerrit.cloudera.org:8080/10579
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I4a2bc7138f074ab8d74f21ace2eb2b8018c53f50
Gerrit-Change-Number: 10579
Gerrit-PatchSet: 5
Gerrit-Owner: Will Berkeley <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot
Gerrit-Reviewer: Will Berkeley <[email protected]>
