[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