From 9f9081f979cac18d381a349bf736d65ecddbcf02 Mon Sep 17 00:00:00 2001
From: Tim Rozet <trozet@redhat.com>
Date: Mon, 18 Apr 2016 12:01:43 -0400
Subject: Fixes timing issue with instack IP and non-default libvirt networks

Changes include:
 - Refactored + added longer timeout to detecting instack VM IP
 - Added support for using non-default libvirt IP range (192.168.122.1)

JIRA: APEX-139 APEX-140

Change-Id: I41a35c971d4af5999f5dd3355cea507c8dcf4edf
Signed-off-by: Tim Rozet <trozet@redhat.com>
---
 ci/deploy.sh | 38 ++++++++++++++++++++------------------
 1 file changed, 20 insertions(+), 18 deletions(-)

diff --git a/ci/deploy.sh b/ci/deploy.sh
index 1d2655c6..2c8455d0 100755
--- a/ci/deploy.sh
+++ b/ci/deploy.sh
@@ -535,30 +535,26 @@ function setup_instack_vm {
 
   sleep 3 # let DHCP happen
 
-  CNT=10
+  CNT=20
   echo -n "${blue}Waiting for instack's dhcp address${reset}"
-  while ! grep instack /var/lib/libvirt/dnsmasq/default.leases > /dev/null && [ $CNT -gt 0 ]; do
-      echo -n "."
-      sleep 3
-      CNT=CNT-1
-  done
-
   # get the instack VM IP
-  UNDERCLOUD=$(grep instack /var/lib/libvirt/dnsmasq/default.leases | awk '{print $3}' | head -n 1)
-  if [ -z "$UNDERCLOUD" ]; then
+  while [ $CNT -gt 0 ]; do
     #if not found then dnsmasq may be using leasefile-ro
     instack_mac=$(virsh domiflist instack | grep default | \
                   grep -Eo "[0-9a-f\]+:[0-9a-f\]+:[0-9a-f\]+:[0-9a-f\]+:[0-9a-f\]+:[0-9a-f\]+")
     UNDERCLOUD=$(/usr/sbin/arp -e | grep ${instack_mac} | awk {'print $1'})
 
-    if [ -z "$UNDERCLOUD" ]; then
-      echo "\n\nNever got IP for Instack. Can Not Continue."
-      exit 1
-    else
-      echo -e "${blue}\rInstack VM has IP $UNDERCLOUD${reset}"
+    if [ -n "$UNDERCLOUD" ]; then
+      echo -e "${blue}\nInstack VM has IP $UNDERCLOUD${reset}"
+      break
     fi
-  else
-     echo -e "${blue}\rInstack VM has IP $UNDERCLOUD${reset}"
+    sleep 6
+    CNT=$((CNT-1))
+  done
+
+  if [ -z "$UNDERCLOUD" ]; then
+    echo "${red}\n\nNever got IP for Instack. Can Not Continue.${reset}"
+    exit 1
   fi
 
   CNT=10
@@ -566,7 +562,7 @@ function setup_instack_vm {
   while ! ping -c 1 $UNDERCLOUD > /dev/null && [ $CNT -gt 0 ]; do
       echo -n "."
       sleep 3
-      CNT=$CNT-1
+      CNT=$((CNT-1))
   done
   if [ "$CNT" -eq 0 ]; then
       echo "Failed to contact Instack. Can Not Continue"
@@ -576,7 +572,7 @@ function setup_instack_vm {
   while ! ssh -T ${SSH_OPTIONS[@]} "root@$UNDERCLOUD" "echo ''" 2>&1> /dev/null && [ $CNT -gt 0 ]; do
       echo -n "."
       sleep 3
-      CNT=$CNT-1
+      CNT=$((CNT-1))
   done
   if [ "$CNT" -eq 0 ]; then
       echo "Failed to connect to Instack. Can Not Continue"
@@ -714,6 +710,12 @@ print data['nodes'][$i]['mac'][0]"
       DEPLOY_OPTIONS+=" --libvirt-type qemu"
       INSTACKENV=$CONFIG/instackenv-virt.json
 
+      # Check for non-default libvirt network
+      if ! echo $UNDERCLOUD | grep 192.168.122 1>/dev/null; then
+        libvirt_net_workaround=$(echo $UNDERCLOUD | sed -E 's/\.[0-9]+$//')
+        sed -i 's/192.168.122/'"$libvirt_net_workaround"'/g' $INSTACKENV
+      fi
+
       # upload instackenv file to Instack for virtual deployment
       scp ${SSH_OPTIONS[@]} $INSTACKENV "stack@$UNDERCLOUD":instackenv.json
   fi
-- 
cgit 1.2.3-korg