[ovs-dev] [ovn-controller-vtep V3 3/7] ovn-sbctl: Add ovn-sbctl.

Ben Pfaff blp at nicira.com
Thu Jul 16 20:43:32 UTC 2015


On Wed, Jul 15, 2015 at 11:02:29PM -0700, Alex Wang wrote:
> On Wed, Jul 15, 2015 at 3:48 PM, Ben Pfaff <blp at nicira.com> wrote:
> 
> > On Mon, Jul 13, 2015 at 08:22:39PM -0700, Alex Wang wrote:
> > > This commit adds ovn-sbctl to ovn family by using the db-ctl-base
> > > library.
> > >
> > > As a side effect, we move the ovn-nbctl/ovn-sbctl related files
> > > into ovn/utilities.
> > >
> > > Signed-off-by: Alex Wang <alexw at nicira.com>
> > > Acked-by: Ben Pfaff <blp at nicira.com>
> > >
> > > ---
> > > V2->V3:
> > > - rebase to base.
> > > - adopt Russell's review comments.
> > > - move ovn-nbctl/sbctl related files into ovn/utilities.
> > >
> > > PATCH->V2:
> > > - change command add/del-ch to add/del-chassis.
> > > - refine the manual based on comments from Ben.
> >
> > I get a test failure in a new test due to:
> >     ovn-sbctl: unknown command 'add-chassis'; use --help for help
> >
> >
> 
> I tried it on my 12.04 vm and another rhel7 vm...  Could not run into this
> issue.

OK, maybe I did something wrong somehow.  If you happen to post another
revision, I'll try it again; otherwise if it makes it into master, no
big deal, I'll just find and fix the problem.



More information about the dev mailing list