Merge pull request #6703 from mjstapp/fix_bgp_30_secs

tests: print actual time spent waiting for bgp convergence
This commit is contained in:
Donald Sharp 2020-07-09 20:29:08 -04:00 committed by GitHub
commit eb94db103c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -1467,9 +1467,10 @@ def clear_bgp_and_verify(tgen, topo, router):
rnode = tgen.routers()[router]
peer_uptime_before_clear_bgp = {}
sleeptime = 3
# Verifying BGP convergence before bgp clear command
for retry in range(44):
sleeptime = 3
# Waiting for BGP to converge
logger.info(
"Waiting for %s sec for BGP to converge on router" " %s...",
@ -1536,8 +1537,8 @@ def clear_bgp_and_verify(tgen, topo, router):
)
else:
errormsg = (
"TIMEOUT!! BGP is not converged in 30 seconds for"
" router {}".format(router)
"TIMEOUT!! BGP is not converged in {} seconds for"
" router {}".format(retry * sleeptime, router)
)
return errormsg
@ -1552,7 +1553,7 @@ def clear_bgp_and_verify(tgen, topo, router):
peer_uptime_after_clear_bgp = {}
# Verifying BGP convergence after bgp clear command
for retry in range(44):
sleeptime = 3
# Waiting for BGP to converge
logger.info(
"Waiting for %s sec for BGP to converge on router" " %s...",
@ -1615,8 +1616,8 @@ def clear_bgp_and_verify(tgen, topo, router):
)
else:
errormsg = (
"TIMEOUT!! BGP is not converged in 30 seconds for"
" router {}".format(router)
"TIMEOUT!! BGP is not converged in {} seconds for"
" router {}".format(retry * sleeptime, router)
)
return errormsg