[ovs-dev] [PATCH] openvswitch init script modifications for kvm and lxc hosts

Patrick Mullaney pm.mullaney at gmail.com
Thu Dec 16 02:48:38 UTC 2010


Hi Ben,

> There's some confusion here I think.  XenServer is a whole OS, just as
> SuSE is.  As far as I know, no one carves out bits and pieces of
> XenServer and installs them on top of other distributions.  (Maybe you
> know differently?  I am no expert on this.)
Actually, I'm not an expert here either. Its probably just my
confusion of xen vs. xenserver.

> 
> So, assuming I'm right about that, then there may be no need for it on
> SuSE, because admins want OVS to run if it is installed (as you say).
> 
> > > It doesn't make sense to install etc_init.d_openvswitch-xapi-update on
> > > SuSE at all, since it will at most print an error message and exit with
> > > status 1.  So it seems unnecessary to modify that file.
> > 
> > I can not package it, but on the chance that someone wanted to use suse
> > and xenserver, it might be easier if it was just there. My changes were
> > just about LSB compliance, other distros might care about this too.
> 
> Ah, OK.  Fine.
> 
> > Another approach might be to move basic startup into a separate common
> > script area and then have the xenserver scripts deal with starting
> > stuff specific to it. I don't have xenserver here and didn't watnt to
> > break it on a first attempt. The nice part is I think this should
> > work for both cases now.
> 
> That's a good idea too.
> 
> I haven't done that yet, but I've tweaked these patches to my liking and
> I'll send them back out to you and the list in a minute.  I've retained
> your authorship and sign-offs, and tried to note the changes that I made
> in the proper format, but please do review the changes that I made to
> ensure that they make sense.
Cool - thanks. I'll check them over.

> 
> Thanks,
> 
> Ben.






More information about the dev mailing list