diff options
author | Ivan Dyukov <i.dyukov@samsung.com> | 2018-06-05 17:37:42 +0300 |
---|---|---|
committer | Ben Pfaff <blp@ovn.org> | 2018-06-14 16:56:27 -0700 |
commit | fcb8664777be142fda9c56fd45f141049fbd0482 (patch) | |
tree | c530cf8d3b270292a0a26bca492aa50e9165544a /tests | |
parent | 19aac14ae4b2479dc96d098069a7ffa15de6cae8 (diff) | |
download | openvswitch-fcb8664777be142fda9c56fd45f141049fbd0482.tar.gz |
tests/stp: Make validation of flows before changing of topology.
The change fixes random stp test failure. Accuracy is about 20%.
Failed test is following:
2337: STP - flush the fdb and mdb when topology changed
In some cases, a validation is executed after topology change and
it increase time of stp stabilization. To prevent this, delay
which wait validation is added before deleting a port.
CC: Tonghao Zhang <xiangxia.m.yue@gmail.com>
Fixes: 427e9751f300 ("tests: Add and improve stp tests.")
Signed-off-by: Ivan Dyukov <i.dyukov@samsung.com>
Signed-off-by: Ben Pfaff <blp@ovn.org>
Diffstat (limited to 'tests')
-rw-r--r-- | tests/stp.at | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/tests/stp.at b/tests/stp.at index 3d85acdb9..7ddacfc3a 100644 --- a/tests/stp.at +++ b/tests/stp.at @@ -562,6 +562,9 @@ OVS_WAIT_UNTIL([ovs-appctl mdb/show br0 | grep 'querier']) OVS_WAIT_UNTIL([ovs-appctl mdb/show br1 | grep 'querier']) OVS_WAIT_UNTIL([ovs-appctl mdb/show br2 | grep 'querier']) +# Make sure that validation of flows was before changing of topology. +ovs-appctl revalidator/wait + # del p2 on the br0, the topology will be changed AT_CHECK([ovs-vsctl del-port br0 p2]) |