You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@whirr.apache.org by as...@apache.org on 2011/04/15 11:38:49 UTC

svn commit: r1092642 - in /incubator/whirr/trunk: CHANGES.txt core/src/main/java/org/apache/whirr/actions/ScriptBasedClusterAction.java

Author: asavu
Date: Fri Apr 15 09:38:49 2011
New Revision: 1092642

URL: http://svn.apache.org/viewvc?rev=1092642&view=rev
Log:
WHIRR-284. Runurl should only be installed when needed (tomwhite via asavu)

Modified:
    incubator/whirr/trunk/CHANGES.txt
    incubator/whirr/trunk/core/src/main/java/org/apache/whirr/actions/ScriptBasedClusterAction.java

Modified: incubator/whirr/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/incubator/whirr/trunk/CHANGES.txt?rev=1092642&r1=1092641&r2=1092642&view=diff
==============================================================================
--- incubator/whirr/trunk/CHANGES.txt (original)
+++ incubator/whirr/trunk/CHANGES.txt Fri Apr 15 09:38:49 2011
@@ -33,6 +33,8 @@ Trunk (unreleased changes)
 
     WHIRR-282. Set number of Hadoop slots based on hardware (tomwhite via asavu)
 
+    WHIRR-284. Runurl should only be installed when needed (tomwhite via asavu)
+
   BUG FIXES
 
     WHIRR-253. ZooKeeper service should only authorize ingress to ZooKeeper 

Modified: incubator/whirr/trunk/core/src/main/java/org/apache/whirr/actions/ScriptBasedClusterAction.java
URL: http://svn.apache.org/viewvc/incubator/whirr/trunk/core/src/main/java/org/apache/whirr/actions/ScriptBasedClusterAction.java?rev=1092642&r1=1092641&r2=1092642&view=diff
==============================================================================
--- incubator/whirr/trunk/core/src/main/java/org/apache/whirr/actions/ScriptBasedClusterAction.java (original)
+++ incubator/whirr/trunk/core/src/main/java/org/apache/whirr/actions/ScriptBasedClusterAction.java Fri Apr 15 09:38:49 2011
@@ -60,7 +60,6 @@ public abstract class ScriptBasedCluster
     Cluster newCluster = cluster;
     for (InstanceTemplate instanceTemplate : clusterSpec.getInstanceTemplates()) {
       StatementBuilder statementBuilder = new StatementBuilder();
-      statementBuilder.addStatement(Statements.call("install_runurl"));
       ComputeServiceContext computServiceContext = // TODO: shouldn't create lots of these
         ComputeServiceContextBuilder.build(getComputeServiceContextFactory(), clusterSpec);
       FirewallManager firewallManager = new FirewallManager(computServiceContext,