[ovs-dev] [PATCH] xenserver: Fix issue with deleting network UUID on VLAN destruction

[Justin Pettit]

Whoops, this is actually Bug #1973.

--Justin


On Nov 3, 2009, at 9:38 PM, Justin Pettit wrote:

> 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 #1983
> ---
> .../opt_xensource_libexec_interface-reconfigure    |    8 ++++----
> 1 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/xenserver/opt_xensource_libexec_interface-reconfigure b/ 
> xenserver/opt_xensource_libexec_interface-reconfigure
> index 82c3863..59036a5 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]
> -- 
> 1.5.5
>
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev_openvswitch.org