From ba0026272cee5a863060fa978bcbcdc57fe6b21e Mon Sep 17 00:00:00 2001 From: Donald Sharp Date: Tue, 27 Jul 2021 09:38:14 -0400 Subject: [PATCH] tests: Increase timeout for loaded systems When running this test on a locally loaded system I am seeing the static route as `queued` still after 1 second. Let's just blanket increase the timeout to something longer to give a very loaded system more time to install the route. Output on my test system when it was loaded: INFO topolog.r1:topogen.py:880 vtysh result: { "4.5.1.0/24":[ { "prefix":"4.5.1.0/24", "prefixLen":24, "protocol":"static", "vrfId":0, "vrfName":"default", "selected":true, "destSelected":true, "distance":1, "metric":0, "queued":true, "table":254, "internalStatus":8, "internalFlags":73, "internalNextHopNum":1, "internalNextHopActiveNum":1, "uptime":"00:00:00", "nexthops":[ { "flags":1, "ip":"192.168.216.3", "afi":"ipv4", "interfaceIndex":11, "interfaceName":"r1-eth6", "active":true, "weight":1 } ] }, I suspect 10 seconds should be enough( I would hope ). Signed-off-by: Donald Sharp --- tests/topotests/zebra_rib/test_zebra_rib.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/topotests/zebra_rib/test_zebra_rib.py b/tests/topotests/zebra_rib/test_zebra_rib.py index 9fcf7b6820..7ae95730d5 100644 --- a/tests/topotests/zebra_rib/test_zebra_rib.py +++ b/tests/topotests/zebra_rib/test_zebra_rib.py @@ -124,7 +124,7 @@ def test_zebra_kernel_admin_distance(): "show ip route 4.5.{}.0 json".format(i), expected, ) - _, result = topotest.run_and_expect(test_func, None, count=2, wait=0.5) + _, result = topotest.run_and_expect(test_func, None, count=20, wait=0.5) assertmsg = '"r1" JSON output mismatches' assert result is None, assertmsg # tgen.mininet_cli() @@ -145,7 +145,7 @@ def test_zebra_kernel_override(): test_func = partial( topotest.router_json_cmp, r1, "show ip route 4.5.1.0 json", expected ) - _, result = topotest.run_and_expect(test_func, None, count=2, wait=0.5) + _, result = topotest.run_and_expect(test_func, None, count=20, wait=0.5) assert result is None, '"r1" JSON output mismatches' logger.info( @@ -158,7 +158,7 @@ def test_zebra_kernel_override(): test_func = partial( topotest.router_json_cmp, r1, "show ip route 4.5.1.0 json", expected ) - _, result = topotest.run_and_expect(test_func, None, count=2, wait=0.5) + _, result = topotest.run_and_expect(test_func, None, count=20, wait=0.5) assert result is None, '"r1" JSON output mismatches' -- 2.39.5