[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-changelog] Don't treat the failure of ifconfig vifX.Y down as a fatal error. This hasn't



# HG changeset patch
# User emellor@xxxxxxxxxxxxxxxxxxxxxx
# Node ID 8aabf8959595e676f05f08ab17e25575982b1438
# Parent  e48bb8550f95c151dcda95a043af88e12fc899d9
Don't treat the failure of ifconfig vifX.Y down as a fatal error.  This hasn't
emerged as a problem in the field, AFAIK, but it seems that distro scripts
could have brought the device down already by this point (Debian stable's
net.agent does ifdown vifX.Y, and it wouldn't surprise me if someone decided
to do ifconfig vifX.Y down if this failed, for example).

Logging but ignoring the failure seems sensible.

Signed-off-by: Ewan Mellor <ewan@xxxxxxxxxxxxx>

diff -r e48bb8550f95 -r 8aabf8959595 tools/examples/vif-bridge
--- a/tools/examples/vif-bridge Wed Nov 23 19:13:13 2005
+++ b/tools/examples/vif-bridge Wed Nov 23 19:16:54 2005
@@ -62,7 +62,7 @@
 
     offline)
         # vifs are auto-removed from bridge.
-        ifconfig "$vif" down || fatal "ifconfig $vif down failed"
+        ifconfig "$vif" down || log err "ifconfig $vif down failed"
         ;;
 esac
 

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.