tools: hv: ignore a NIC if it has been configured
authorsixiao@microsoft.com <sixiao@microsoft.com>
Fri, 14 Jul 2017 17:47:20 +0000 (10:47 -0700)
committerDavid S. Miller <davem@davemloft.net>
Sun, 16 Jul 2017 23:36:01 +0000 (16:36 -0700)
Let bondvf.sh ignore this NIC if it has been configured, to prevent
user configuration from being overwritten unexpectly.

Signed-off-by: Simon Xiao <sixiao@microsoft.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
tools/hv/bondvf.sh

index 89b25068cd98775d9f10f3cb409012034b34ed3e..80f102860cf81deb13c37de1ebe5b87f747883ca 100755 (executable)
@@ -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