summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDonald Sharp <sharpd@nvidia.com>2022-05-19 19:28:43 -0400
committerDonald Sharp <sharpd@nvidia.com>2022-05-20 10:31:48 -0400
commit006ef829d84f3d9d0a2bee486af1aca987dec36c (patch)
tree69ec537b6b15d3249717c4afb8f652bdcb0a6605
parentcb8018f4c3f07cd88946c1d248af34de15eaacc4 (diff)
tests: Allow a bit longer for bfd topo tests to synchronize
Allowing only 4 seconds for a bfd test to synchronize is going to run into problems on extremely loaded systems. The test system should value it actually converged over it actually converged in a reasonable time, especially on test systems that are loaded because of many multiples of tests running at the same time. If it is important to actually test that something got done by the RFC, the CI system as it is currently written is not the correct place for this. Signed-off-by: Donald Sharp <sharpd@nvidia.com>
-rw-r--r--tests/topotests/bfd_topo1/test_bfd_topo1.py2
-rw-r--r--tests/topotests/bfd_topo2/test_bfd_topo2.py2
2 files changed, 2 insertions, 2 deletions
diff --git a/tests/topotests/bfd_topo1/test_bfd_topo1.py b/tests/topotests/bfd_topo1/test_bfd_topo1.py
index adf02b02d4..c9020f16d3 100644
--- a/tests/topotests/bfd_topo1/test_bfd_topo1.py
+++ b/tests/topotests/bfd_topo1/test_bfd_topo1.py
@@ -100,7 +100,7 @@ def test_bfd_connection():
test_func = partial(
topotest.router_json_cmp, router, "show bfd peers json", expected
)
- _, result = topotest.run_and_expect(test_func, None, count=8, wait=0.5)
+ _, result = topotest.run_and_expect(test_func, None, count=30, wait=0.5)
assertmsg = '"{}" JSON output mismatches'.format(router.name)
assert result is None, assertmsg
diff --git a/tests/topotests/bfd_topo2/test_bfd_topo2.py b/tests/topotests/bfd_topo2/test_bfd_topo2.py
index 57ce0cdf09..a9b9358ef0 100644
--- a/tests/topotests/bfd_topo2/test_bfd_topo2.py
+++ b/tests/topotests/bfd_topo2/test_bfd_topo2.py
@@ -144,7 +144,7 @@ def test_bfd_connection():
test_func = partial(
topotest.router_json_cmp, router, "show bfd peers json", expected
)
- _, result = topotest.run_and_expect(test_func, None, count=8, wait=0.5)
+ _, result = topotest.run_and_expect(test_func, None, count=30, wait=0.5)
assertmsg = '"{}" JSON output mismatches'.format(router.name)
assert result is None, assertmsg