From 3797454a2a57b710c5eb17fc225d61abbe91c0fc Mon Sep 17 00:00:00 2001 From: Donald Sharp Date: Wed, 28 Aug 2024 15:05:40 -0400 Subject: [PATCH] tests: Ensure bgp suppress fib has a chance to transmit data to peer Giving only 5 seconds to pass bgp data to peers on a heavily loaded system is a recipe for not having fun. Add more time. Signed-off-by: Donald Sharp --- tests/topotests/bgp_suppress_fib/test_bgp_suppress_fib.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/topotests/bgp_suppress_fib/test_bgp_suppress_fib.py b/tests/topotests/bgp_suppress_fib/test_bgp_suppress_fib.py index f7ac827e2633..3932c29b984a 100644 --- a/tests/topotests/bgp_suppress_fib/test_bgp_suppress_fib.py +++ b/tests/topotests/bgp_suppress_fib/test_bgp_suppress_fib.py @@ -140,7 +140,7 @@ def test_bgp_better_admin_won(): topotest.router_json_cmp, r3, "show ip route 40.0.0.0 json", expected ) - _, result = topotest.run_and_expect(test_func, None, count=10, wait=0.5) + _, result = topotest.run_and_expect(test_func, None, count=30, wait=1) assertmsg = '"r3" route to 40.0.0.0 should have been lost' assert result is None, assertmsg @@ -155,7 +155,7 @@ def test_bgp_better_admin_won(): "show ip route 40.0.0.0 json", expected, ) - _, result = topotest.run_and_expect(test_func, None, count=10, wait=0.5) + _, result = topotest.run_and_expect(test_func, None, count=30, wait=1) assertmsg = '"r3" route to 40.0.0.0 did not come back' assert result is None, assertmsg @@ -196,7 +196,7 @@ def test_bgp_allow_as_in(): expected, ) - _, result = topotest.run_and_expect(test_func, None, count=10, wait=0.5) + _, result = topotest.run_and_expect(test_func, None, count=30, wait=1) assertmsg = '"r1" 192.168.1.1/32 route should have arrived' assert result is None, assertmsg @@ -212,7 +212,7 @@ def test_bgp_allow_as_in(): expected, ) - _, result = topotest.run_and_expect(test_func, None, count=10, wait=0.5) + _, result = topotest.run_and_expect(test_func, None, count=30, wait=1) assertmsg = '"r2" 192.168.1.1/32 route should be gone' assert result is None, assertmsg