Skip to content

Commit

Permalink
topotests: bgp_features, do not test inconsistent 'flags' value
Browse files Browse the repository at this point in the history
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 <[email protected]>
  • Loading branch information
pguibert6WIND committed May 31, 2024
1 parent 74a12f9 commit 66d2111
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion tests/topotests/bgp_features/r1/ip_route.json
Original file line number Diff line number Diff line change
Expand Up @@ -253,7 +253,6 @@
"weight":1
},
{
"flags":1,
"ip":"192.168.1.2",
"afi":"ipv4",
"interfaceName":"r1-eth1",
Expand Down

0 comments on commit 66d2111

Please sign in to comment.