Repository: ambari
Updated Branches:
  refs/heads/trunk 5718670c2 -> 1f6a8f710


AMBARI-17861. Ambari principal should be part of nimbus.admins for Storm View - 
addendum (Sriharsha Chintalapani via smohanty)


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

Branch: refs/heads/trunk
Commit: 1f6a8f710bcc99f195475f1a85726e394bab766e
Parents: 5718670
Author: Sumit Mohanty <[email protected]>
Authored: Sat Jul 23 16:08:12 2016 -0700
Committer: Sumit Mohanty <[email protected]>
Committed: Sat Jul 23 16:08:12 2016 -0700

----------------------------------------------------------------------
 .../STORM/0.9.1/package/scripts/params_linux.py                | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/1f6a8f71/ambari-server/src/main/resources/common-services/STORM/0.9.1/package/scripts/params_linux.py
----------------------------------------------------------------------
diff --git 
a/ambari-server/src/main/resources/common-services/STORM/0.9.1/package/scripts/params_linux.py
 
b/ambari-server/src/main/resources/common-services/STORM/0.9.1/package/scripts/params_linux.py
index 1b7446b..4fa318b 100644
--- 
a/ambari-server/src/main/resources/common-services/STORM/0.9.1/package/scripts/params_linux.py
+++ 
b/ambari-server/src/main/resources/common-services/STORM/0.9.1/package/scripts/params_linux.py
@@ -144,8 +144,7 @@ if security_enabled:
   _hostname_lowercase = config['hostname'].lower()
   _storm_principal_name = 
config['configurations']['storm-env']['storm_principal_name']
   storm_jaas_principal = 
_storm_principal_name.replace('_HOST',_hostname_lowercase)
-  _ambari_principal_name = 
config['configurations']['cluster-env']['ambari_principal_name']
-  ambari_bare_jaas_principal = get_bare_principal(_ambari_principal_name)
+  _ambari_principal_name = 
default('/configurations/cluster-env/ambari_principal_name', None)
   storm_keytab_path = config['configurations']['storm-env']['storm_keytab']
 
   if stack_supports_storm_kerberos:
@@ -153,7 +152,8 @@ if security_enabled:
     _storm_ui_jaas_principal_name = 
config['configurations']['storm-env']['storm_ui_principal_name']
     storm_ui_jaas_principal = 
_storm_ui_jaas_principal_name.replace('_HOST',_hostname_lowercase)
     storm_bare_jaas_principal = get_bare_principal(_storm_principal_name)
-    ambari_bare_jaas_principal = get_bare_principal(_ambari_principal_name)
+    if _ambari_principal_name:
+      ambari_bare_jaas_principal = get_bare_principal(_ambari_principal_name)
     _nimbus_principal_name = 
config['configurations']['storm-env']['nimbus_principal_name']
     nimbus_jaas_principal = _nimbus_principal_name.replace('_HOST', 
_hostname_lowercase)
     nimbus_bare_jaas_principal = get_bare_principal(_nimbus_principal_name)

Reply via email to