Re: [ovs-dev] [ovn-controller-gw 3/4] ovn-sbctl: Add ovn-sbctl.

2015-07-03 Thread Alex Wang
Thx for the review, will working on converting ovn-nbctl once this series is committed~ ;D On Fri, Jul 3, 2015 at 9:29 AM, Ben Pfaff wrote: > On Fri, Jun 26, 2015 at 08:45:51AM -0700, Alex Wang wrote: > > This commit adds ovn-sbctl to ovn family by using the db-ctl-base > > library. > > > > Sign

Re: [ovs-dev] [ovn-controller-gw 3/4] ovn-sbctl: Add ovn-sbctl.

2015-07-03 Thread Ben Pfaff
On Fri, Jun 26, 2015 at 08:45:51AM -0700, Alex Wang wrote: > This commit adds ovn-sbctl to ovn family by using the db-ctl-base > library. > > Signed-off-by: Alex Wang This is really great! Thank you for writing this. In the documentation (which is good--thank you), I would consider including,

Re: [ovs-dev] [ovn-controller-gw 3/4] ovn-sbctl: Add ovn-sbctl.

2015-07-03 Thread Ben Pfaff
On Fri, Jun 26, 2015 at 09:00:11AM -0700, Alex Wang wrote: > Want to discuss if we may want to refine ovn-nbctl to the same format as > ovn-sbctl > (using lib/db-ctl-cmds module). I think it's a good idea. ___ dev mailing list dev@openvswitch.org http://

Re: [ovs-dev] [ovn-controller-gw 3/4] ovn-sbctl: Add ovn-sbctl.

2015-06-26 Thread Alex Wang
Hey Ben, Russell, Want to discuss if we may want to refine ovn-nbctl to the same format as ovn-sbctl (using lib/db-ctl-cmds module). The benefit of such refinement is that ovn-nbctl can have the common ovsdb operations (e.g. list, set, create, ... like in ovs-vsctl). Since my testing does not re