proxmox-mirrors/mirror_frr
1
0
Fork 0
mirror of https://git.proxmox.com/git/mirror_frr synced 2025-08-09 23:48:11 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

doc: Check for convergence, not sleep for convergence

Browse Source 
Signed-off-by: Donald Sharp <sharpd@nvidia.com>
This commit is contained in:
Donald Sharp 2021-04-09 12:06:55 -04:00
parent 750b1916b9
commit 248e7b3af1
1 changed files with 9 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
doc/developer/topotests.rst
View File

@ -480,6 +480,15 @@ Some things to keep in mind:
- Avoid including unstable data in your test: don't rely on link-local - Avoid including unstable data in your test: don't rely on link-local
addresses or ifindex values, for example, because these can change addresses or ifindex values, for example, because these can change
from run to run. from run to run.
- Using sleep is almost never appropriate to wait for some convergence
event as the sole item done. As an example: if the test resets the peers
in BGP, the test should look for the peers reconverging instead of just
sleeping an arbitrary amount of time and continuing on. It is ok to
use sleep in a tight loop with appropriate show commands to ensure that
the protocol reaches the desired state. This should be bounded by
appropriate timeouts for the protocol in question though. See
verify_bgp_convergence as a good example of this. If you are having
troubles figuring out what to look for, please do not be afraid to ask.
Topotest File Hierarchy Topotest File Hierarchy