[ovs-dev] [PATCH] ovn-sb.xml: Update and improve documentation.

[Russell Bryant]

On Tue, Jun 6, 2017 at 6:32 PM, Ben Pfaff <blp at ovn.org> wrote:
> Some of the abbreviations at the head of this document, like LN and PN,
> turn out to not be very useful, so expand them for clarity.
>
> Some of the statements in this document are more about planning the design
> than the current design.  Remove these for clarity.
>
> Port_Binding rows used to all be about physical locations, except for
> patch ports, but there are more kinds of rows now.  Elaborate for clarity.
>
> Expand on the purpose of the Datapath_Binding table.
>
> Signed-off-by: Ben Pfaff <blp at ovn.org>
> ---
>  ovn/ovn-sb.xml | 89 +++++++++++++++++++++++++++++++++-------------------------
>  1 file changed, 50 insertions(+), 39 deletions(-)

Acked-by: Russell Bryant <russell at ovn.org>

with one comment:

>    <p>
> -    LN data is likely to change more quickly than PN data.  This is especially
> -    true in a container environment where VMs are created and destroyed (and
> -    therefore added to and deleted from logical switches) quickly.
> +    Logical network data is likely to change more quickly than physical network
> +    data.  This is especially true in a container environment where VMs are
> +    created and destroyed (and therefore added to and deleted from logical
> +    switches) quickly.
>    </p>

"... true in a container environment where VMs are ..."

I would change "VMs" to "containers" here.




-- 
Russell Bryant