mirror of
https://git.proxmox.com/git/mirror_frr
synced 2025-08-16 08:32:45 +00:00
Merge pull request #11227 from donaldsharp/verify_not_queued
tests: Ensure routes are not queued when calling verify_rib
This commit is contained in:
commit
8eeadd88e9
@ -100,7 +100,7 @@ def test_bfd_connection():
|
|||||||
test_func = partial(
|
test_func = partial(
|
||||||
topotest.router_json_cmp, router, "show bfd peers json", expected
|
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)
|
assertmsg = '"{}" JSON output mismatches'.format(router.name)
|
||||||
assert result is None, assertmsg
|
assert result is None, assertmsg
|
||||||
|
|
||||||
|
@ -144,7 +144,7 @@ def test_bfd_connection():
|
|||||||
test_func = partial(
|
test_func = partial(
|
||||||
topotest.router_json_cmp, router, "show bfd peers json", expected
|
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)
|
assertmsg = '"{}" JSON output mismatches'.format(router.name)
|
||||||
assert result is None, assertmsg
|
assert result is None, assertmsg
|
||||||
|
|
||||||
|
@ -3245,7 +3245,7 @@ def configure_interface_mac(tgen, input_dict):
|
|||||||
#############################################
|
#############################################
|
||||||
# Verification APIs
|
# Verification APIs
|
||||||
#############################################
|
#############################################
|
||||||
@retry(retry_timeout=12)
|
@retry(retry_timeout=40)
|
||||||
def verify_rib(
|
def verify_rib(
|
||||||
tgen,
|
tgen,
|
||||||
addr_type,
|
addr_type,
|
||||||
@ -3383,6 +3383,10 @@ def verify_rib(
|
|||||||
st_found = True
|
st_found = True
|
||||||
found_routes.append(st_rt)
|
found_routes.append(st_rt)
|
||||||
|
|
||||||
|
if "queued" in rib_routes_json[st_rt][0]:
|
||||||
|
errormsg = "Route {} is queued\n".format(st_rt)
|
||||||
|
return errormsg
|
||||||
|
|
||||||
if fib and next_hop:
|
if fib and next_hop:
|
||||||
if type(next_hop) is not list:
|
if type(next_hop) is not list:
|
||||||
next_hop = [next_hop]
|
next_hop = [next_hop]
|
||||||
@ -3607,6 +3611,10 @@ def verify_rib(
|
|||||||
st_found = True
|
st_found = True
|
||||||
found_routes.append(st_rt)
|
found_routes.append(st_rt)
|
||||||
|
|
||||||
|
if "queued" in rib_routes_json[st_rt][0]:
|
||||||
|
errormsg = "Route {} is queued\n".format(st_rt)
|
||||||
|
return errormsg
|
||||||
|
|
||||||
if next_hop:
|
if next_hop:
|
||||||
if type(next_hop) is not list:
|
if type(next_hop) is not list:
|
||||||
next_hop = [next_hop]
|
next_hop = [next_hop]
|
||||||
|
Loading…
Reference in New Issue
Block a user