On 22/04/2021 21:14, Han Zhou wrote: > This patch improves the test case by binding 2 VIFs on the HV instead of > one, to make sure conjunction is still used and the scenario is still > tested by this test case when a following patch optimizes conjunction flows. > > Signed-off-by: Han Zhou <[email protected]> > --- > v1->v2: no change > > tests/ovn.at | 94 ++++++++++++++++++++++++++++------------------------ > 1 file changed, 50 insertions(+), 44 deletions(-) > > diff --git a/tests/ovn.at b/tests/ovn.at > index 3d0a7f63f..55444bbd7 100644 > --- a/tests/ovn.at > +++ b/tests/ovn.at > @@ -25111,10 +25111,12 @@ ovs-vsctl add-br br-phys > ovn_attach n1 br-phys 192.168.0.10 > > as hv1 > -ovs-vsctl \ > - -- add-port br-int vif1 \ > - -- set Interface vif1 external_ids:iface-id=sw0-p1 \ > - ofport-request=1 > +for i in 1 2; do > + ovs-vsctl \ > + -- add-port br-int vif$i \ > + -- set Interface vif$i external_ids:iface-id=sw0-p$i \ > + ofport-request=$i > +done > > check as hv1 > ovs-vsctl set open . external_ids:ovn-monitor-all=true > @@ -25122,10 +25124,10 @@ ovs-vsctl set open . > external_ids:ovn-monitor-all=true > check ovn-nbctl ls-add sw0 > check ovn-nbctl pg-add pg1 > check ovn-nbctl pg-add pg2 > -check ovn-nbctl lsp-add sw0 sw0-p2 > -check ovn-nbctl lsp-set-addresses sw0-p2 "00:00:00:00:00:02 192.168.47.2" > check ovn-nbctl lsp-add sw0 sw0-p3 > check ovn-nbctl lsp-set-addresses sw0-p3 "00:00:00:00:00:03 192.168.47.3" > +check ovn-nbctl lsp-add sw0 sw0-p4 > +check ovn-nbctl lsp-set-addresses sw0-p4 "00:00:00:00:00:04 192.168.47.4" > > # Pause ovn-northd. When it is resumed, all the below NB updates > # will be sent in one transaction. > @@ -25135,8 +25137,10 @@ check as northd-backup ovn-appctl -t NORTHD_TYPE > pause > > check ovn-nbctl lsp-add sw0 sw0-p1 > check ovn-nbctl lsp-set-addresses sw0-p1 "00:00:00:00:00:01 192.168.47.1" > -check ovn-nbctl pg-set-ports pg1 sw0-p1 sw0-p2 > -check ovn-nbctl pg-set-ports pg2 sw0-p3 > +check ovn-nbctl lsp-add sw0 sw0-p2 > +check ovn-nbctl lsp-set-addresses sw0-p2 "00:00:00:00:00:02 192.168.47.2" > +check ovn-nbctl pg-set-ports pg1 sw0-p1 sw0-p2 sw0-p3 > +check ovn-nbctl pg-set-ports pg2 sw0-p4 > check ovn-nbctl acl-add pg1 to-lport 1002 "outport == @pg1 && ip4 && ip4.src > == \$pg2_ip4 && udp && udp.dst >= 1 && udp.dst <= 65535" allow-related > > # resume ovn-northd now. This should result in a single update message > @@ -25144,11 +25148,11 @@ check ovn-nbctl acl-add pg1 to-lport 1002 "outport > == @pg1 && ip4 && ip4.src == > check as northd ovn-appctl -t NORTHD_TYPE resume > > AS_BOX([Wait for sw0-p1 to be up]) Update AS_BOX above^ as it does not match the code anymore. > -wait_for_ports_up sw0-p1 > +wait_for_ports_up sw0-p1 sw0-p2 > > # When the port group pg1 is updated, it should not result in > # any assert in ovn-controller. > -ovn-nbctl --wait=hv pg-set-ports pg1 sw0-p1 sw0-p2 sw0-p3 > +ovn-nbctl --wait=hv pg-set-ports pg1 sw0-p1 sw0-p2 sw0-p3 sw0-p4 > AT_CHECK([kill -0 $(cat hv1/ovn-controller.pid)]) > check ovn-nbctl --wait=hv sync > > @@ -25156,40 +25160,42 @@ check ovn-nbctl --wait=hv sync > AT_CHECK([as hv1 ovs-ofctl dump-flows br-int table=45 | ofctl_strip_all | \ > grep "priority=2002" | grep conjunction | \ > sed 's/conjunction([[^)]]*)/conjunction()/g' | sort], [0], [dnl > - table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.3,tp_dst=0x10/0xfff0 > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.3,tp_dst=0x100/0xff00 > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.3,tp_dst=0x1000/0xf000 > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.3,tp_dst=0x2/0xfffe > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.3,tp_dst=0x20/0xffe0 > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.3,tp_dst=0x200/0xfe00 > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.3,tp_dst=0x2000/0xe000 > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.3,tp_dst=0x4/0xfffc > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.3,tp_dst=0x40/0xffc0 > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.3,tp_dst=0x400/0xfc00 > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.3,tp_dst=0x4000/0xc000 > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.3,tp_dst=0x8/0xfff8 > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.3,tp_dst=0x80/0xff80 > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.3,tp_dst=0x800/0xf800 > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.3,tp_dst=0x8000/0x8000 > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.3,tp_dst=1 > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x100/0x100,reg15=0x3,metadata=0x1,nw_src=192.168.47.3 > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.3,tp_dst=0x10/0xfff0 > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.3,tp_dst=0x100/0xff00 > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.3,tp_dst=0x1000/0xf000 > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.3,tp_dst=0x2/0xfffe > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.3,tp_dst=0x20/0xffe0 > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.3,tp_dst=0x200/0xfe00 > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.3,tp_dst=0x2000/0xe000 > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.3,tp_dst=0x4/0xfffc > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.3,tp_dst=0x40/0xffc0 > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.3,tp_dst=0x400/0xfc00 > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.3,tp_dst=0x4000/0xc000 > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.3,tp_dst=0x8/0xfff8 > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.3,tp_dst=0x80/0xff80 > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.3,tp_dst=0x800/0xf800 > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.3,tp_dst=0x8000/0x8000 > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.3,tp_dst=1 > actions=conjunction() > - table=45, > priority=2002,udp,reg0=0x80/0x80,reg15=0x3,metadata=0x1,nw_src=192.168.47.3 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.4,tp_dst=0x10/0xfff0 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.4,tp_dst=0x100/0xff00 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.4,tp_dst=0x1000/0xf000 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.4,tp_dst=0x2/0xfffe > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.4,tp_dst=0x20/0xffe0 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.4,tp_dst=0x200/0xfe00 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.4,tp_dst=0x2000/0xe000 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.4,tp_dst=0x4/0xfffc > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.4,tp_dst=0x40/0xffc0 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.4,tp_dst=0x400/0xfc00 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.4,tp_dst=0x4000/0xc000 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.4,tp_dst=0x8/0xfff8 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.4,tp_dst=0x80/0xff80 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.4,tp_dst=0x800/0xf800 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.4,tp_dst=0x8000/0x8000 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x100/0x100,metadata=0x1,nw_src=192.168.47.4,tp_dst=1 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x100/0x100,reg15=0x3,metadata=0x1,nw_src=192.168.47.4 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x100/0x100,reg15=0x4,metadata=0x1,nw_src=192.168.47.4 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.4,tp_dst=0x10/0xfff0 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.4,tp_dst=0x100/0xff00 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.4,tp_dst=0x1000/0xf000 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.4,tp_dst=0x2/0xfffe > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.4,tp_dst=0x20/0xffe0 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.4,tp_dst=0x200/0xfe00 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.4,tp_dst=0x2000/0xe000 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.4,tp_dst=0x4/0xfffc > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.4,tp_dst=0x40/0xffc0 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.4,tp_dst=0x400/0xfc00 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.4,tp_dst=0x4000/0xc000 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.4,tp_dst=0x8/0xfff8 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.4,tp_dst=0x80/0xff80 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.4,tp_dst=0x800/0xf800 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.4,tp_dst=0x8000/0x8000 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x80/0x80,metadata=0x1,nw_src=192.168.47.4,tp_dst=1 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x80/0x80,reg15=0x3,metadata=0x1,nw_src=192.168.47.4 > actions=conjunction() > + table=45, > priority=2002,udp,reg0=0x80/0x80,reg15=0x4,metadata=0x1,nw_src=192.168.47.4 > actions=conjunction() > ]) > > OVN_CLEANUP([hv1]) >
_______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
