bug 762063: make sure that every device is setup with the correct avmshell
authorBrent Baker <brbaker@adobe.com>
Thu, 06 Dec 2012 08:08:40 -0800
changeset 7581 c7b1844ed46dd8b8e6474590dec174bb7029b8f3
parent 7580 167a6c1b492ec6d7647e27b5949e1720e095c9ae
child 7582 a099bc675915c8812d47de5f98e5e5adfac473d3
push id4262
push userdschaffe@adobe.com
push dateWed, 30 Jan 2013 19:01:31 +0000
bugs762063, 1144118
bug 762063: make sure that every device is setup with the correct avmshell Previous code assumed that there would only be a single device and as soon as it saw the same host during deployment it stopped deploying. Really need to check each "thread" host to see if it is a new device. CL@1144118
build/buildbot/slaves/all/winrs-shell-deployer.sh
--- a/build/buildbot/slaves/all/winrs-shell-deployer.sh
+++ b/build/buildbot/slaves/all/winrs-shell-deployer.sh
@@ -69,17 +69,24 @@ do
         exit 1
     fi
 
 # check if already deployed on the host
     echo "$already_deployed" | grep "$WINRS_SHELL_REMOTE_HOST" > /dev/null
     res=$?
     if [ "$res" == "0" ]
     then
-        break     # skip deploying since the host already is setup        
+        echo "host: ${WINRS_SHELL_REMOTE_HOST} already setup, skipping"
+        # increment the counter, check if we are finished looping
+        count=$[count+1]
+        if [ "$count" = "$threadcount" ]
+        then
+            break
+        fi
+        continue     # skip deploying since the host already is setup
     else
         already_deployed="$WINRS_SHELL_REMOTE_HOST $already_deployed"  # add host to deploy list
     fi
 
     echo ""
     echo "===================================================="
     echo "setting up client $count"
     echo "WINRS_SHELL_REMOTE_USER" = "$WINRS_SHELL_REMOTE_USER"