Attila Bukor has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15681 )
Change subject: KUDU-3078 Add Ranger tests to master_authz-itest ...................................................................... KUDU-3078 Add Ranger tests to master_authz-itest This commit refactors the existing Sentry integration tests to prepare for Ranger integration tests. It changes HMS and Sentry integration test base classes to test harnesses that don't inherit from Gtest to simplify inheritance when using typed tests. master_sentry-itest is renamed to master_authz-itest to generalize it, and most tests are changed to also run with Ranger. Those that aren't test behavior specific to Sentry. A later patch will do the same for ts_sentry-itest. 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). Change-Id: I25dc67516cd61f0624914989f8db4c4f94d7e3bf Reviewed-on: http://gerrit.cloudera.org:8080/15681 Tested-by: Kudu Jenkins Reviewed-by: Andrew Wong <[email protected]> --- M src/kudu/integration-tests/CMakeLists.txt M src/kudu/integration-tests/hms_itest-base.cc M src/kudu/integration-tests/hms_itest-base.h A src/kudu/integration-tests/master_authz-itest.cc M src/kudu/integration-tests/master_hms-itest.cc D src/kudu/integration-tests/master_sentry-itest.cc M src/kudu/integration-tests/ts_sentry-itest.cc M src/kudu/ranger/mini_ranger-test.cc M src/kudu/ranger/mini_ranger.cc M src/kudu/ranger/mini_ranger.h 10 files changed, 1,881 insertions(+), 1,303 deletions(-) Approvals: Kudu Jenkins: Verified Andrew Wong: Looks good to me, approved -- 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: merged Gerrit-Change-Id: I25dc67516cd61f0624914989f8db4c4f94d7e3bf Gerrit-Change-Number: 15681 Gerrit-PatchSet: 18 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)
