HADOOP-5925. EC2 scripts should exit on error.

git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/trunk@788699 13f79535-47bb-0310-9956-ffa450edef68

CHANGES.txt

@@ -465,6 +465,8 @@ Trunk (unreleased changes)
     commands that do not complete within a certain amount of time.
     (Sreekanth Ramakrishnan via yhemanth)
 
+    HADOOP-5925. EC2 scripts should exit on error. (tomwhite)
+
   OPTIMIZATIONS
 
     HADOOP-5595. NameNode does not need to run a replicator to choose a

src/contrib/ec2/bin/cmd-hadoop-cluster

@@ -17,6 +17,8 @@
 
 # Run commands on master or specified node of a running Hadoop EC2 cluster.
 
+set -o errexit
+
 # if no args specified, show usage
 if [ $# = 0 ]; then
   echo "Command required!"

src/contrib/ec2/bin/create-hadoop-image

@@ -18,6 +18,8 @@
 # Create a Hadoop AMI.
 # Inspired by Jonathan Siegel's EC2 script (http://blogsiegel.blogspot.com/2006/08/sandboxing-amazon-ec2.html)
 
+set -o errexit
+
 # Import variables
 bin=`dirname "$0"`
 bin=`cd "$bin"; pwd`

src/contrib/ec2/bin/delete-hadoop-cluster

@@ -15,7 +15,9 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-# Delete the groups an local files associated with a cluster.
+# Delete the groups and local files associated with a cluster.
+
+set -o errexit
 
 if [ -z $1 ]; then
   echo "Cluster name required!"
@@ -42,17 +44,17 @@ bin=`cd "$bin"; pwd`
 rm -f $MASTER_IP_PATH
 rm -f $MASTER_PRIVATE_IP_PATH
 
-ec2-describe-group | egrep "[[:space:]]$CLUSTER_MASTER[[:space:]]" > /dev/null
+if ec2-describe-group $CLUSTER_MASTER > /dev/null 2>&1; then
-if [ $? -eq 0 ]; then
+  if ec2-describe-group $CLUSTER > /dev/null 2>&1; then
+    echo "Revoking authorization between $CLUSTER_MASTER and $CLUSTER"
+    ec2-revoke $CLUSTER_MASTER -o $CLUSTER -u $AWS_ACCOUNT_ID || true
+    ec2-revoke $CLUSTER -o $CLUSTER_MASTER -u $AWS_ACCOUNT_ID || true
+  fi
   echo "Deleting group $CLUSTER_MASTER"
-  ec2-revoke $CLUSTER_MASTER -o $CLUSTER -u $AWS_ACCOUNT_ID
-fi
-
-ec2-describe-group | egrep "[[:space:]]$CLUSTER[[:space:]]" > /dev/null
-if [ $? -eq 0 ]; then
-  echo "Deleting group $CLUSTER"
-  ec2-revoke $CLUSTER -o $CLUSTER_MASTER -u $AWS_ACCOUNT_ID
-fi
-
   ec2-delete-group $CLUSTER_MASTER
+fi
+
+if ec2-describe-group $CLUSTER > /dev/null 2>&1; then
+  echo "Deleting group $CLUSTER"
   ec2-delete-group $CLUSTER
+fi

src/contrib/ec2/bin/hadoop-ec2

@@ -15,6 +15,8 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
+set -o errexit
+
 bin=`dirname "$0"`
 bin=`cd "$bin"; pwd`
 

src/contrib/ec2/bin/launch-hadoop-cluster

@@ -17,6 +17,8 @@
 
 # Launch an EC2 cluster of Hadoop instances.
 
+set -o errexit
+
 # Import variables
 bin=`dirname "$0"`
 bin=`cd "$bin"; pwd`

src/contrib/ec2/bin/launch-hadoop-master

@@ -17,6 +17,8 @@
 
 # Launch an EC2 Hadoop master.
 
+set -o errexit
+
 if [ -z $1 ]; then
   echo "Cluster name required!"
   exit -1
@@ -46,8 +48,7 @@ if [ ! -z "$MASTER_EC2_HOST" ]; then
   exit 0
 fi
 
-ec2-describe-group | egrep "[[:space:]]$CLUSTER_MASTER[[:space:]]" > /dev/null
+if ! ec2-describe-group $CLUSTER_MASTER > /dev/null 2>&1; then
-if [ ! $? -eq 0 ]; then
   echo "Creating group $CLUSTER_MASTER"
   ec2-add-group $CLUSTER_MASTER -d "Group for Hadoop Master."
   ec2-authorize $CLUSTER_MASTER -o $CLUSTER_MASTER -u $AWS_ACCOUNT_ID
@@ -61,8 +62,7 @@ if [ ! $? -eq 0 ]; then
   fi
 fi
 
-ec2-describe-group | egrep "[[:space:]]$CLUSTER[[:space:]]" > /dev/null
+if ! ec2-describe-group $CLUSTER > /dev/null 2>&1; then
-if [ ! $? -eq 0 ]; then
   echo "Creating group $CLUSTER"
   ec2-add-group $CLUSTER -d "Group for Hadoop Slaves."
   ec2-authorize $CLUSTER -o $CLUSTER -u $AWS_ACCOUNT_ID
@@ -105,8 +105,7 @@ MASTER_EC2_ZONE=`ec2-describe-instances $INSTANCE | grep INSTANCE | grep running
 echo $MASTER_EC2_ZONE > $MASTER_ZONE_PATH
 
 while true; do
-  REPLY=`ssh $SSH_OPTS "root@$MASTER_EC2_HOST" 'echo "hello"'`
+  if ssh $SSH_OPTS "root@$MASTER_EC2_HOST" 'echo "hello"' > /dev/null 2>&1; then
-  if [ ! -z $REPLY ]; then
    break;
   fi
   sleep 5

src/contrib/ec2/bin/launch-hadoop-slaves

@@ -17,6 +17,8 @@
 
 # Launch an EC2 Hadoop slaves.
 
+set -o errexit
+
 if [ -z $1 ]; then
   echo "Cluster name required!"
   exit -1

src/contrib/ec2/bin/list-hadoop-clusters

@@ -17,6 +17,8 @@
 
 # List running clusters.
 
+set -o errexit
+
 # Import variables
 bin=`dirname "$0"`
 bin=`cd "$bin"; pwd`

src/contrib/ec2/bin/terminate-hadoop-cluster

@@ -17,6 +17,8 @@
 
 # Terminate a cluster.
 
+set -o errexit
+
 if [ -z $1 ]; then
   echo "Cluster name required!"
   exit -1