]> git.puffer.fish Git - matthieu/frr.git/commit
tests: Do not do a initial_timeout of 2 seconds for verify_bgp_rib
authorDonald Sharp <sharpd@nvidia.com>
Thu, 16 Sep 2021 19:12:13 +0000 (15:12 -0400)
committerDonald Sharp <sharpd@nvidia.com>
Thu, 16 Sep 2021 19:12:13 +0000 (15:12 -0400)
commit4367df157529b3c0f556333bbe41ccd125bd7c6c
treefa18960a68e89812dc18786b5f4b3e268c9bf3de
parent68a8bd50d415e6f44b810a3ad21d646c7fb10da4
tests: Do not do a initial_timeout of 2 seconds for verify_bgp_rib

A bunch of tests have this pattern:

a) Install a new prefix into bgp
b) Run this loop:
foreach (router in topology) {
verify_bgp_rib(router)
}

This is to ensure that the prefix is actually disseminated.
The problem with this, of course, is that a wait of 2 seconds
for every item in that loop makes no sense.  As that the initial
router verification of it's bgp rib will wait 2 seconds and
all the remaining bgp routers in the topology will have gotten
the data.  So we end up waiting a bunch of extra time.

Remove the initial_wait time for verify_bgp_rib.  Also
increase the failure wait time to 30 seconds.  This is
to give a bigger window for bgp to send it's data for
our test systems that could be under heavy load.  In the
normal case tests will never hit this.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>
tests/topotests/lib/bgp.py