Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13139 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/13139 Reviewed-by: Andrew Wong <aw...@cloudera.com> Reviewed-by: Hao Hao <hao....@cloudera.com> Tested-by: Alexey Serbin <aser...@cloudera.com> --- 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(-) Approvals: Andrew Wong: Looks good to me, approved Hao Hao: Looks good to me, approved Alexey Serbin: Verified -- 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: merged Gerrit-Change-Id: If49f3ebdea22ea0df3aaec313b4db949dc834943 Gerrit-Change-Number: 13139 Gerrit-PatchSet: 8 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>
