xenserver: Don't start OVS if network.conf isn't "openvswitch"
authorJustin Pettit <jpettit@nicira.com>
Tue, 8 Jun 2010 06:17:26 +0000 (23:17 -0700)
committerJustin Pettit <jpettit@nicira.com>
Tue, 8 Jun 2010 17:55:07 +0000 (10:55 -0700)
The OVS processes would start as long as "/etc/xensource/network.conf"
didn't contain "bridge".  The other OVS scripts, however, would complain
(and not run) if it wasn't "vswitch" or "openvswitch".  This commit will
only start the OVS processes if a value of "vswitch" or "openvswitch" is
present, so some consistency is provided.

(If "/etc/xensource/network.conf" is not "bridge", "vswitch", or
"openvswitch", then XAPI will refuse to run anyway, so not much is going
to happen on the system.)

xenserver/etc_init.d_openvswitch

index ce75480..a2c26a0 100755 (executable)
 test -e /etc/sysconfig/openvswitch && . /etc/sysconfig/openvswitch
 
 NETWORK_MODE=$(cat /etc/xensource/network.conf)
-if test "$NETWORK_MODE" = bridge; then
-    echo "Open vSwitch disabled (/etc/xensource/network.conf is 'bridge')" >&2
-    exit 0
-fi
+case $NETWORK_MODE in
+    vswitch|openvswitch)
+        ;;
+    *)
+        echo "Open vSwitch disabled (/etc/xensource/network.conf is not 'openvswitch')" >&2
+        exit 0
+        ;;
+esac
 
 # General config variables in /etc/sysconfig/openvswitch
 if test "$PRODUCT_VERSION" = "5.5.0"; then