]> git.proxmox.com Git - mirror_frr.git/commit
tests: Ensure topotests are actually waiting appropriately for peers to come up
authorDonald Sharp <sharpd@cumulusnetworks.com>
Thu, 29 Aug 2019 13:33:47 +0000 (09:33 -0400)
committerDonald Sharp <sharpd@cumulusnetworks.com>
Thu, 29 Aug 2019 13:33:47 +0000 (09:33 -0400)
commit955e2c3ec50b8497e66778caced3b419c3dad366
treec3a766a94d47a37eeec0c169c3e8a0bcda2b10df
parentcf3b331d453c28ddb26a5bdbc53aacf8b5c55fa0
tests: Ensure topotests are actually waiting appropriately for peers to come up

The FRR bgp topotests are employing a luCommand that looks for bgp peering
to be up on the first router with a `wait` sub-command.  Please note that
a variety of tests are using this.  This wait command has a variety of time
outs being used `30`, `90`, and `300`.  BGP peering with how we compile
it have very long timers and 30( and possibly 90) seconds is clearly not enough when
we are waiting, given the nature of our test beds.  Additionally we were employing a model
where once the first summary command succeeded we automatically assumed that all
subsuquent summary commands( to look at other routers ) would not need to
possibly wait.  This is insufficient in that if I have multiple peerings in
multiple vrf's there is no guarantee that one router peers being up will
be sufficient information to know that all the other routers peers are up.

Modify the test cases to be a bit more conformant about this and to
allow peer checks to actually wait a reasonable amount of time for
all peers to have a chance to come up.

Signed-off-by: Donald Sharp <sharpdc@cumulusnetworks.com>
tests/topotests/bgp_l3vpn_to_bgp_direct/scripts/adjacencies.py
tests/topotests/bgp_l3vpn_to_bgp_vrf/scripts/adjacencies.py
tests/topotests/bgp_rfapi_basic_sanity/scripts/adjacencies.py