[ovs-dev] [PATCH] INSTALL.XenServer: Document Open vSwitch boot process on XenServer.

[Ben Pfaff]

On Tue, May 10, 2011 at 12:33:01AM -0700, Justin Pettit wrote:
> On May 9, 2011, at 11:27 AM, Ben Pfaff wrote:
> 
> > +        * XenServer expects there to be no bridges configured at
> > +          startup, but the OVS configuration database likely still has
> > +          bridges configured from before reboot.  To match XenServer
> 
> I'd say "boot" instead of "reboot", since it sounds to me like it's
> specific to reboots.

Well, the first time you boot OVS, it has no bridges configured.  It
will only have bridges at startup if XenServer (or something else)
configures some and then you *re*boot.  That's what I had in mind in
that phrasing, anyhow.

I left it this way for now but of course it's easy to revise later.

> > +        * Queries the Open vSwitch manager setting from the XAPI
> > +          database for the XenServer pool.
> 
> Since you've been very specific, it may be good to indicate that
> it's the (now) poorly named "vswitch_controller" parameter.

OK, done.

> > +        * If XAPI and OVS are configured for different managers, or if
> > +          OVS is configured for a manager but XAPI is not, runs
> > +          "ovs-vsctl emer-reset" to bring the Open vSwitch
> > +          configuration to a known state.  (This deconfigures any
> > +          manager as a side effect.)
> 
> It only deconfigures the OVS configuration, correct?  I would
> explicitly state that it "deconfigures any manager configuration
> from the OVS database as a side effect", or something to that
> effect.

I changed this to read "One effect of emer-reset is to deconfigure any
manager from the OVS database."  I guess it's not really a side
effect, since it's one intended purpose of the command.

> Thanks for writing this up.  I'm sure no one will read it, but we
> can now point them to it when they ask on the mailing list.  ;-)

Yes... and now I won't have to go look it all up again myself, either.

I pushed this, with these changes.