[ovs-dev] [suse 4/7] xenserver: Only start ovs-external-ids on XenServer.

Justin Pettit jpettit at nicira.com
Tue Dec 28 19:08:24 UTC 2010


Looks good.

--Justin


On Dec 15, 2010, at 11:19 AM, Ben Pfaff wrote:

> From: Patrick Mullaney <pm.mullaney at gmail.com>
> 
> ovs-external-ids's job is to synchronize UUIDs between XAPI and OVSDB.
> Non-XenServer systems don't have XAPI so they don't need ovs-external-ids.
> 
> Signed-off-by: Patrick Mullaney <pm.mullaney at gmail.com>
> Signed-off-by: Ben Pfaff <blp at nicira.com>
> ---
> xenserver/etc_init.d_openvswitch |   10 ++++++----
> 1 files changed, 6 insertions(+), 4 deletions(-)
> 
> diff --git a/xenserver/etc_init.d_openvswitch b/xenserver/etc_init.d_openvswitch
> index de749a1..e4f9964 100755
> --- a/xenserver/etc_init.d_openvswitch
> +++ b/xenserver/etc_init.d_openvswitch
> @@ -394,10 +394,12 @@ function start {
>         start_brcompatd
>     fi
> 
> -    # Start daemon to monitor external ids
> -    PYTHONPATH=/usr/share/openvswitch/python \
> -               /usr/share/openvswitch/scripts/ovs-external-ids \
> -               --pidfile --detach $monitor_opt "$VSWITCHD_OVSDB_SERVER"
> +    if [ -f /etc/xensource-inventory ]; then
> +        # Start daemon to monitor external ids
> +        PYTHONPATH=/usr/share/openvswitch/python \
> +                   /usr/share/openvswitch/scripts/ovs-external-ids \
> +                   --pidfile --detach $monitor_opt "$VSWITCHD_OVSDB_SERVER"
> +    fi
> 
>     touch /var/lock/subsys/openvswitch
> }
> -- 
> 1.7.1
> 
> 
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev_openvswitch.org





More information about the dev mailing list