On Thu, Jul 26, 2018 at 3:51 AM, Jakub Sitnicki <j...@redhat.com> wrote:
>
> Be user-friendly and allow using port group's name as its identifier in
> database commands.
>
> Signed-off-by: Jakub Sitnicki <j...@redhat.com>
> ---
> ovn/utilities/ovn-nbctl.c | 3 +++
> tests/ovn-nbctl.at | 14 ++++++++++++++
> 2 files changed, 17 insertions(+)
>
> diff --git a/ovn/utilities/ovn-nbctl.c b/ovn/utilities/ovn-nbctl.c
> index 3c3e582cb..f99b81bc0 100644
> --- a/ovn/utilities/ovn-nbctl.c
> +++ b/ovn/utilities/ovn-nbctl.c
> @@ -4369,6 +4369,9 @@ static const struct ctl_table_class
tables[NBREC_N_TABLES] = {
> [NBREC_TABLE_ADDRESS_SET].row_ids[0]
> = {&nbrec_address_set_col_name, NULL, NULL},
>
> + [NBREC_TABLE_PORT_GROUP].row_ids[0]
> + = {&nbrec_port_group_col_name, NULL, NULL},
> +
> [NBREC_TABLE_ACL].row_ids[0] = {&nbrec_acl_col_name, NULL, NULL},
> };
>
> diff --git a/tests/ovn-nbctl.at b/tests/ovn-nbctl.at
> index 64e217654..069b7b5b6 100644
> --- a/tests/ovn-nbctl.at
> +++ b/tests/ovn-nbctl.at
> @@ -1505,3 +1505,17 @@ AT_CHECK([grep 'command takes at most .*
arguments' stderr], [0], [ignore])
>
> OVN_NBCTL_TEST_STOP
> AT_CLEANUP
> +
> +dnl ---------------------------------------------------------------------
> +
> +AT_SETUP([ovn-nbctl - Port Groups])
> +OVN_NBCTL_TEST_START
> +
> +dnl Check that port group can be looked up by name
> +AT_CHECK([ovn-nbctl create Port_Group name=pg0], [0], [ignore])
> +AT_CHECK([ovn-nbctl get Port_Group pg0 name], [0], [dnl
> +"pg0"
> +])
> +
> +OVN_NBCTL_TEST_STOP
> +AT_CLEANUP
> --
> 2.14.4
>
> _______________________________________________
> dev mailing list
> d...@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
Thanks Jakub!
Acked-by: Han Zhou <hzh...@ebay.com>
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
