On Fri, Jun 2, 2023 at 12:12 AM Han Zhou <hz...@ovn.org> wrote: > > MAC_Binding table is to store neighbour cache for logical router > datapaths to resolve unknown IP addresses to MAC. The datapath field is > always logical router and the logical_port feild is always logical > router port. The test case used logical switch and logical switch ports > by mistake. Correct it to use logical router and logical router ports. > > Signed-off-by: Han Zhou <hz...@ovn.org> > Reviewed-by: Ales Musil <amu...@redhat.com>
Acked-by: Numan Siddique <num...@ovn.org> Numan > --- > tests/ovn.at | 26 ++++++++++++-------------- > 1 file changed, 12 insertions(+), 14 deletions(-) > > diff --git a/tests/ovn.at b/tests/ovn.at > index d8c694002b8c..5e6a8fefa36c 100644 > --- a/tests/ovn.at > +++ b/tests/ovn.at > @@ -9218,23 +9218,21 @@ sim_add hv1 > as hv1 > ovs-vsctl -- add-br br-phys > ovn_attach n1 br-phys 192.168.0.1 > -# Create logical switch ls0 > -ovn-nbctl ls-add ls0 > -# Create ports lp0, lp1 in ls0 > -ovn-nbctl lsp-add ls0 lp0 > -ovn-nbctl lsp-add ls0 lp1 > -ovn-nbctl lsp-set-addresses lp0 "f0:00:00:00:00:01 192.168.0.1" > -ovn-nbctl lsp-set-addresses lp1 "f0:00:00:00:00:02 192.168.0.2" > +# Create logical router lr0 > +ovn-nbctl lr-add lr0 > +# Create ports lrp0, lrp1 in lr0 > +ovn-nbctl lrp-add lr0 lrp0 f0:00:00:00:00:01 192.168.0.1/24 > +ovn-nbctl lrp-add lr0 lrp1 f0:00:00:00:00:02 192.168.1.1/24 > dp_uuid=$(fetch_column Datapath_Binding _uuid) > -ovn-sbctl create MAC_Binding ip=10.0.0.1 datapath=$dp_uuid logical_port=lp0 > mac="mac1" > -ovn-sbctl create MAC_Binding ip=10.0.0.1 datapath=$dp_uuid logical_port=lp1 > mac="mac2" > +ovn-sbctl create MAC_Binding ip=10.0.0.1 datapath=$dp_uuid logical_port=lrp0 > mac="mac1" > +ovn-sbctl create MAC_Binding ip=10.0.0.1 datapath=$dp_uuid logical_port=lrp1 > mac="mac2" > ovn-sbctl find MAC_Binding > -# Delete port lp0 and check that its MAC_Binding is deleted. > -ovn-nbctl lsp-del lp0 > +# Delete port lrp0 and check that its MAC_Binding is deleted. > +ovn-nbctl lrp-del lrp0 > ovn-sbctl find MAC_Binding > -wait_row_count MAC_Binding 0 logical_port=lp0 > -# Delete logical switch ls0 and check that its MAC_Binding is deleted. > -ovn-nbctl ls-del ls0 > +wait_row_count MAC_Binding 0 logical_port=lrp0 > +# Delete logical switch lr0 and check that its MAC_Binding is deleted. > +ovn-nbctl lr-del lr0 > ovn-sbctl find MAC_Binding > wait_row_count MAC_Binding 0 > > -- > 2.30.2 > > _______________________________________________ > dev mailing list > d...@openvswitch.org > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev