You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by sr...@apache.org on 2016/06/13 19:46:33 UTC

ambari git commit: AMBARI-17189. Change in Atlas authorization from class based to value based (Gautam Borad via srimanth)

Repository: ambari
Updated Branches:
  refs/heads/trunk 3935198af -> 98873150a


AMBARI-17189. Change in Atlas authorization from class based to value based (Gautam Borad via srimanth)


Project: http://git-wip-us.apache.org/repos/asf/ambari/repo
Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/98873150
Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/98873150
Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/98873150

Branch: refs/heads/trunk
Commit: 98873150a97bd66a72905ba435f7d2420b2c763a
Parents: 3935198
Author: Srimanth Gunturi <sg...@hortonworks.com>
Authored: Mon Jun 13 12:46:16 2016 -0700
Committer: Srimanth Gunturi <sg...@hortonworks.com>
Committed: Mon Jun 13 12:46:16 2016 -0700

----------------------------------------------------------------------
 .../main/resources/stacks/HDP/2.5/services/stack_advisor.py   | 4 ++--
 .../src/test/python/stacks/2.5/common/test_stack_advisor.py   | 7 ++++---
 2 files changed, 6 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/98873150/ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py
----------------------------------------------------------------------
diff --git a/ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py b/ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py
index 1bcc09e..db2986b 100644
--- a/ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py
+++ b/ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py
@@ -341,9 +341,9 @@ class HDP25StackAdvisor(HDP24StackAdvisor):
       ranger_atlas_plugin_enabled = services['configurations']['ranger-atlas-plugin-properties']['properties']['ranger-atlas-plugin-enabled']
 
     if ranger_atlas_plugin_enabled and (ranger_atlas_plugin_enabled.lower() == 'Yes'.lower()):
-      putAtlasApplicationProperty('atlas.authorizer.impl','org.apache.ranger.authorization.atlas.authorizer.RangerAtlasAuthorizer')
+      putAtlasApplicationProperty('atlas.authorizer.impl','ranger')
     else:
-      putAtlasApplicationProperty('atlas.authorizer.impl','org.apache.atlas.authorize.SimpleAtlasAuthorizer')
+      putAtlasApplicationProperty('atlas.authorizer.impl','simple')
 
   def recommendHBASEConfigurations(self, configurations, clusterData, services, hosts):
     super(HDP25StackAdvisor, self).recommendHBASEConfigurations(configurations, clusterData, services, hosts)

http://git-wip-us.apache.org/repos/asf/ambari/blob/98873150/ambari-server/src/test/python/stacks/2.5/common/test_stack_advisor.py
----------------------------------------------------------------------
diff --git a/ambari-server/src/test/python/stacks/2.5/common/test_stack_advisor.py b/ambari-server/src/test/python/stacks/2.5/common/test_stack_advisor.py
index f8803e4..4085ad6 100644
--- a/ambari-server/src/test/python/stacks/2.5/common/test_stack_advisor.py
+++ b/ambari-server/src/test/python/stacks/2.5/common/test_stack_advisor.py
@@ -6428,7 +6428,7 @@ class TestHDP25StackAdvisor(TestCase):
           "atlas.kafka.zookeeper.connect": "c6401.ambari.apache.org",
           'atlas.server.address.id1': "c6401.ambari.apache.org:21000",
           'atlas.server.ids': "id1",
-          'atlas.authorizer.impl':'org.apache.ranger.authorization.atlas.authorizer.RangerAtlasAuthorizer'
+          'atlas.authorizer.impl':'ranger'
         }
       },
       "logsearch-solr-env": {
@@ -6622,10 +6622,11 @@ class TestHDP25StackAdvisor(TestCase):
 
     self.stackAdvisor.recommendAtlasConfigurations(configurations, clusterData, services, hosts)
     # test for Ranger Atlas plugin disabled
-    self.assertEquals(configurations['application-properties']['properties']['atlas.authorizer.impl'], 'org.apache.atlas.authorize.SimpleAtlasAuthorizer', 'Test atlas.authorizer.impl with Ranger Atlas plugin is disabled ')
+    self.assertEquals(configurations['application-properties']['properties']['atlas.authorizer.impl'], 'simple', 'Test atlas.authorizer.impl with Ranger Atlas plugin is disabled ')
 
     configurations['ranger-atlas-plugin-properties']['properties']['ranger-atlas-plugin-enabled'] = 'Yes'
-    configurations['application-properties']['properties']['atlas.authorizer.impl'] =  'org.apache.ranger.authorization.atlas.authorizer.RangerAtlasAuthorizer'
+    # configurations['application-properties']['properties']['atlas.authorizer.impl'] =  'ranger'
+    self.stackAdvisor.recommendAtlasConfigurations(configurations,clusterData,services,hosts)
     self.assertEquals(configurations, expected)
 
     services['ambari-server-properties'] = {'java.home': '/usr/jdk64/jdk1.7.3_23'}