On Thu, Apr 23, 2020 at 6:06 PM Flavio Fernandes <[email protected]> wrote:
>
> The test case "76: ovn -- 4 HV, 1 LS, 1 LR, packet test with HA
> distributed router gateway port" fails sometimes. This patch fixes
> this by leveraging the command ovn-nbctl sync. It also adds a
> delay before sending a packet between foo1 and outside1, so it does
> not get out of order with the gratuitous arp broadcast.
>
> The failed test would look like this:
> <snip>
> checking packets in ext1/vif1-tx.pcap against ext1-vif1.expected:
> ovn.at:12: waiting until $PYTHON "$ovs_srcdir/utilities/ovs-pcap.in"
> $rcv_pcap > $rcv_text
> rcv_n=`wc -l < "$rcv_text"`
> echo "rcv_n=$rcv_n exp_n=$exp_n"
> test $rcv_n -ge $exp_n...
> <snip>
> ovn.at:12: wait succeeded after 2 seconds
> ./ovn.at:8865: sort $rcv_text
> expout 2020-04-22 15:24:29.948703642 -0400
> /home/ffernand/work/ovn/tests/testsuite.dir/at-groups/76/stdout
> 2020-04-22 15:24:29.948703642 -0400
> -1,2 +1,2
>
> -f0000001020400000201020308004500001c000000003f110100c0a80102ac1001030035111100080000
>
> +ffffffffffff00000201020308060001080006040001000002010203ac100101000000000000ac100101
>
> ffffffffffff00000201020308060001080006040001000002010203ac100101000000000000ac100101
> 76. ovn.at:8695: 76. ovn -- 4 HV, 1 LS, 1 LR, packet test with HA
> distributed router gateway port (ovn.at:8695): FAILED (ovn.at:8865)
>
> Reported-by: Dumitru Ceara <[email protected]>
> Signed-off-by: Flavio Fernandes <[email protected]>
> ---
Thanks. I applied this patch to master.
Numan
> tests/ovn.at | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/tests/ovn.at b/tests/ovn.at
> index 2a7ee7528..149345101 100644
> --- a/tests/ovn.at
> +++ b/tests/ovn.at
> @@ -8818,8 +8818,7 @@ as ext1 ovs-vsctl set open .
> external-ids:ovn-bridge-mappings=phys:br-phys
> AT_CHECK([ovn-nbctl --timeout=3 --wait=sb sync], [0], [ignore])
>
> # Allow some time for ovn-northd and ovn-controller to catch up.
> -# XXX This should be more systematic.
> -sleep 2
> +ovn-nbctl --wait=hv sync
>
> ip_to_hex() {
> printf "%02x%02x%02x%02x" "$@"
> @@ -8886,6 +8885,8 @@ grep actions=mod_dl_dst:f0:00:00:01:02:04 | wc -l` -eq 1
> fi
> as ext1 reset_pcap_file ext1-vif1 ext1/vif1
>
> + sleep 1
> +
> # Resend packet from foo1 to outside1
> as hv1 ovs-appctl netdev-dummy/receive hv1-vif1 $packet
>
> --
> 2.17.1
>
> _______________________________________________
> dev mailing list
> [email protected]
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
