-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
topotests: add bgp_large_comm_list_match test
Add a topotest to check for proper functioning of the bgp large community list match operation under a route-map. Signed-off-by: Philippe Guibert <[email protected]>
- Loading branch information
1 parent
cd1c3a5
commit 8ba3d83
Showing
8 changed files
with
239 additions
and
0 deletions.
There are no files selected for viewing
Empty file.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,28 @@ | ||
! | ||
router bgp 65001 | ||
no bgp ebgp-requires-policy | ||
neighbor 192.168.0.2 remote-as external | ||
neighbor 192.168.0.2 timers 1 3 | ||
neighbor 192.168.0.2 timers connect 1 | ||
address-family ipv4 | ||
redistribute connected | ||
neighbor 192.168.0.2 route-map r2 out | ||
exit-address-family | ||
! | ||
ip prefix-list p1 seq 5 permit 172.16.255.1/32 | ||
ip prefix-list p3 seq 5 permit 172.16.255.3/32 | ||
ip prefix-list p4 seq 5 permit 172.16.255.4/32 | ||
! | ||
route-map r2 permit 10 | ||
match ip address prefix-list p1 | ||
set large-community 65001:1:1 65001:2:1 | ||
route-map r2 permit 20 | ||
match ip address prefix-list p3 | ||
set large-community 65001:3:1 | ||
route-map r2 permit 30 | ||
match ip address prefix-list p4 | ||
set large-community 65001:10:1 65001:12:1 65001:13:1 | ||
exit | ||
route-map r2 permit 40 | ||
exit | ||
! |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,12 @@ | ||
! | ||
interface lo | ||
ip address 172.16.255.1/32 | ||
ip address 172.16.255.2/32 | ||
ip address 172.16.255.3/32 | ||
ip address 172.16.255.4/32 | ||
! | ||
interface r1-eth0 | ||
ip address 192.168.0.1/24 | ||
! | ||
ip forwarding | ||
! |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,24 @@ | ||
! | ||
!debug bgp updates | ||
! | ||
router bgp 65002 | ||
no bgp ebgp-requires-policy | ||
neighbor 192.168.0.1 remote-as external | ||
neighbor 192.168.0.1 timers 1 3 | ||
neighbor 192.168.0.1 timers connect 1 | ||
neighbor 192.168.1.3 remote-as external | ||
neighbor 192.168.1.3 timers 1 3 | ||
neighbor 192.168.1.3 timers connect 1 | ||
address-family ipv4 | ||
neighbor 192.168.0.1 route-map r1 in | ||
neighbor 192.168.0.1 soft-reconfiguration inbound | ||
exit-address-family | ||
! | ||
bgp large-community-list 1 seq 5 permit 65001:1:1 65001:2:1 | ||
bgp large-community-list 1 seq 10 permit 65001:3:1 | ||
! | ||
route-map r1 deny 10 | ||
match large-community 1 | ||
route-map r1 permit 20 | ||
exit | ||
! |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,9 @@ | ||
! | ||
interface r2-eth0 | ||
ip address 192.168.0.2/24 | ||
! | ||
interface r2-eth1 | ||
ip address 192.168.1.2/24 | ||
! | ||
ip forwarding | ||
! |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,21 @@ | ||
! | ||
!debug bgp updates | ||
! | ||
router bgp 65003 | ||
no bgp ebgp-requires-policy | ||
neighbor 192.168.1.2 remote-as external | ||
neighbor 192.168.1.2 timers 1 3 | ||
neighbor 192.168.1.2 timers connect 1 | ||
address-family ipv4 | ||
neighbor 192.168.1.2 route-map r1 in | ||
neighbor 192.168.1.2 soft-reconfiguration inbound | ||
exit-address-family | ||
! | ||
bgp large-community-list 2 seq 10 permit 65001:12:1 | ||
! | ||
route-map r1 deny 10 | ||
match large-community 2 any | ||
exit | ||
route-map r1 permit 20 | ||
exit | ||
! |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
! | ||
interface r3-eth0 | ||
ip address 192.168.1.3/24 | ||
! | ||
ip forwarding | ||
! |
139 changes: 139 additions & 0 deletions
139
tests/topotests/bgp_large_comm_list_match/test_bgp_large_comm_list_match.py
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,139 @@ | ||
#!/usr/bin/env python | ||
# SPDX-License-Identifier: ISC | ||
|
||
# | ||
# Copyright 2023 by 6WIND S.A. | ||
# | ||
|
||
""" | ||
Check if BGP large-community-list works as OR if multiple large-community entries specified, | ||
like: | ||
bgp large-community-list 1 seq 5 permit 65001:1:1 65002:2:1 | ||
bgp large-community-list 1 seq 10 permit 65001:3:1 | ||
! | ||
route-map test deny 10 | ||
match community 1 | ||
route-map test permit 20 | ||
Here, we should deny routes in/out if the path has: | ||
(65001:1:1 OR 65001:2:1) OR 65001:3:1. | ||
""" | ||
|
||
import os | ||
import sys | ||
import json | ||
import pytest | ||
import functools | ||
|
||
CWD = os.path.dirname(os.path.realpath(__file__)) | ||
sys.path.append(os.path.join(CWD, "../")) | ||
|
||
# pylint: disable=C0413 | ||
from lib import topotest | ||
from lib.topogen import Topogen, TopoRouter, get_topogen | ||
from lib.common_config import step | ||
|
||
pytestmark = [pytest.mark.bgpd] | ||
|
||
|
||
def build_topo(tgen): | ||
for routern in range(1, 4): | ||
tgen.add_router("r{}".format(routern)) | ||
|
||
switch = tgen.add_switch("s1") | ||
switch.add_link(tgen.gears["r1"]) | ||
switch.add_link(tgen.gears["r2"]) | ||
switch = tgen.add_switch("s2") | ||
switch.add_link(tgen.gears["r3"]) | ||
switch.add_link(tgen.gears["r2"]) | ||
|
||
|
||
def setup_module(mod): | ||
tgen = Topogen(build_topo, mod.__name__) | ||
tgen.start_topology() | ||
|
||
router_list = tgen.routers() | ||
|
||
for i, (rname, router) in enumerate(router_list.items(), 1): | ||
router.load_config( | ||
TopoRouter.RD_ZEBRA, os.path.join(CWD, "{}/zebra.conf".format(rname)) | ||
) | ||
router.load_config( | ||
TopoRouter.RD_BGP, os.path.join(CWD, "{}/bgpd.conf".format(rname)) | ||
) | ||
|
||
tgen.start_router() | ||
|
||
|
||
def teardown_module(mod): | ||
tgen = get_topogen() | ||
tgen.stop_topology() | ||
|
||
|
||
def test_bgp_large_comm_list_match(): | ||
tgen = get_topogen() | ||
|
||
if tgen.routers_have_failure(): | ||
pytest.skip(tgen.errors) | ||
|
||
router = tgen.gears["r2"] | ||
|
||
def _bgp_converge(): | ||
output = json.loads( | ||
router.vtysh_cmd( | ||
"show bgp ipv4 unicast neighbors 192.168.0.1 filtered-routes json" | ||
) | ||
) | ||
expected = { | ||
"receivedRoutes": { | ||
"172.16.255.1/32": { | ||
"path": "65001", | ||
}, | ||
"172.16.255.3/32": { | ||
"path": "65001", | ||
}, | ||
} | ||
} | ||
return topotest.json_cmp(output, expected) | ||
|
||
step("Initial BGP converge between R1 and R2") | ||
test_func = functools.partial(_bgp_converge) | ||
_, result = topotest.run_and_expect(test_func, None, count=60, wait=0.5) | ||
assert ( | ||
result is None | ||
), "Failed to filter BGP UPDATES with large-community-list on R2" | ||
|
||
|
||
def test_bgp_large_comm_list_match_any(): | ||
tgen = get_topogen() | ||
|
||
if tgen.routers_have_failure(): | ||
pytest.skip(tgen.errors) | ||
|
||
router = tgen.gears["r3"] | ||
|
||
def _bgp_converge(): | ||
output = json.loads( | ||
router.vtysh_cmd( | ||
"show bgp ipv4 unicast neighbors 192.168.1.2 filtered-routes json" | ||
) | ||
) | ||
expected = { | ||
"receivedRoutes": { | ||
"172.16.255.4/32": { | ||
"path": "65002 65001", | ||
}, | ||
} | ||
} | ||
return topotest.json_cmp(output, expected) | ||
|
||
step("Initial BGP converge between R3 and R2") | ||
test_func = functools.partial(_bgp_converge) | ||
_, result = topotest.run_and_expect(test_func, None, count=60, wait=0.5) | ||
assert result is None, "Failed to filter BGP UPDATES with large-community-list on R3" | ||
|
||
|
||
if __name__ == "__main__": | ||
args = ["-s"] + sys.argv[1:] | ||
sys.exit(pytest.main(args)) |