Repository: ambari
Updated Branches:
  refs/heads/branch-2.2 86bb1415b -> d50d4a845


AMBARI-15295. HBase RS decommission issue in secure cluster.(vbrodetskyi)


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

Branch: refs/heads/branch-2.2
Commit: d50d4a8458abdcce8c0cff1261f93cb4a7fbf60b
Parents: 86bb141
Author: Vitaly Brodetskyi <[email protected]>
Authored: Wed Mar 9 13:28:54 2016 +0200
Committer: Vitaly Brodetskyi <[email protected]>
Committed: Wed Mar 9 13:28:54 2016 +0200

----------------------------------------------------------------------
 .../0.96.0.2.0/package/scripts/hbase_decommission.py  |  6 +++---
 .../HBASE/0.96.0.2.0/package/scripts/params_linux.py  |  2 ++
 .../python/stacks/2.0.6/HBASE/test_hbase_master.py    | 14 +++++++-------
 3 files changed, 12 insertions(+), 10 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/d50d4a84/ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/hbase_decommission.py
----------------------------------------------------------------------
diff --git 
a/ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/hbase_decommission.py
 
b/ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/hbase_decommission.py
index ca7e1ef..022465a 100644
--- 
a/ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/hbase_decommission.py
+++ 
b/ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/hbase_decommission.py
@@ -63,7 +63,7 @@ def hbase_decommission(env):
     for host in hosts:
       if host:
         regiondrainer_cmd = format(
-          "{kinit_cmd} {hbase_cmd} --config {hbase_conf_dir} org.jruby.Main 
{region_drainer} remove {host}")
+          "{kinit_cmd} {hbase_cmd} --config {hbase_conf_dir} 
{master_security_config} org.jruby.Main {region_drainer} remove {host}")
         Execute(regiondrainer_cmd,
                 user=params.hbase_user,
                 logoutput=True
@@ -75,9 +75,9 @@ def hbase_decommission(env):
     for host in hosts:
       if host:
         regiondrainer_cmd = format(
-          "{kinit_cmd} {hbase_cmd} --config {hbase_conf_dir} org.jruby.Main 
{region_drainer} add {host}")
+          "{kinit_cmd} {hbase_cmd} --config {hbase_conf_dir} 
{master_security_config} org.jruby.Main {region_drainer} add {host}")
         regionmover_cmd = format(
-          "{kinit_cmd} {hbase_cmd} --config {hbase_conf_dir} org.jruby.Main 
{region_mover} unload {host}")
+          "{kinit_cmd} {hbase_cmd} --config {hbase_conf_dir} 
{master_security_config} org.jruby.Main {region_mover} unload {host}")
 
         Execute(regiondrainer_cmd,
                 user=params.hbase_user,

http://git-wip-us.apache.org/repos/asf/ambari/blob/d50d4a84/ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/params_linux.py
----------------------------------------------------------------------
diff --git 
a/ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/params_linux.py
 
b/ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/params_linux.py
index 04ab4c8..7e7a070 100644
--- 
a/ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/params_linux.py
+++ 
b/ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/params_linux.py
@@ -200,9 +200,11 @@ kinit_path_local = 
get_kinit_path(default('/configurations/kerberos-env/executab
 if security_enabled:
   kinit_cmd = format("{kinit_path_local} -kt {hbase_user_keytab} 
{hbase_principal_name};")
   kinit_cmd_master = format("{kinit_path_local} -kt {master_keytab_path} 
{master_jaas_princ};")
+  master_security_config = 
format("-Djava.security.auth.login.config={hbase_conf_dir}/hbase_master_jaas.conf")
 else:
   kinit_cmd = ""
   kinit_cmd_master = ""
+  master_security_config = ""
 
 #log4j.properties
 if (('hbase-log4j' in config['configurations']) and ('content' in 
config['configurations']['hbase-log4j'])):

http://git-wip-us.apache.org/repos/asf/ambari/blob/d50d4a84/ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_master.py
----------------------------------------------------------------------
diff --git 
a/ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_master.py 
b/ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_master.py
index 8de7f96..6a4a2ef 100644
--- a/ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_master.py
+++ b/ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_master.py
@@ -123,19 +123,19 @@ class TestHBaseMaster(RMFTestCase):
                               content = StaticFile('draining_servers.rb'),
                               mode = 0755,
                               )
-    self.assertResourceCalled('Execute', ' /usr/lib/hbase/bin/hbase --config 
/etc/hbase/conf org.jruby.Main /usr/lib/hbase/bin/draining_servers.rb add 
host1',
+    self.assertResourceCalled('Execute', ' /usr/lib/hbase/bin/hbase --config 
/etc/hbase/conf  org.jruby.Main /usr/lib/hbase/bin/draining_servers.rb add 
host1',
                               logoutput = True,
                               user = 'hbase',
                               )
-    self.assertResourceCalled('Execute', ' /usr/lib/hbase/bin/hbase --config 
/etc/hbase/conf org.jruby.Main /usr/lib/hbase/bin/region_mover.rb unload host1',
+    self.assertResourceCalled('Execute', ' /usr/lib/hbase/bin/hbase --config 
/etc/hbase/conf  org.jruby.Main /usr/lib/hbase/bin/region_mover.rb unload 
host1',
                               logoutput = True,
                               user = 'hbase',
                               )
-    self.assertResourceCalled('Execute', ' /usr/lib/hbase/bin/hbase --config 
/etc/hbase/conf org.jruby.Main /usr/lib/hbase/bin/draining_servers.rb add 
host2',
+    self.assertResourceCalled('Execute', ' /usr/lib/hbase/bin/hbase --config 
/etc/hbase/conf  org.jruby.Main /usr/lib/hbase/bin/draining_servers.rb add 
host2',
                               logoutput = True,
                               user = 'hbase',
                               )
-    self.assertResourceCalled('Execute', ' /usr/lib/hbase/bin/hbase --config 
/etc/hbase/conf org.jruby.Main /usr/lib/hbase/bin/region_mover.rb unload host2',
+    self.assertResourceCalled('Execute', ' /usr/lib/hbase/bin/hbase --config 
/etc/hbase/conf  org.jruby.Main /usr/lib/hbase/bin/region_mover.rb unload 
host2',
                               logoutput = True,
                               user = 'hbase',
                               )
@@ -154,7 +154,7 @@ class TestHBaseMaster(RMFTestCase):
                               content = StaticFile('draining_servers.rb'),
                               mode = 0755,
                               )
-    self.assertResourceCalled('Execute', ' /usr/lib/hbase/bin/hbase --config 
/etc/hbase/conf org.jruby.Main /usr/lib/hbase/bin/draining_servers.rb remove 
host1',
+    self.assertResourceCalled('Execute', ' /usr/lib/hbase/bin/hbase --config 
/etc/hbase/conf  org.jruby.Main /usr/lib/hbase/bin/draining_servers.rb remove 
host1',
                               logoutput = True,
                               user = 'hbase',
                               )
@@ -222,11 +222,11 @@ class TestHBaseMaster(RMFTestCase):
                               content = StaticFile('draining_servers.rb'),
                               mode = 0755,
                               )
-    self.assertResourceCalled('Execute', '/usr/bin/kinit -kt 
/etc/security/keytabs/hbase.service.keytab 
hbase/[email protected]; /usr/lib/hbase/bin/hbase --config 
/etc/hbase/conf org.jruby.Main /usr/lib/hbase/bin/draining_servers.rb add 
host1',
+    self.assertResourceCalled('Execute', '/usr/bin/kinit -kt 
/etc/security/keytabs/hbase.service.keytab 
hbase/[email protected]; /usr/lib/hbase/bin/hbase --config 
/etc/hbase/conf 
-Djava.security.auth.login.config=/etc/hbase/conf/hbase_master_jaas.conf 
org.jruby.Main /usr/lib/hbase/bin/draining_servers.rb add host1',
                               logoutput = True,
                               user = 'hbase',
                               )
-    self.assertResourceCalled('Execute', '/usr/bin/kinit -kt 
/etc/security/keytabs/hbase.service.keytab 
hbase/[email protected]; /usr/lib/hbase/bin/hbase --config 
/etc/hbase/conf org.jruby.Main /usr/lib/hbase/bin/region_mover.rb unload host1',
+    self.assertResourceCalled('Execute', '/usr/bin/kinit -kt 
/etc/security/keytabs/hbase.service.keytab 
hbase/[email protected]; /usr/lib/hbase/bin/hbase --config 
/etc/hbase/conf 
-Djava.security.auth.login.config=/etc/hbase/conf/hbase_master_jaas.conf 
org.jruby.Main /usr/lib/hbase/bin/region_mover.rb unload host1',
                               logoutput = True,
                               user = 'hbase',
                               )

Reply via email to