Skip to content

Commit ecf1586

Browse files
yue-fred-gaodcaugher
authored andcommitted
Issue sonic-net#21205: Fix swss memory usage check failure in test_bgp_stress_link_flap in sonic-vpp (sonic-net#21206)
Description of PR Summary: Fixes sonic-net#21205 Type of change Bug fix Testbed and Framework(new/improvement) New Test case Skipped for non-supported platforms Test case improvement Approach What is the motivation for this PR? Makes test_bgp_stress_link_flap more robust in sonic-vpp How did you do it? Increase delay for sonic-vpp before teardown. How did you verify/test it? Run the test multiple times and it passed Any platform specific information? specific to sonic-vpp platform signed-off-by: [email protected]
1 parent a7a42f9 commit ecf1586

File tree

1 file changed

+8
-4
lines changed

1 file changed

+8
-4
lines changed

tests/bgp/test_bgp_stress_link_flap.py

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -243,10 +243,10 @@ def test_bgp_stress_link_flap(duthosts, rand_one_dut_hostname, setup, nbrhosts,
243243

244244
# Skip the test on Virtual Switch due to fanout switch dependency and warm reboot
245245
asic_type = duthost.facts['asic_type']
246-
if asic_type == "vs" and (test_type == "fanout" or test_type == "all"):
246+
if (asic_type == "vs" or asic_type == "vpp") and (test_type == "fanout" or test_type == "all"):
247247
pytest.skip("Stress link flap test is not supported on Virtual Switch")
248248

249-
if asic_type != "vs":
249+
if asic_type != "vs" and asic_type != "vpp":
250250
delay_time = SLEEP_DURATION
251251
else:
252252
delay_time = SLEEP_DURATION * 100
@@ -318,7 +318,11 @@ async def flap_interfaces():
318318

319319
asyncio.run(flap_interfaces())
320320

321-
logger.info("Test Completed, waiting for 60 seconds to stabilize the system")
322-
time.sleep(60)
321+
if asic_type == "vpp":
322+
sleep_time = 180
323+
else:
324+
sleep_time = 60
325+
logger.info("Test Completed, waiting for {} seconds to stabilize the system".format(sleep_time))
326+
time.sleep(sleep_time)
323327

324328
return

0 commit comments

Comments
 (0)