From 473cda35bb2294107e0ad89baa5ec45676218461 Mon Sep 17 00:00:00 2001 From: Ejub Sabic Date: Tue, 16 Jun 2026 08:37:44 +0200 Subject: [PATCH] fix: flaky dhcp option 121 test This commit increases attempts on checks in this particular test from default 10 to 60. This has been introduced to match worst case scenario where after 3 retries(5s) each dhcp waits 30 seconds before next 3 retries. Commits yields to much more stable test behaviour. Resolves: #1515 Signed-off-by: Ejub Sabic --- test/case/dhcp/client_routes/test.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/case/dhcp/client_routes/test.py b/test/case/dhcp/client_routes/test.py index ec3aef26..452963a8 100755 --- a/test/case/dhcp/client_routes/test.py +++ b/test/case/dhcp/client_routes/test.py @@ -81,7 +81,7 @@ with infamy.Test() as test: with infamy.dhcp.Server(netns, prefix=PREFIX, router=ROUTER): with test.step("Verify client has route 10.0.0.0/24 via 192.168.0.254 (option 121)"): print("Verify client use classless routes, option 121") - until(lambda: route.ipv4_route_exist(client, PREFIX, ROUTER)) + until(lambda: route.ipv4_route_exist(client, PREFIX, ROUTER), attempts=60) with test.step("Verify client has default route via 192.168.0.254 (not use option 3)"): print("Verify client did *not* use option 3") @@ -90,6 +90,6 @@ with infamy.Test() as test: with test.step("Verify client still has canary route to 20.0.0.0/24 via 192.168.0.2"): until(lambda: route.ipv4_route_exist(client, CANARY, - CANHOP, pref=250)) + CANHOP, pref=250), attempts=60) test.succeed()