[ovs-dev] [PATCH ovn v10 8/8] tests: Enhance ovn-performance testing by adding gw router port.

Dumitru Ceara dceara at redhat.com
Tue Jun 9 13:10:31 UTC 2020


On 6/9/20 1:25 PM, Numan Siddique wrote:
> 
> 
> On Tue, Jun 9, 2020 at 4:21 PM Dumitru Ceara <dceara at redhat.com
> <mailto:dceara at redhat.com>> wrote:
> 
>     On 6/8/20 3:50 PM, numans at ovn.org <mailto:numans at ovn.org> wrote:
>     > From: Numan Siddique <numans at ovn.org <mailto:numans at ovn.org>>
>     >
>     > This covers the scenario of setting up/deleting of BFD tunnels
>     > for HA.
>     >
>     > Signed-off-by: Numan Siddique <numans at ovn.org <mailto:numans at ovn.org>>
> 
>     Hi Numan,
> 
>     Should this have a "Fixes" tag?
> 
> 
> I don't think so. This just adds additional tests.
> 
> Thanks
> Numan
>  

I see, ok, the new tests pass for me:

Tested-by: Dumitru Ceara <dceara at redhat.com>

Thanks,
Dumitru

> 
>     Thanks,
>     Dumitru
> 
>     > ---
>     >  tests/ovn-performance.at <http://ovn-performance.at> | 104
>     +++++++++++++++++++++++++++++++++++++++
>     >  1 file changed, 104 insertions(+)
>     >
>     > diff --git a/tests/ovn-performance.at <http://ovn-performance.at>
>     b/tests/ovn-performance.at <http://ovn-performance.at>
>     > index a12757e18..2a15cb473 100644
>     > --- a/tests/ovn-performance.at <http://ovn-performance.at>
>     > +++ b/tests/ovn-performance.at <http://ovn-performance.at>
>     > @@ -239,6 +239,16 @@ for i in 1 2; do
>     >      ovn_attach n1 br-phys 192.168.0.$i
>     >  done
>>     > +for i in 1 2 3; do
>     > +    sim_add gw$i
>     > +    as gw$i
>     > +    ovs-vsctl add-br br-phys
>     > +    ovs-vsctl add-br br-ex
>     > +    ovs-vsctl set open .
>     external_ids:ovn-bridge-mappings="public:br-ex"
>     > +    j=$((i + 2))
>     > +    ovn_attach n1 br-phys 192.168.0.$j
>     > +done
>     > +
>     >  # Wait for the tunnel ports to be created and up.
>     >  # Otherwise this may affect the lflow_run count.
>>     > @@ -399,6 +409,100 @@ OVN_CONTROLLER_EXPECT_NO_HIT(
>     >      [ovn-nbctl --wait=hv acl-add pg1 to-lport 1001 'outport ==
>     @pg1 && ip4.src == $pg1_ip4' allow]
>     >  )
>>     > +# Create a public logical switch and attach the router to it.
>     > +OVN_CONTROLLER_EXPECT_NO_HIT(
>     > +    [hv1 hv2], [lflow_run],
>     > +    [ovn-nbctl --wait=hv ls-add public]
>     > +)
>     > +
>     > +OVN_CONTROLLER_EXPECT_NO_HIT(
>     > +    [hv1 hv2], [lflow_run],
>     > +    [ovn-nbctl --wait=hv lsp-add public public-lr1]
>     > +)
>     > +
>     > +OVN_CONTROLLER_EXPECT_NO_HIT(
>     > +    [hv1 hv2], [lflow_run],
>     > +    [ovn-nbctl --wait=hv lsp-set-type public-lr1 router]
>     > +)
>     > +
>     > +OVN_CONTROLLER_EXPECT_NO_HIT(
>     > +    [hv1 hv2], [lflow_run],
>     > +    [ovn-nbctl --wait=hv lsp-set-addresses public-lr1 router]
>     > +)
>     > +
>     > +OVN_CONTROLLER_EXPECT_NO_HIT(
>     > +    [hv1 hv2], [lflow_run],
>     > +    [ovn-nbctl --wait=hv lsp-set-options public-lr1
>     router-port=lr1-public]
>     > +)
>     > +
>     > +OVN_CONTROLLER_EXPECT_NO_HIT(
>     > +    [hv1 hv2], [lflow_run],
>     > +    [ovn-nbctl --wait=hv lrp-add lr1 lr1-public 00:00:20:20:12:13
>     172.168.0.100/24 <http://172.168.0.100/24>]
>     > +)
>     > +
>     > +OVN_CONTROLLER_EXPECT_NO_HIT(
>     > +    [hv1 hv2], [lflow_run],
>     > +    [ovn-nbctl --wait=hv lsp-add public ln-public]
>     > +)
>     > +
>     > +OVN_CONTROLLER_EXPECT_NO_HIT(
>     > +    [hv1 hv2], [lflow_run],
>     > +    [ovn-nbctl --wait=hv lsp-set-type ln-public localnet]
>     > +)
>     > +
>     > +OVN_CONTROLLER_EXPECT_NO_HIT(
>     > +    [hv1 hv2], [lflow_run],
>     > +    [ovn-nbctl --wait=hv lsp-set-addresses ln-public unknown]
>     > +)
>     > +
>     > +OVN_CONTROLLER_EXPECT_NO_HIT(
>     > +    [hv1 hv2], [lflow_run],
>     > +    [ovn-nbctl --wait=hv lsp-set-options ln-public
>     network_name=public]
>     > +)
>     > +
>     > +OVN_CONTROLLER_EXPECT_HIT_COND(
>     > +    [hv1 hv2 gw1 gw2 gw3], [lflow_run], [=0 =0 >0 =0 =0],
>     > +    [ovn-nbctl --wait=hv lrp-set-gateway-chassis lr1-public gw1 30]
>     > +)
>     > +
>     > +# After this, BFD should be enabled from hv1 and hv2 to gw1.
>     > +# So there should be lflow_run hits in hv1, hv2, gw1 and gw2
>     > +OVN_CONTROLLER_EXPECT_HIT_COND(
>     > +    [hv1 hv2 gw1 gw2 gw3], [lflow_run], [>0 >0 >0 >0 =0],
>     > +    [ovn-nbctl --wait=hv lrp-set-gateway-chassis lr1-public gw2 20]
>     > +)
>     > +
>     > +OVN_CONTROLLER_EXPECT_HIT(
>     > +    [hv1 hv2 gw1 gw2 gw3], [lflow_run],
>     > +    [ovn-nbctl --wait=hv lrp-set-gateway-chassis lr1-public gw3 10]
>     > +)
>     > +
>     > +# Make gw2 master.
>     > +OVN_CONTROLLER_EXPECT_NO_HIT(
>     > +    [hv1 hv2 gw1 gw2 gw3], [lflow_run],
>     > +    [ovn-nbctl --wait=hv lrp-set-gateway-chassis lr1-public gw2 40]
>     > +)
>     > +
>     > +# Delete gw2 from gateway chassis
>     > +OVN_CONTROLLER_EXPECT_HIT(
>     > +    [hv1 hv2 gw1 gw2 gw3], [lflow_run],
>     > +    [ovn-nbctl --wait=hv lrp-del-gateway-chassis lr1-public gw2]
>     > +)
>     > +
>     > +# Delete gw1 from gateway chassis
>     > +# After this, the BFD should be disabled entirely as gw3 is the
>     > +# only gateway chassis.
>     > +OVN_CONTROLLER_EXPECT_HIT_COND(
>     > +    [hv1 hv2 gw1 gw2 gw3], [lflow_run],  [>0 >0 >0 =0 >0],
>     > +    [ovn-nbctl --wait=hv lrp-del-gateway-chassis lr1-public gw1]
>     > +)
>     > +
>     > +# Delete gw3 from gateway chassis. There should be no lflow_run.
>     > +OVN_CONTROLLER_EXPECT_NO_HIT(
>     > +    [hv1 hv2 gw1 gw2 gw3], [lflow_run],
>     > +    [ovn-nbctl --wait=hv lrp-del-gateway-chassis lr1-public gw3]
>     > +)
>     > +
>     >  for i in 1 2; do
>     >      j=$((i%2 + 1))
>     >      lp=lp$i
>     >
> 
>     _______________________________________________
>     dev mailing list
>     dev at openvswitch.org <mailto:dev at openvswitch.org>
>     https://mail.openvswitch.org/mailman/listinfo/ovs-dev
> 



More information about the dev mailing list