From: Donald Sharp Date: Tue, 14 Jun 2022 16:21:19 +0000 (-0400) Subject: tests: Fix verify_rib such that it will look at the selected route X-Git-Tag: base_8.4~349^2 X-Git-Url: https://git.puffer.fish/?a=commitdiff_plain;h=853585a7cd6eeb527fbd8a9ed02c86eb8c7aafa1;p=matthieu%2Ffrr.git tests: Fix verify_rib such that it will look at the selected route When you have a static route with multiple different admin distances there exists a chance that route will have been installed multiple times due to system load when inserted at about the same time. If this is the case then the verify_rib function can and will select the wrong route that happens to have a nexthop group that is still installed. Modify verify_rib to ensure that the route that is going to be looked at for nexthop correctness is the actual installed route, not a previous version of it. Signed-off-by: Donald Sharp --- diff --git a/tests/topotests/lib/common_config.py b/tests/topotests/lib/common_config.py index a7d8b2c9b5..26113b72b1 100644 --- a/tests/topotests/lib/common_config.py +++ b/tests/topotests/lib/common_config.py @@ -3384,6 +3384,9 @@ def verify_rib( next_hop = [next_hop] for mnh in range(0, len(rib_routes_json[st_rt])): + if not "selected" in rib_routes_json[st_rt][mnh]: + continue + if ( "fib" in rib_routes_json[st_rt][mnh]["nexthops"][0]