From: Justin Pettit Date: Wed, 4 Nov 2009 05:19:47 +0000 (-0800) Subject: xenserver: Fix issue with deleting network UUID on VLAN destruction X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=2a577bd807023009afab02d975787d67389c4a27;p=openvswitch xenserver: Fix issue with deleting network UUID on VLAN destruction In XenServer, a VLAN is considered an additional network with its own UUID. The interface-reconfigure script properly adds this network UUID to the configuration script, but commit 774428 removed the code that would remove this information on VLAN destruction. Ian Campbell was the author of that commit and felt that reverting this part was safe. Bug #1973 --- diff --git a/xenserver/opt_xensource_libexec_interface-reconfigure b/xenserver/opt_xensource_libexec_interface-reconfigure index 82c38636..59036a56 100755 --- a/xenserver/opt_xensource_libexec_interface-reconfigure +++ b/xenserver/opt_xensource_libexec_interface-reconfigure @@ -1518,10 +1518,10 @@ def action_down(pif): ifdown(ipdev) if dp: - #nw = db.get_pif_record(pif)['network'] - #nwrec = db.get_network_record(nw) - #cfgmod_argv += ['# deconfigure xs-network-uuids'] - #cfgmod_argv += ['--del-entry=bridge.%s.xs-network-uuids=%s' % (bridge,nwrec['uuid'])] + nw = db.get_pif_record(pif)['network'] + nwrec = db.get_network_record(nw) + cfgmod_argv += ['# deconfigure xs-network-uuids'] + cfgmod_argv += ['--del-entry=bridge.%s.xs-network-uuids=%s' % (bridge,nwrec['uuid'])] log("deconfigure ipdev %s on %s" % (ipdev,bridge)) cfgmod_argv += ["# deconfigure ipdev %s" % ipdev]