Hello Kudu Jenkins, Andrew Wong, Adar Dembo, Hao Hao,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/13139
to look at the new patch set (#7).
Change subject: [tools] command to reset authz cache in kudu CLI
......................................................................
[tools] command to reset authz cache in kudu CLI
Added command to reset authz privileges cache into kudu CLI.
Synopsis:
kudu master authz_cache reset <all_master_addresses_in_cluster>
The tool requires all masters' end-points to be specified in the command
line. This is to keep things as consistent as possible, since resetting
the authz privileges cache only on a subset of masters could lead to
unexpected surprises upon master leadership change. It's possible
to reset authz privileges cache at any subset of masters in a Kudu
cluster adding --force flag. For example, to reset authz cache only
at one master out of three in multi-master Kudu cluster, run
kudu master authz_cache reset --force <master_address>
Added corresponding tests as well. A scenario for a 'positive' case
is not among the added ones: it will be a separate effort to bring in
HMS+Sentry into the framework for testing kudu CLI tool. However,
there is coverage for a 'positive' case for cache reset elsewhere.
In particular, the scenarios of the SentryAuthzProviderCacheITest
give appropriate coverage using the ResetAuthzCache RPC endpoint.
Those scenarios test and AdminCliTest.TestAuthzResetCacheNotImplemented
should provide enough coverage.
Change-Id: If49f3ebdea22ea0df3aaec313b4db949dc834943
---
M src/kudu/tools/kudu-admin-test.cc
M src/kudu/tools/kudu-tool-test.cc
M src/kudu/tools/tool_action_master.cc
3 files changed, 323 insertions(+), 69 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/39/13139/7
--
To view, visit http://gerrit.cloudera.org:8080/13139
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: If49f3ebdea22ea0df3aaec313b4db949dc834943
Gerrit-Change-Number: 13139
Gerrit-PatchSet: 7
Gerrit-Owner: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Hao Hao <hao....@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
