[ovs-dev] [PATCH 5/7] ovn-nb: Add Load-balancer table to schema.

Ben Pfaff blp at ovn.org
Sun Jul 3 04:45:57 UTC 2016


On Wed, Jun 29, 2016 at 01:17:09AM -0700, Gurucharan Shetty wrote:
> Also add the ability to run database commands on the
> new schema using ovn-nbctl.
> 
> Signed-off-by: Gurucharan Shetty <guru at ovn.org>

I agree with Zong's suggestion to add examples.

It is a little unusual, for OVS, to add documentation for a new feature
in a different commit from the implementation for the new feature.  It
might make sense to squash this with the implementation patch.

I haven't read ahead to the implementation patches, but I am pleased
with the docoumentation and schema for the moment, especially if you add
an example, so:
Acked-by: Ben Pfaff <blp at ovn.org>



More information about the dev mailing list