[Xen-API] [PATCH 10 of 12] [PATCH] xenserver: Add assertion and additional comments to ovs-vsctl commands

[David Scott <dave.scott@xxxxxxxxxxxxx>]

# HG changeset patch
# User Ben Pfaff <blp@xxxxxxxxxx>
# Date 1278595551 -3600
# Node ID b06691271200bfb9903e3e49b2dd833583b797aa
# Parent  73cc0a5b9f6e8c3d1762f66244c96fe2238c519c
[PATCH] xenserver: Add assertion and additional comments to ovs-vsctl commands.

>From 86e1bb44dadf886a34bfef00b5523873081f32fe Mon Sep 17 00:00:00 2001
Date: Mon, 22 Feb 2010 13:42:02 -0800
I found these useful while debugging.

Signed-off-by: Ben Pfaff <blp@xxxxxxxxxx>
Signed-off-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
---
 ...ensource_libexec_InterfaceReconfigureVswitch.py |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)

diff -r 73cc0a5b9f6e -r b06691271200 scripts/InterfaceReconfigureVswitch.py
--- a/scripts/InterfaceReconfigureVswitch.py    Thu Jul 08 14:25:51 2010 +0100
+++ b/scripts/InterfaceReconfigureVswitch.py    Thu Jul 08 14:25:51 2010 +0100
@@ -171,10 +171,13 @@
     vsctl_argv = []
     extra_up_ports = []
 
+    assert not pif_is_vlan(pif)
     bridge = pif_bridge_name(pif)
 
     physical_devices = datapath_get_physical_pifs(pif)
 
+    vsctl_argv += ['## configuring datapath %s' % bridge]
+
     # Determine additional devices to deconfigure.
     #
     # Given all physical devices which are part of this PIF we need to
@@ -258,6 +261,8 @@
         vsctl_argv += ['--', '--may-exist', 'add-port', bridge, iface]
 
     vsctl_argv += set_br_external_ids(pif)
+    vsctl_argv += ['## done configuring datapath %s' % bridge]
+
     return vsctl_argv,extra_up_ports
 
 def deconfigure_bridge(pif):
 scripts/InterfaceReconfigureVswitch.py |  5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)

Attachment: txtkqG95MPecy.txt
Description: Text Data

_______________________________________________
xen-api mailing list
xen-api@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/mailman/listinfo/xen-api