Hello Kudu Jenkins, Andrew Wong, 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 (#5).
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 in each group of masters/tablet servers in a
cluster (i.e. intra-group differences), and also differences between
masters' and tablet servers' flags (i.e. cross-group differences).
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
The newly introduced functionality is enabled by default for the
'time_source' flags category. The list of group categories to check
can be customized by adding
--flags_categories_to_check=<comma-separated-list-of-flags-categories>
to the 'kudu cluster ksck' invocation. Use an empty string to specify
an empty list of flags categories to check.
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, 582 insertions(+), 174 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/98/15298/5
--
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: 5
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Volodymyr Verovkin <[email protected]>
