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 <g...@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 <b...@ovn.org>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to