svn commit: r1339761 - in /incubator/ambari/branches/ambari-186: CHANGES.txt hmc/js/manageServices.js

[ra...@apache.org]

Author: ramya
Date: Thu May 17 18:08:04 2012
New Revision: 1339761

URL: http://svn.apache.org/viewvc?rev=1339761&view=rev
Log:
AMBARI-258. Start/Stop service show services that are not dependent on the service being worked on

Modified:
    incubator/ambari/branches/ambari-186/CHANGES.txt
    incubator/ambari/branches/ambari-186/hmc/js/manageServices.js

Modified: incubator/ambari/branches/ambari-186/CHANGES.txt
URL: http://svn.apache.org/viewvc/incubator/ambari/branches/ambari-186/CHANGES.txt?rev=1339761&r1=1339760&r2=1339761&view=diff
==============================================================================
--- incubator/ambari/branches/ambari-186/CHANGES.txt (original)
+++ incubator/ambari/branches/ambari-186/CHANGES.txt Thu May 17 18:08:04 2012
@@ -2,6 +2,9 @@ Ambari Change log
 
 Release 0.x.x - unreleased
 
+  AMBARI-258. Start/Stop service show services that are not dependent on the 
+  service being worked on (Vinod via ramya)
+
   AMBARI-251. Oozie link is not displayed even when Oozie is installed 
   (vgogate via ramya)
 

Modified: incubator/ambari/branches/ambari-186/hmc/js/manageServices.js
URL: http://svn.apache.org/viewvc/incubator/ambari/branches/ambari-186/hmc/js/manageServices.js?rev=1339761&r1=1339760&r2=1339761&view=diff
==============================================================================
--- incubator/ambari/branches/ambari-186/hmc/js/manageServices.js (original)
+++ incubator/ambari/branches/ambari-186/hmc/js/manageServices.js Thu May 17 18:08:04 2012
@@ -208,7 +208,7 @@ function getServiceConfigurationMarkup( 
 
 function serviceManagementActionClickHandler( action, serviceName ) {
 
-  var affectedServices;
+  var affectedServices = [];
 
   var confirmationDataPanelTitle = '';
   var confirmationDataPanelBodyContent = '';
@@ -302,7 +302,19 @@ function serviceManagementActionClickHan
     confirmationDataPanelBodyContent = "We are now going to stop all the services in the cluster";
   }
 
+  // Add the list of dependencies
   if(action =='start' || action == 'stop') {
+
+    // Clean up the affected-services list to only include appropriate installed long-running services
+    var deps = affectedServices;
+    affectedServices = [];
+    for (dep in deps) {
+      var svc = deps[dep];
+      if (clusterServices.hasOwnProperty(svc) && (clusterServices[svc].isEnabled == 1) && clusterServices[svc].attributes.runnable ) {
+        affectedServices.push(svc);
+      }
+    }
+
     var dependencyMarkup = "";
     for (affectedSrvc in affectedServices) {
       if (clusterServices[affectedServices[affectedSrvc]].attributes.runnable) {