AMBARI-1334. Show hosts that have failed install tasks as red to allow the user to easily identify source of failure. (yusaku)

git-svn-id: https://svn.apache.org/repos/asf/incubator/ambari/trunk@1441687 13f79535-47bb-0310-9956-ffa450edef68

CHANGES.txt

@@ -38,6 +38,9 @@ Trunk (unreleased changes):
 
  IMPROVEMENTS
 
+ AMBARI-1334. Show hosts that have failed install tasks as "red" to allow the user
+ to easily identify source of failure. (yusaku)
+
  AMBARI-1333. Add username validation for Ambari local users. (yusaku) 
 
  AMBARI-1329. Adjust job browser column sizing. (yusaku)

ambari-web/app/controllers/wizard/step9_controller.js

@@ -338,7 +338,7 @@ App.WizardStep9Controller = Em.Controller.extend({
     if (actions.someProperty('Tasks.status', 'FAILED') || actions.someProperty('Tasks.status', 'ABORTED') || actions.someProperty('Tasks.status', 'TIMEDOUT')) {
       contentHost.set('status', 'warning');
     }
-    if (this.get('content.cluster.status') === 'PENDING' && this.isMasterFailed(actions)) {
+    if (this.get('content.cluster.status') === 'PENDING' && actions.someProperty('Tasks.status', 'FAILED')) {
       contentHost.set('status', 'failed');
     }
   },
@@ -405,19 +405,6 @@ App.WizardStep9Controller = Em.Controller.extend({
     return failed;
   },
 
-  //return true if there is at least one FAILED task of master component install
-  isMasterFailed: function(polledData) {
-    var result = false;
-    polledData.filterProperty('Tasks.status', 'FAILED').mapProperty('Tasks.role').forEach (
-        function (task) {
-          if (!['DATANODE', 'TASKTRACKER', 'HBASE_REGIONSERVER', 'GANGLIA_MONITOR'].contains(task)) {
-            result = true;
-          }
-        }
-    );
-    return result;
-  },
-
   // makes a state transition
   // PENDING -> INSTALLED
   // PENDING -> INSTALL FAILED