Browse Source

AMBARI-6350 Unable to restart all host-components on 110 node cluster. (Buzhor Denys via atkach)

atkach 11 years ago
parent
commit
b7cd438a51

+ 6 - 10
ambari-web/app/utils/batch_scheduled_requests.js

@@ -218,8 +218,8 @@ module.exports = {
       }
     }
     if (hostComponentsList.length > 0) {
-      var operation_level = this.getOperationLevelobject(level, hosts.uniq().join(","),
-        componentServiceMap[hostComponentsList[0].get("componentName")], hostComponentsList[0].get("componentName"));
+      var operation_level = this.getOperationLevelobject(level, componentServiceMap[hostComponentsList[0].get("componentName")],
+        hostComponentsList[0].get("componentName"));
     }
 
 
@@ -244,23 +244,19 @@ module.exports = {
 
   /**
    * @param {String} level - operation level name, can be ("CLUSTER", "SERVICE", "HOST", "HOSTCOMPONENT")
-   * @param {String} hostName get host name or hostNames as String("host1,host2")
    * @param {String} serviceName
    * @param {String} componentName
    * @returns {Object} {{level: *, cluster_name: *}} - operation level object
    * @method getOperationLevelobject - create operation level object to be included into ajax query
    */
-  getOperationLevelobject: function(level, hostName, serviceName, componentName) {
+  getOperationLevelobject: function(level, serviceName, componentName) {
     var operationLevel = {
       "level": level,
       "cluster_name": App.get("clusterName")
     };
-    if (level === "HOST") {
-      operationLevel["host_name"] = hostName;
-    } else if (level === "SERVICE") {
+    if (level === "SERVICE") {
       operationLevel["service_name"] = serviceName;
-    } else {
-      operationLevel["host_name"] = hostName;
+    } else if(level !== "HOST") {
       operationLevel["service_name"] = serviceName;
       operationLevel["hostcomponent_name"] = componentName;
     }
@@ -572,4 +568,4 @@ module.exports = {
       successCallback({});
     }
   }
-};
+};

+ 1 - 1
ambari-web/test/controllers/main/host_test.js

@@ -303,7 +303,7 @@ describe('MainHostController', function () {
 
   });
 
-  describe.only('#getRegExp()', function() {
+  describe('#getRegExp()', function() {
     before(function() {
       hostController = App.MainHostController.create({});
     });