mirror of
https://git.proxmox.com/git/mirror_frr
synced 2025-07-25 15:18:24 +00:00
tests: Add a test case for BGP AS-PATH 0 in the path
We should withdraw prefixes with ASN 0 within the path. Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
This commit is contained in:
parent
88cc0ce4e8
commit
d0046e9acc
0
tests/topotests/bgp_aspath_zero/__init__.py
Normal file
0
tests/topotests/bgp_aspath_zero/__init__.py
Normal file
53
tests/topotests/bgp_aspath_zero/exabgp.env
Normal file
53
tests/topotests/bgp_aspath_zero/exabgp.env
Normal file
@ -0,0 +1,53 @@
|
|||||||
|
[exabgp.api]
|
||||||
|
encoder = text
|
||||||
|
highres = false
|
||||||
|
respawn = false
|
||||||
|
socket = ''
|
||||||
|
|
||||||
|
[exabgp.bgp]
|
||||||
|
openwait = 60
|
||||||
|
|
||||||
|
[exabgp.cache]
|
||||||
|
attributes = true
|
||||||
|
nexthops = true
|
||||||
|
|
||||||
|
[exabgp.daemon]
|
||||||
|
daemonize = true
|
||||||
|
pid = '/var/run/exabgp/exabgp.pid'
|
||||||
|
user = 'exabgp'
|
||||||
|
##daemonize = false
|
||||||
|
|
||||||
|
[exabgp.log]
|
||||||
|
all = false
|
||||||
|
configuration = true
|
||||||
|
daemon = true
|
||||||
|
destination = '/var/log/exabgp.log'
|
||||||
|
enable = true
|
||||||
|
level = INFO
|
||||||
|
message = false
|
||||||
|
network = true
|
||||||
|
packets = false
|
||||||
|
parser = false
|
||||||
|
processes = true
|
||||||
|
reactor = true
|
||||||
|
rib = false
|
||||||
|
routes = false
|
||||||
|
short = false
|
||||||
|
timers = false
|
||||||
|
|
||||||
|
[exabgp.pdb]
|
||||||
|
enable = false
|
||||||
|
|
||||||
|
[exabgp.profile]
|
||||||
|
enable = false
|
||||||
|
file = ''
|
||||||
|
|
||||||
|
[exabgp.reactor]
|
||||||
|
speed = 1.0
|
||||||
|
|
||||||
|
[exabgp.tcp]
|
||||||
|
acl = false
|
||||||
|
bind = ''
|
||||||
|
delay = 0
|
||||||
|
once = false
|
||||||
|
port = 179
|
17
tests/topotests/bgp_aspath_zero/peer1/exabgp.cfg
Normal file
17
tests/topotests/bgp_aspath_zero/peer1/exabgp.cfg
Normal file
@ -0,0 +1,17 @@
|
|||||||
|
neighbor 10.0.0.1 {
|
||||||
|
router-id 10.0.0.2;
|
||||||
|
local-address 10.0.0.2;
|
||||||
|
local-as 65001;
|
||||||
|
peer-as 65534;
|
||||||
|
|
||||||
|
static {
|
||||||
|
route 192.168.100.101/32 {
|
||||||
|
next-hop 10.0.0.2;
|
||||||
|
}
|
||||||
|
|
||||||
|
route 192.168.100.102/32 {
|
||||||
|
as-path [65000 0 65001];
|
||||||
|
next-hop 10.0.0.2;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
6
tests/topotests/bgp_aspath_zero/r1/bgpd.conf
Normal file
6
tests/topotests/bgp_aspath_zero/r1/bgpd.conf
Normal file
@ -0,0 +1,6 @@
|
|||||||
|
!
|
||||||
|
router bgp 65534
|
||||||
|
no bgp ebgp-requires-policy
|
||||||
|
neighbor 10.0.0.2 remote-as external
|
||||||
|
neighbor 10.0.0.2 timers 3 10
|
||||||
|
!
|
6
tests/topotests/bgp_aspath_zero/r1/zebra.conf
Normal file
6
tests/topotests/bgp_aspath_zero/r1/zebra.conf
Normal file
@ -0,0 +1,6 @@
|
|||||||
|
!
|
||||||
|
interface r1-eth0
|
||||||
|
ip address 10.0.0.1/24
|
||||||
|
!
|
||||||
|
ip forwarding
|
||||||
|
!
|
121
tests/topotests/bgp_aspath_zero/test_bgp_aspath_zero.py
Normal file
121
tests/topotests/bgp_aspath_zero/test_bgp_aspath_zero.py
Normal file
@ -0,0 +1,121 @@
|
|||||||
|
#!/usr/bin/env python
|
||||||
|
|
||||||
|
#
|
||||||
|
# Copyright (c) 2021 by
|
||||||
|
# Donatas Abraitis <donatas.abraitis@gmail.com>
|
||||||
|
#
|
||||||
|
# Permission to use, copy, modify, and/or distribute this software
|
||||||
|
# for any purpose with or without fee is hereby granted, provided
|
||||||
|
# that the above copyright notice and this permission notice appear
|
||||||
|
# in all copies.
|
||||||
|
#
|
||||||
|
# THE SOFTWARE IS PROVIDED "AS IS" AND NETDEF DISCLAIMS ALL WARRANTIES
|
||||||
|
# WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
|
||||||
|
# MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL NETDEF BE LIABLE FOR
|
||||||
|
# ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY
|
||||||
|
# DAMAGES WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS,
|
||||||
|
# WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS
|
||||||
|
# ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR PERFORMANCE
|
||||||
|
# OF THIS SOFTWARE.
|
||||||
|
#
|
||||||
|
|
||||||
|
"""
|
||||||
|
Test if BGP UPDATE with AS-PATH attribute with value zero (0)
|
||||||
|
is threated as withdrawal.
|
||||||
|
"""
|
||||||
|
|
||||||
|
import os
|
||||||
|
import sys
|
||||||
|
import json
|
||||||
|
import time
|
||||||
|
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.topolog import logger
|
||||||
|
from mininet.topo import Topo
|
||||||
|
|
||||||
|
pytestmark = [pytest.mark.bgpd]
|
||||||
|
|
||||||
|
|
||||||
|
class BgpAggregatorAsnZero(Topo):
|
||||||
|
def build(self, *_args, **_opts):
|
||||||
|
tgen = get_topogen(self)
|
||||||
|
|
||||||
|
r1 = tgen.add_router("r1")
|
||||||
|
peer1 = tgen.add_exabgp_peer(
|
||||||
|
"peer1", ip="10.0.0.2", defaultRoute="via 10.0.0.1"
|
||||||
|
)
|
||||||
|
|
||||||
|
switch = tgen.add_switch("s1")
|
||||||
|
switch.add_link(r1)
|
||||||
|
switch.add_link(peer1)
|
||||||
|
|
||||||
|
|
||||||
|
def setup_module(mod):
|
||||||
|
tgen = Topogen(BgpAggregatorAsnZero, mod.__name__)
|
||||||
|
tgen.start_topology()
|
||||||
|
|
||||||
|
router = tgen.gears["r1"]
|
||||||
|
router.load_config(TopoRouter.RD_ZEBRA, os.path.join(CWD, "r1/zebra.conf"))
|
||||||
|
router.load_config(TopoRouter.RD_BGP, os.path.join(CWD, "r1/bgpd.conf"))
|
||||||
|
router.start()
|
||||||
|
|
||||||
|
peer = tgen.gears["peer1"]
|
||||||
|
peer.start(os.path.join(CWD, "peer1"), os.path.join(CWD, "exabgp.env"))
|
||||||
|
|
||||||
|
|
||||||
|
def teardown_module(mod):
|
||||||
|
tgen = get_topogen()
|
||||||
|
tgen.stop_topology()
|
||||||
|
|
||||||
|
|
||||||
|
def test_bgp_aggregator_zero():
|
||||||
|
tgen = get_topogen()
|
||||||
|
|
||||||
|
if tgen.routers_have_failure():
|
||||||
|
pytest.skip(tgen.errors)
|
||||||
|
|
||||||
|
def _bgp_converge():
|
||||||
|
output = json.loads(
|
||||||
|
tgen.gears["r1"].vtysh_cmd("show ip bgp neighbor 10.0.0.2 json")
|
||||||
|
)
|
||||||
|
expected = {
|
||||||
|
"10.0.0.2": {
|
||||||
|
"bgpState": "Established",
|
||||||
|
"addressFamilyInfo": {"ipv4Unicast": {"acceptedPrefixCounter": 1}},
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return topotest.json_cmp(output, expected)
|
||||||
|
|
||||||
|
test_func = functools.partial(_bgp_converge)
|
||||||
|
success, result = topotest.run_and_expect(test_func, None, count=60, wait=0.5)
|
||||||
|
assert result is None, "More than one prefix seen at r1, SHOULD be only one."
|
||||||
|
|
||||||
|
def _bgp_has_correct_routes_without_asn_0():
|
||||||
|
output = json.loads(tgen.gears["r1"].vtysh_cmd("show ip bgp json"))
|
||||||
|
expected = {"routes": {"192.168.100.101/32": [{"valid": True}]}}
|
||||||
|
return topotest.json_cmp(output, expected)
|
||||||
|
|
||||||
|
test_func = functools.partial(_bgp_has_correct_routes_without_asn_0)
|
||||||
|
success, result = topotest.run_and_expect(test_func, None, count=60, wait=0.5)
|
||||||
|
assert result is None, "Failed listing 192.168.100.101/32, SHOULD be accepted."
|
||||||
|
|
||||||
|
|
||||||
|
def test_memory_leak():
|
||||||
|
"Run the memory leak test and report results."
|
||||||
|
tgen = get_topogen()
|
||||||
|
if not tgen.is_memleak_enabled():
|
||||||
|
pytest.skip("Memory leak test/report is disabled")
|
||||||
|
|
||||||
|
tgen.report_memory_leaks()
|
||||||
|
|
||||||
|
|
||||||
|
if __name__ == "__main__":
|
||||||
|
args = ["-s"] + sys.argv[1:]
|
||||||
|
sys.exit(pytest.main(args))
|
Loading…
Reference in New Issue
Block a user