Hello Kudu Jenkins, Adar Dembo, Volodymyr Verovkin,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/15298
to look at the new patch set (#2).
Change subject: [ksck] report on misconfiguration for flag categories
......................................................................
[ksck] report on misconfiguration for flag categories
This patch adds functionality to report on misconfiguration of flags
in pre-defined flag categories. The 'misconfiguration' means different
values for same flags among all masters and tablet servers in a Kudu
cluster, and differences between masters' and tablet servers' settings
for same flags.
With this patch, two categories are defined and ready for use:
* 'time_source' includes the following flags
** --builtin_ntp_servers
** --time_source
* 'unusual' includes all experimental, hidden, and unsafe flags
To use the newly introduced functionality, add
--flags_categories_to_check=<comma-separated-list-of-flags-categories>
to the 'kudu cluster ksck' invocation.
For testing, I ran a small Kudu cluster and verified that ksck reports
on misconfiguration as expected:
* if tablet servers are run with different flags
* if masters are run with flag settings different from those used for
tablet servers
This patch also contains assorted re-factoring in related areas
of code. I'm planning to add automated tests to cover the newly
introduced functionality in a separate patch.
Change-Id: I2afbfac9327f85b212ecf8d8f43a2139f90db6bb
---
M src/kudu/rebalance/cluster_status.cc
M src/kudu/rebalance/cluster_status.h
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_remote.h
M src/kudu/tools/ksck_results.cc
M src/kudu/tools/ksck_results.h
M src/kudu/tools/tool_action_cluster.cc
10 files changed, 592 insertions(+), 165 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/98/15298/2
--
To view, visit http://gerrit.cloudera.org:8080/15298
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I2afbfac9327f85b212ecf8d8f43a2139f90db6bb
Gerrit-Change-Number: 15298
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Volodymyr Verovkin <[email protected]>
