ambari-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ababiic...@apache.org
Subject ambari git commit: AMBARI-13656 oozie service check failing after reassigning host with custom service user. (ababiichuk)
Date Fri, 30 Oct 2015 13:04:40 GMT
Repository: ambari
Updated Branches:
  refs/heads/branch-2.1 d1c068e3f -> 74eb8d1de


AMBARI-13656 oozie service check failing after reassigning host with custom service user.
(ababiichuk)


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

Branch: refs/heads/branch-2.1
Commit: 74eb8d1de9114adea154bdcc7ec9e85d1b299444
Parents: d1c068e
Author: aBabiichuk <ababiichuk@cybervisiontech.com>
Authored: Fri Oct 30 15:01:23 2015 +0200
Committer: aBabiichuk <ababiichuk@cybervisiontech.com>
Committed: Fri Oct 30 15:04:26 2015 +0200

----------------------------------------------------------------------
 .../main/service/reassign/step4_controller.js   | 38 ++++++++++++++------
 .../service/reassign/step4_controller_test.js   |  4 ++-
 .../main/service/reassign_controller_test.js    |  1 +
 3 files changed, 32 insertions(+), 11 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/74eb8d1d/ambari-web/app/controllers/main/service/reassign/step4_controller.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/controllers/main/service/reassign/step4_controller.js b/ambari-web/app/controllers/main/service/reassign/step4_controller.js
index dffdf9e..f197d84 100644
--- a/ambari-web/app/controllers/main/service/reassign/step4_controller.js
+++ b/ambari-web/app/controllers/main/service/reassign/step4_controller.js
@@ -543,6 +543,7 @@ App.ReassignMasterWizardStep4Controller = App.HighAvailabilityProgressPageContro
       case 'OOZIE_SERVER':
         urlParams.push('(type=oozie-site&tag=' + data.Clusters.desired_configs['oozie-site'].tag
+ ')');
         urlParams.push('(type=core-site&tag=' + data.Clusters.desired_configs['core-site'].tag
+ ')');
+        urlParams.push('(type=oozie-env&tag=' + data.Clusters.desired_configs['oozie-env'].tag
+ ')');
         break;
       case 'HIVE_SERVER':
       case 'HIVE_METASTORE':
@@ -585,16 +586,19 @@ App.ReassignMasterWizardStep4Controller = App.HighAvailabilityProgressPageContro
     this.setAdditionalConfigs(configs, componentName, targetHostName);
     this.setSecureConfigs(secureConfigs, configs, componentName);
 
-    if (componentName === 'NAMENODE') {
-      this.setSpecificNamenodeConfigs(configs, targetHostName);
-    }
-
-    if (componentName === 'RESOURCEMANAGER') {
-      this.setSpecificResourceMangerConfigs(configs, targetHostName);
-    }
-
-    if (componentName === 'HIVE_METASTORE' || componentName === 'HIVE_SERVER') {
-      this.setSpecificHiveConfigs(configs, targetHostName);
+    switch (componentName) {
+      case 'NAMENODE':
+        this.setSpecificNamenodeConfigs(configs, targetHostName);
+        break;
+      case 'RESOURCEMANAGER':
+        this.setSpecificResourceMangerConfigs(configs, targetHostName);
+        break;
+      case 'HIVE_METASTORE':
+      case 'HIVE_SERVER':
+        this.setSpecificHiveConfigs(configs, targetHostName);
+        break;
+      case 'OOZIE_SERVER':
+        this.setSpecificOozieConfigs(configs, targetHostName);
     }
 
     this.saveClusterStatus(secureConfigs, this.getComponentDir(configs, componentName));
@@ -742,6 +746,20 @@ App.ReassignMasterWizardStep4Controller = App.HighAvailabilityProgressPageContro
   },
 
   /**
+   * set specific configs which applies only to Oozie related configs
+   * @param configs
+   * @param targetHostName
+   */
+  setSpecificOozieConfigs: function (configs, targetHostName) {
+    var sourceHostName = this.get('content.reassignHosts.source'),
+      oozieServerHosts = App.HostComponent.find().filterProperty('componentName', 'OOZIE_SERVER')
+        .mapProperty('hostName').removeObject(sourceHostName).addObject(targetHostName).uniq().join(','),
+      oozieUser = configs['oozie-env']['oozie_user'];
+
+    configs['core-site']['hadoop.proxyuser.' + oozieUser + '.hosts'] = oozieServerHosts;
+  },
+
+  /**
    * set specific configs which applies only to Hive related configs
    * @param configs
    * @param targetHostName

http://git-wip-us.apache.org/repos/asf/ambari/blob/74eb8d1d/ambari-web/test/controllers/main/service/reassign/step4_controller_test.js
----------------------------------------------------------------------
diff --git a/ambari-web/test/controllers/main/service/reassign/step4_controller_test.js b/ambari-web/test/controllers/main/service/reassign/step4_controller_test.js
index 7a7b576..f0e845c 100644
--- a/ambari-web/test/controllers/main/service/reassign/step4_controller_test.js
+++ b/ambari-web/test/controllers/main/service/reassign/step4_controller_test.js
@@ -490,7 +490,8 @@ describe('App.ReassignMasterWizardStep4Controller', function () {
         componentName: 'OOZIE_SERVER',
         result: [
           "(type=oozie-site&tag=6)",
-          "(type=core-site&tag=2)"
+          "(type=core-site&tag=2)",
+          "(type=oozie-env&tag=2)"
         ]
       },
       {
@@ -510,6 +511,7 @@ describe('App.ReassignMasterWizardStep4Controller', function () {
           'mapred-site': {tag: 4},
           'yarn-site': {tag: 5},
           'oozie-site': {tag: 6},
+          'oozie-env': {tag: 2},
           'webhcat-site': {tag: 7},
           'yarn-env': {tag: 8},
           'accumulo-site': {tag: 9}

http://git-wip-us.apache.org/repos/asf/ambari/blob/74eb8d1d/ambari-web/test/controllers/main/service/reassign_controller_test.js
----------------------------------------------------------------------
diff --git a/ambari-web/test/controllers/main/service/reassign_controller_test.js b/ambari-web/test/controllers/main/service/reassign_controller_test.js
index 1745723..0848938 100644
--- a/ambari-web/test/controllers/main/service/reassign_controller_test.js
+++ b/ambari-web/test/controllers/main/service/reassign_controller_test.js
@@ -54,6 +54,7 @@ describe('App.ReassignMasterController', function () {
       it('check ' + c.componentName, function () {
         reassignMasterController.set('content.reassign', {'component_name': c.componentName});
         expect(reassignMasterController.get('totalSteps')).to.equal(c.result);
+        reassignMasterController.set('content.reassign', {service_id:null});
       });
     });
   });


Mime
View raw message