From 66d21113155bf8fefdeddd4bb366e7fc82ec4c79 Mon Sep 17 00:00:00 2001 From: Philippe Guibert Date: Thu, 7 Mar 2024 11:02:37 +0100 Subject: [PATCH] topotests: bgp_features, do not test inconsistent 'flags' value After disabling the 'bgp suppress-no-fib' service, the test fails to find the expected bgp route to '192.168.0.7': two paths are available (from BGP and OSPF), and only the OSPF one is selected, but the calculated recursive nexthop for BGP should not be installed. Actually, in the FIB, the '192.168.0.7' prefix selected is the one from OSPF. In the test, the bgp nexthop-group functionality is turned on: The BGP route for '192.168.0.7' uses a shared recursive nexthop which is already installed. But because the BGP route has a bigger distance than ospf, the BGP route is not installed. Because the nexthop is shared and not self to the route, do ignore the flags value of the nexthop, in this test. Signed-off-by: Philippe Guibert --- tests/topotests/bgp_features/r1/ip_route.json | 1 - 1 file changed, 1 deletion(-) diff --git a/tests/topotests/bgp_features/r1/ip_route.json b/tests/topotests/bgp_features/r1/ip_route.json index 045c723eedca..4913da828e33 100644 --- a/tests/topotests/bgp_features/r1/ip_route.json +++ b/tests/topotests/bgp_features/r1/ip_route.json @@ -253,7 +253,6 @@ "weight":1 }, { - "flags":1, "ip":"192.168.1.2", "afi":"ipv4", "interfaceName":"r1-eth1",