[ovs-dev] [PATCH] xenserver: Avoid error due to missing MTU fields on XenServer 5.5.

Justin Pettit jpettit at nicira.com
Thu Feb 25 01:13:07 UTC 2010


I know you already pushed this, but it looks good to me.

--Justin


On Feb 24, 2010, at 3:48 PM, Ben Pfaff wrote:

> The network records in XenServer 5.5 do not have an MTU field, so allow
> these to be missing.
> 
> Diagnosed-by: Reid Price <reid at nicira.com>
> CC: Ian Campbell <ian.campbell at citrix.com>
> Signed-off-by: Ben Pfaff <blp at nicira.com>
> ---
> .../opt_xensource_libexec_InterfaceReconfigure.py  |    4 ++++
> 1 files changed, 4 insertions(+), 0 deletions(-)
> 
> diff --git a/xenserver/opt_xensource_libexec_InterfaceReconfigure.py b/xenserver/opt_xensource_libexec_InterfaceReconfigure.py
> index 3c3a019..570ebcc 100644
> --- a/xenserver/opt_xensource_libexec_InterfaceReconfigure.py
> +++ b/xenserver/opt_xensource_libexec_InterfaceReconfigure.py
> @@ -402,6 +402,10 @@ class DatabaseCache(object):
>                 if f == "PIFs":
>                     # drop PIFs on other hosts
>                     self.__networks[n][f] = [p for p in rec[f] if self.__pif_on_host(p)]
> +                elif f == "MTU" and f not in rec:
> +                    # XenServer 5.5 network records did not have an
> +                    # MTU field, so allow this to be missing.
> +                    pass
>                 else:
>                     self.__networks[n][f] = rec[f]
>             self.__networks[n]['other_config'] = {}
> -- 
> 1.6.6.1
> 
> 
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev_openvswitch.org





More information about the dev mailing list