From: Tim Rozet <trozet@redhat.com>
Date: Wed, 19 Aug 2015 20:20:01 +0000 (-0400)
Subject: Removes pxebr in clean.sh, updates setup.sh
X-Git-Tag: arno.2015.2.0~52^2^2~15
X-Git-Url: https://gerrit.opnfv.org/gerrit/gitweb?p=genesis.git;a=commitdiff_plain;h=30361efb7d3bf9525e9d9aadd5c137e9a654d1a4

Removes pxebr in clean.sh, updates setup.sh

pxebr bridge is needed by Fuel, but causes issues with Foreman.  It is
now removed in clean.sh.  setup.sh is also updated to be executable and
fixes a bug where grep was being used on output that is in stderr.

JIRA: OCTO-109

Change-Id: I037e168e40f3352a61a604ed6cea21d10250b434
Signed-off-by: Tim Rozet <trozet@redhat.com>
---

diff --git a/common/ci/clean.sh b/common/ci/clean.sh
index bb47479..da75205 100755
--- a/common/ci/clean.sh
+++ b/common/ci/clean.sh
@@ -14,7 +14,7 @@ reset=`tput sgr0`
 blue=`tput setaf 4`
 red=`tput setaf 1`
 green=`tput setaf 2`
-
+pxe_bridge='pxebr'
 vm_dir=/var/opt/opnfv
 ##END VARS
 
@@ -234,3 +234,21 @@ for kernel_mod in vboxnetadp vboxnetflt vboxpci vboxdrv kvm_intel kvm; do
     echo "${blue}Removed Kernel Module: $kernel_mod ${reset}"
   fi
 done
+
+###remove pxebr
+echo "${blue}Checking whether PXE bridge ${pxe_bridge} exists${reset}"
+if ! brctl show ${pxe_bridge} 2>&1 | grep -i 'No such device'; then
+  echo "${blue}PXE bridge detected. Removing...${reset}"
+  if ifconfig | grep ${pxe_bridge}; then
+    ifdown ${pxe_bridge}
+  fi
+  brctl delbr ${pxe_bridge}
+  if ifconfig | grep ${pxe_bridge} || brctl show | grep ${pxe_bridge}; then
+    echo "${red}Error trying to remove ${pxe_bridge}${reset}"
+    exit 1
+  else
+    echo "${blue}PXE bridge ${pxe_bridge} removed${reset}"
+  fi
+else
+  echo "${blue}PXE bridge ${pxe_bridge} does not exist${reset}"
+fi
diff --git a/common/ci/setup.sh b/common/ci/setup.sh
old mode 100644
new mode 100755
index 73b0aa6..3ffd36b
--- a/common/ci/setup.sh
+++ b/common/ci/setup.sh
@@ -128,7 +128,7 @@ setup_pxe_bridge() {
 
   #Check whether PXE bridge exists
   echo "${blue}Checking whether PXE bridge ${pxe_bridge} exists${reset}"
-  if brctl show ${pxe_bridge} | grep 'No such device'; then
+  if brctl show ${pxe_bridge} 2>&1 | grep 'No such device'; then
     echo "${blue}Creating PXE bridge ${pxe_bridge}${reset}"
     brctl addbr ${pxe_bridge}
   else
@@ -140,10 +140,10 @@ setup_pxe_bridge() {
 
   #Add VLAN 0 (PXE) interface to PXE bridge
   echo "${blue}Checking whether VLAN 0 (PXE) interface ${pxe_interface} is added to PXE bridge ${pxe_bridge} exists${reset}"
-  if ! brctl show ${pxe_bridge} | grep ${pxe_interface}; then
+  if ! brctl show ${pxe_bridge} 2>&1 | grep ${pxe_interface}; then
     echo "${blue}Adding VLAN 0 (PXE) interface ${pxe_interface} to PXE bridge ${pxe_bridge}${reset}"
     brctl addif ${pxe_bridge} ${pxe_interface}
-    if ! brctl show ${pxe_bridge} | grep ${pxe_interface}; then
+    if ! brctl show ${pxe_bridge} 2>&1 | grep ${pxe_interface}; then
       echo "${red}Could not add VLAN 0 (PXE) interface ${pxe_interface} to PXE bridge ${pxe_bridge}${reset}"
       exit 1
     fi