Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, Grant Henke, Hao Hao,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/15681
to look at the new patch set (#8).
Change subject: KUDU-3078 Add Ranger tests to master_authz-itest
......................................................................
KUDU-3078 Add Ranger tests to master_authz-itest
master_sentry-itest is renamed to master_authz-itest to generalize it
and the tests were changed to also run with Ranger.
Ranger doesn't support adding new policy items (users and privileges) to
existing policies, so MiniRanger::AddPolicy stores policies in a member
variable and recreates them completely when a new item is added to an
existing policy (resource). This broke an existing test in
mini_ranger-test which was testing persistence by expecting an error
after adding an existing policy after restarting Ranger, this test is
fetching the number of policies now instead.
These new integration tests uncovered a previously unknown bug in Ranger
authorization provider and Ranger client which is also fixed now.
Change-Id: I25dc67516cd61f0624914989f8db4c4f94d7e3bf
---
M src/kudu/integration-tests/CMakeLists.txt
R src/kudu/integration-tests/master_authz-itest.cc
M src/kudu/master/ranger_authz_provider.cc
M src/kudu/ranger/mini_ranger-test.cc
M src/kudu/ranger/mini_ranger.cc
M src/kudu/ranger/mini_ranger.h
M src/kudu/ranger/ranger_client-test.cc
M src/kudu/ranger/ranger_client.cc
M src/kudu/ranger/ranger_client.h
9 files changed, 637 insertions(+), 244 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/81/15681/8
--
To view, visit http://gerrit.cloudera.org:8080/15681
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I25dc67516cd61f0624914989f8db4c4f94d7e3bf
Gerrit-Change-Number: 15681
Gerrit-PatchSet: 8
Gerrit-Owner: Attila Bukor <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Hao Hao <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
