Let bondvf.sh ignore this NIC if it has been configured, to prevent
user configuration from being overwritten unexpectly.

Signed-off-by: Simon Xiao <six...@microsoft.com>
---
 tools/hv/bondvf.sh | 27 +++++++++++++++++++++++++--
 1 file changed, 25 insertions(+), 2 deletions(-)

diff --git a/tools/hv/bondvf.sh b/tools/hv/bondvf.sh
index 89b2506..80f1028 100755
--- a/tools/hv/bondvf.sh
+++ b/tools/hv/bondvf.sh
@@ -211,6 +211,30 @@ function create_bond {
 
        echo $'\nBond name:' $bondname
 
+       if [ $distro == ubuntu ]
+       then
+               local mainfn=$cfgdir/interfaces
+               local s="^[ \t]*(auto|iface|mapping|allow-.*)[ \t]+${bondname}"
+
+               grep -E "$s" $mainfn
+               if [ $? -eq 0 ]
+               then
+                       echo "WARNING: ${bondname} has been configured already"
+                       return
+               fi
+       elif [ $distro == redhat ] || [ $distro == suse ]
+       then
+               local fn=$cfgdir/ifcfg-$bondname
+               if [ -f $fn ]
+               then
+                       echo "WARNING: ${bondname} has been configured already"
+                       return
+               fi
+       else
+               echo "Unsupported Distro: ${distro}"
+               return
+       fi
+
        echo configuring $primary
        create_eth_cfg_pri_$distro $primary $bondname
 
@@ -219,8 +243,6 @@ function create_bond {
 
        echo creating: $bondname with primary slave: $primary
        create_bond_cfg_$distro $bondname $primary $secondary
-
-       let bondcnt=bondcnt+1
 }
 
 for (( i=0; i < $eth_cnt-1; i++ ))
@@ -228,5 +250,6 @@ do
         if [ -n "${list_match[$i]}" ]
         then
                create_bond ${list_eth[$i]} ${list_match[$i]}
+               let bondcnt=bondcnt+1
         fi
 done
-- 
2.7.4

Reply via email to