scottk/wlan-lanforge-scripts
1
0
Fork 0
mirror of https://github.com/Telecominfraproject/wlan-lanforge-scripts.git synced 2025-11-04 04:38:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

lfutils: wait_until_ports_disappear will return True or False

Browse Source 
In all return paths.
And remove redundant check for being done removing ports or not.

Signed-off-by: Ben Greear <greearb@candelatech.com>
This commit is contained in:
Ben Greear
2022-01-29 06:11:51 -08:00
committed by shivam
parent 40bc34f04d
commit 41f587e424
1 changed files with 5 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
py-json/LANforge/LFUtils.py
View File

@@ -526,15 +526,15 @@ def wait_until_ports_admin_up(resource_id=0, base_url="http://localhost:8080", p
return False
def waitUntilPortsDisappear(base_url="http://localhost:8080", port_list=(), debug=False):
wait_until_ports_disappear(base_url, port_list, debug)
def waitUntilPortsDisappear(base_url="http://localhost:8080", port_list=(), debug=False, timeout=360):
wait_until_ports_disappear(base_url, port_list, debug=debug, timeout_sec=timeout)
def wait_until_ports_disappear(base_url="http://localhost:8080", port_list=(), debug=False, timeout_sec=360):
if (port_list is None) or (len(port_list) < 1):
if debug:
logger.debug("LFUtils: wait_until_ports_disappear: empty list, zipping back")
return
logger.debug("LFUtils: wait_until_ports_disappear: empty list, returning")
return True # no ports to remove, so we are done
logger.info("LFUtils: Waiting until {len_port_list} ports disappear...".format(len_port_list=len(port_list)))
url = "/port/1"
@@ -596,11 +596,9 @@ def wait_until_ports_disappear(base_url="http://localhost:8080", port_list=(), d
if debug:
logger.debug('removing port %s' % '.'.join(port))
remove_port(port[1], port[2], base_url)
if len(found_stations) == 0:
return True
sleep(1) # check for ports once per second
logger.info('%s stations were still found' % found_stations)
logger.critical('%s stations were still found' % found_stations)
return False