[ovs-dev] [brcompatd 4/8] ovs-vsctl: New "comment" command.

Ethan Jackson ethan at nicira.com
Tue Jun 7 23:47:36 UTC 2011


Looks Good.

Ethan

On Mon, Jun 6, 2011 at 12:41, Ben Pfaff <blp at nicira.com> wrote:
> This allows processes that call ovs-vsctl to add remarks that explain their
> intentions.
> ---
>  utilities/ovs-vsctl.8.in |    3 +++
>  utilities/ovs-vsctl.c    |    7 +++++--
>  2 files changed, 8 insertions(+), 2 deletions(-)
>
> diff --git a/utilities/ovs-vsctl.8.in b/utilities/ovs-vsctl.8.in
> index 88230be..309ea0a 100644
> --- a/utilities/ovs-vsctl.8.in
> +++ b/utilities/ovs-vsctl.8.in
> @@ -694,6 +694,9 @@ initially connects to the database.
>  Consider specifying \fB\-\-timeout=0\fR along with
>  \fB\-\-wait\-until\fR, to prevent \fBovs\-vsctl\fR from terminating
>  after waiting only at most 5 seconds.
> +.IP "\fBcomment \fR[\fIarg\fR]..."
> +This command has no effect on behavior, but any database log record
> +created by the command will include the command and its arguments.
>  .SH "EXAMPLES"
>  Create a new bridge named br0 and add port eth0 to it:
>  .IP
> diff --git a/utilities/ovs-vsctl.c b/utilities/ovs-vsctl.c
> index 104f65e..df20f06 100644
> --- a/utilities/ovs-vsctl.c
> +++ b/utilities/ovs-vsctl.c
> @@ -3592,7 +3592,9 @@ do_vsctl(const char *args, struct vsctl_command *commands, size_t n_commands,
>         struct vsctl_context ctx;
>
>         vsctl_context_init(&ctx, c, idl, txn, ovs, symtab);
> -        (c->syntax->run)(&ctx);
> +        if (c->syntax->run) {
> +            (c->syntax->run)(&ctx);
> +        }
>         vsctl_context_done(&ctx, c);
>
>         if (ctx.try_again) {
> @@ -3786,7 +3788,8 @@ static const struct vsctl_command_syntax all_commands[] = {
>     /* Switch commands. */
>     {"emer-reset", 0, 0, pre_cmd_emer_reset, cmd_emer_reset, NULL, "", RW},
>
> -    /* Parameter commands. */
> +    /* Database commands. */
> +    {"comment", 0, INT_MAX, NULL, NULL, NULL, "", RO},
>     {"get", 2, INT_MAX, pre_cmd_get, cmd_get, NULL, "--if-exists,--id=", RO},
>     {"list", 1, INT_MAX, pre_cmd_list, cmd_list, NULL, "--columns=", RO},
>     {"find", 1, INT_MAX, pre_cmd_find, cmd_find, NULL, "--columns=", RO},
> --
> 1.7.4.4
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
>



More information about the dev mailing list