From f823cdea3d80207991fa056b7dadea258c4c0583 Mon Sep 17 00:00:00 2001 From: Philippe Guibert Date: Wed, 6 Mar 2024 17:43:16 +0100 Subject: [PATCH] topotests: bgp_peer_type_multipath, fix duplicate nexthop removed check An issue appears when BGP wants to add the 203.0.113.0/30 prefix from eBGP multiple paths: the 10.0.3.2 nexthop has not the 'fib' flag set, whereas it is expected. The test fails only when BGP nexthop group mode is used. The flag value does not change the behaviour. Only the nexthop which has not the 'duplicate nexthop removed' flag is not installed. Because the test deals with detecting the duplicated nexthop detection, do a check on the 'duplicate' flag. Signed-off-by: Philippe Guibert --- .../bgp_peer_type_multipath_relax/r1/prefix1-eBGP-confed.json | 1 + .../bgp_peer_type_multipath_relax/r1/prefix1-eBGP-iBGP.json | 1 + 2 files changed, 2 insertions(+) diff --git a/tests/topotests/bgp_peer_type_multipath_relax/r1/prefix1-eBGP-confed.json b/tests/topotests/bgp_peer_type_multipath_relax/r1/prefix1-eBGP-confed.json index 22ec2c298b0e..791b92df65ee 100644 --- a/tests/topotests/bgp_peer_type_multipath_relax/r1/prefix1-eBGP-confed.json +++ b/tests/topotests/bgp_peer_type_multipath_relax/r1/prefix1-eBGP-confed.json @@ -24,6 +24,7 @@ }, { "fib":true, + "duplicate":true, "ip":"10.0.3.2", "active":true } diff --git a/tests/topotests/bgp_peer_type_multipath_relax/r1/prefix1-eBGP-iBGP.json b/tests/topotests/bgp_peer_type_multipath_relax/r1/prefix1-eBGP-iBGP.json index facddcda463f..1fe9a6799f75 100644 --- a/tests/topotests/bgp_peer_type_multipath_relax/r1/prefix1-eBGP-iBGP.json +++ b/tests/topotests/bgp_peer_type_multipath_relax/r1/prefix1-eBGP-iBGP.json @@ -24,6 +24,7 @@ }, { "fib":true, + "duplicate":true, "ip":"10.0.3.2", "active":true }