Message ID | 20180726105106.642-2-jkbs@redhat.com |
---|---|
State | Superseded |
Headers | show |
Series | Port Group fixes | expand |
On Thu, Jul 26, 2018 at 3:51 AM, Jakub Sitnicki <jkbs@redhat.com> wrote: > > Be user-friendly and allow using port group's name as its identifier in > database commands. > > Signed-off-by: Jakub Sitnicki <jkbs@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 > dev@openvswitch.org > https://mail.openvswitch.org/mailman/listinfo/ovs-dev Thanks Jakub! Acked-by: Han Zhou <hzhou8@ebay.com>
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
Be user-friendly and allow using port group's name as its identifier in database commands. Signed-off-by: Jakub Sitnicki <jkbs@redhat.com> --- ovn/utilities/ovn-nbctl.c | 3 +++ tests/ovn-nbctl.at | 14 ++++++++++++++ 2 files changed, 17 insertions(+)