[ovs-dev] [ovn-controller-vtep V5 03/12] ovn-sbctl: Add ovn-sbctl.

Russell Bryant rbryant at redhat.com
Fri Aug 7 18:13:17 UTC 2015


On 08/07/2015 03:46 AM, Alex Wang wrote:
> This commit adds ovn-sbctl to ovn family by using the db-ctl-base
> library.
> 
> Signed-off-by: Alex Wang <alexw at nicira.com>
> Acked-by: Ben Pfaff <blp at nicira.com>

I verified that the tests pass and also played around with it manually.
 Everything worked as expected.  I only found one minor doc issue.  Nice
work!

Acked-by: Russell Bryant <rbryant at redhat.com>

> +static void
> +usage(void)
> +{
> +    printf("\
> +%s: ovs-vswitchd management utility\n\
> +\n\
> +for debugging and testing only, never use it in production\n\
> +\n\
> +usage: %s [OPTIONS] COMMAND [ARG...]\n\
> +\n\
> +SouthBound DB commands:\n\
> +  show                        print overview of database contents\n\
> +\n\
> +Chassis commands:\n\
> +  chassis-add CHASSIS         create a new chassis named CHASSIS\n\

encap-type and encap-ip are required here, as well.  They're shown in
the man page but missing from this usage output.

-- 
Russell Bryant



More information about the dev mailing list