From 843713317d94f7a4d1fc213adad55b1518d235f2 Mon Sep 17 00:00:00 2001 From: Rafael Zalamena Date: Wed, 28 Feb 2018 13:08:14 -0300 Subject: [PATCH] bfd-topo1: add convergence tests Test if BFD peers have found each other and if the BGP neighors have connected. Signed-off-by: Rafael Zalamena --- tests/topotests/bfd-topo1/__init__.py | 0 tests/topotests/bfd-topo1/r1/bfdd.conf | 6 + tests/topotests/bfd-topo1/r1/bgp_summary.json | 11 ++ tests/topotests/bfd-topo1/r1/bgpd.conf | 4 + tests/topotests/bfd-topo1/r1/peers.json | 8 + tests/topotests/bfd-topo1/r1/zebra.conf | 3 + tests/topotests/bfd-topo1/r2/bfdd.conf | 12 ++ tests/topotests/bfd-topo1/r2/bgp_summary.json | 19 +++ tests/topotests/bfd-topo1/r2/bgpd.conf | 8 + tests/topotests/bfd-topo1/r2/peers.json | 17 ++ tests/topotests/bfd-topo1/r2/zebra.conf | 9 + tests/topotests/bfd-topo1/r3/bfdd.conf | 7 + tests/topotests/bfd-topo1/r3/bgp_summary.json | 11 ++ tests/topotests/bfd-topo1/r3/bgpd.conf | 4 + tests/topotests/bfd-topo1/r3/peers.json | 6 + tests/topotests/bfd-topo1/r3/zebra.conf | 3 + tests/topotests/bfd-topo1/r4/bfdd.conf | 7 + tests/topotests/bfd-topo1/r4/bgp_summary.json | 11 ++ tests/topotests/bfd-topo1/r4/bgpd.conf | 4 + tests/topotests/bfd-topo1/r4/peers.json | 6 + tests/topotests/bfd-topo1/r4/zebra.conf | 3 + tests/topotests/bfd-topo1/test_bfd_topo1.py | 157 ++++++++++++++++++ 22 files changed, 316 insertions(+) create mode 100644 tests/topotests/bfd-topo1/__init__.py create mode 100644 tests/topotests/bfd-topo1/r1/bfdd.conf create mode 100644 tests/topotests/bfd-topo1/r1/bgp_summary.json create mode 100644 tests/topotests/bfd-topo1/r1/bgpd.conf create mode 100644 tests/topotests/bfd-topo1/r1/peers.json create mode 100644 tests/topotests/bfd-topo1/r1/zebra.conf create mode 100644 tests/topotests/bfd-topo1/r2/bfdd.conf create mode 100644 tests/topotests/bfd-topo1/r2/bgp_summary.json create mode 100644 tests/topotests/bfd-topo1/r2/bgpd.conf create mode 100644 tests/topotests/bfd-topo1/r2/peers.json create mode 100644 tests/topotests/bfd-topo1/r2/zebra.conf create mode 100644 tests/topotests/bfd-topo1/r3/bfdd.conf create mode 100644 tests/topotests/bfd-topo1/r3/bgp_summary.json create mode 100644 tests/topotests/bfd-topo1/r3/bgpd.conf create mode 100644 tests/topotests/bfd-topo1/r3/peers.json create mode 100644 tests/topotests/bfd-topo1/r3/zebra.conf create mode 100644 tests/topotests/bfd-topo1/r4/bfdd.conf create mode 100644 tests/topotests/bfd-topo1/r4/bgp_summary.json create mode 100644 tests/topotests/bfd-topo1/r4/bgpd.conf create mode 100644 tests/topotests/bfd-topo1/r4/peers.json create mode 100644 tests/topotests/bfd-topo1/r4/zebra.conf create mode 100644 tests/topotests/bfd-topo1/test_bfd_topo1.py diff --git a/tests/topotests/bfd-topo1/__init__.py b/tests/topotests/bfd-topo1/__init__.py new file mode 100644 index 0000000000..e69de29bb2 diff --git a/tests/topotests/bfd-topo1/r1/bfdd.conf b/tests/topotests/bfd-topo1/r1/bfdd.conf new file mode 100644 index 0000000000..131b01f0d9 --- /dev/null +++ b/tests/topotests/bfd-topo1/r1/bfdd.conf @@ -0,0 +1,6 @@ +bfd + peer 192.168.0.2 + echo-mode + no shutdown + ! +! diff --git a/tests/topotests/bfd-topo1/r1/bgp_summary.json b/tests/topotests/bfd-topo1/r1/bgp_summary.json new file mode 100644 index 0000000000..fa07d60df9 --- /dev/null +++ b/tests/topotests/bfd-topo1/r1/bgp_summary.json @@ -0,0 +1,11 @@ +{ + "ipv4Unicast": { + "as": 101, + "peers": { + "192.168.0.2": { + "remoteAs": 102, + "state": "Established" + } + } + } +} diff --git a/tests/topotests/bfd-topo1/r1/bgpd.conf b/tests/topotests/bfd-topo1/r1/bgpd.conf new file mode 100644 index 0000000000..9a0166a876 --- /dev/null +++ b/tests/topotests/bfd-topo1/r1/bgpd.conf @@ -0,0 +1,4 @@ +router bgp 101 + neighbor 192.168.0.2 remote-as 102 + neighbor 192.168.0.2 bfd +! diff --git a/tests/topotests/bfd-topo1/r1/peers.json b/tests/topotests/bfd-topo1/r1/peers.json new file mode 100644 index 0000000000..f49768ff75 --- /dev/null +++ b/tests/topotests/bfd-topo1/r1/peers.json @@ -0,0 +1,8 @@ +[ + { + "remote-receive-interval": 1000, + "remote-transmit-interval": 500, + "peer": "192.168.0.2", + "status": "up" + } +] diff --git a/tests/topotests/bfd-topo1/r1/zebra.conf b/tests/topotests/bfd-topo1/r1/zebra.conf new file mode 100644 index 0000000000..a14cd7ac88 --- /dev/null +++ b/tests/topotests/bfd-topo1/r1/zebra.conf @@ -0,0 +1,3 @@ +interface r1-eth0 + ip address 192.168.0.1/24 +! diff --git a/tests/topotests/bfd-topo1/r2/bfdd.conf b/tests/topotests/bfd-topo1/r2/bfdd.conf new file mode 100644 index 0000000000..cb4357172b --- /dev/null +++ b/tests/topotests/bfd-topo1/r2/bfdd.conf @@ -0,0 +1,12 @@ +bfd + peer 192.168.0.1 + receive-interval 1000 + transmit-interval 500 + echo-mode + no shutdown + ! + peer 192.168.1.1 + echo-mode + no shutdown + ! +! diff --git a/tests/topotests/bfd-topo1/r2/bgp_summary.json b/tests/topotests/bfd-topo1/r2/bgp_summary.json new file mode 100644 index 0000000000..c0ef11ac5f --- /dev/null +++ b/tests/topotests/bfd-topo1/r2/bgp_summary.json @@ -0,0 +1,19 @@ +{ + "ipv4Unicast": { + "as": 102, + "peers": { + "192.168.0.1": { + "remoteAs": 101, + "state": "Established" + }, + "192.168.1.1": { + "remoteAs": 103, + "state": "Established" + }, + "192.168.2.1": { + "remoteAs": 104, + "state": "Established" + } + } + } +} diff --git a/tests/topotests/bfd-topo1/r2/bgpd.conf b/tests/topotests/bfd-topo1/r2/bgpd.conf new file mode 100644 index 0000000000..0ad2226891 --- /dev/null +++ b/tests/topotests/bfd-topo1/r2/bgpd.conf @@ -0,0 +1,8 @@ +router bgp 102 + neighbor 192.168.0.1 remote-as 101 + neighbor 192.168.0.1 bfd + neighbor 192.168.1.1 remote-as 103 + neighbor 192.168.1.1 bfd + neighbor 192.168.2.1 remote-as 104 + neighbor 192.168.2.1 bfd +! diff --git a/tests/topotests/bfd-topo1/r2/peers.json b/tests/topotests/bfd-topo1/r2/peers.json new file mode 100644 index 0000000000..5035d643c5 --- /dev/null +++ b/tests/topotests/bfd-topo1/r2/peers.json @@ -0,0 +1,17 @@ +[ + { + "peer": "192.168.0.1", + "status": "up" + }, + { + "remote-echo-interval": 100, + "peer": "192.168.1.1", + "status": "up" + }, + { + "remote-transmit-interval": 2000, + "remote-receive-interval": 2000, + "peer": "192.168.2.1", + "status": "up" + } +] diff --git a/tests/topotests/bfd-topo1/r2/zebra.conf b/tests/topotests/bfd-topo1/r2/zebra.conf new file mode 100644 index 0000000000..568abe751e --- /dev/null +++ b/tests/topotests/bfd-topo1/r2/zebra.conf @@ -0,0 +1,9 @@ +interface r2-eth0 + ip address 192.168.0.2/24 +! +interface r2-eth1 + ip address 192.168.1.2/24 +! +interface r2-eth2 + ip address 192.168.2.2/24 +! diff --git a/tests/topotests/bfd-topo1/r3/bfdd.conf b/tests/topotests/bfd-topo1/r3/bfdd.conf new file mode 100644 index 0000000000..a5d38c8162 --- /dev/null +++ b/tests/topotests/bfd-topo1/r3/bfdd.conf @@ -0,0 +1,7 @@ +bfd + peer 192.168.1.2 + echo-interval 100 + echo-mode + no shutdown + ! +! diff --git a/tests/topotests/bfd-topo1/r3/bgp_summary.json b/tests/topotests/bfd-topo1/r3/bgp_summary.json new file mode 100644 index 0000000000..d47833377b --- /dev/null +++ b/tests/topotests/bfd-topo1/r3/bgp_summary.json @@ -0,0 +1,11 @@ +{ + "ipv4Unicast": { + "as": 103, + "peers": { + "192.168.1.2": { + "remoteAs": 102, + "state": "Established" + } + } + } +} diff --git a/tests/topotests/bfd-topo1/r3/bgpd.conf b/tests/topotests/bfd-topo1/r3/bgpd.conf new file mode 100644 index 0000000000..e36a2f9930 --- /dev/null +++ b/tests/topotests/bfd-topo1/r3/bgpd.conf @@ -0,0 +1,4 @@ +router bgp 103 + neighbor 192.168.1.2 remote-as 102 + neighbor 192.168.1.2 bfd +! diff --git a/tests/topotests/bfd-topo1/r3/peers.json b/tests/topotests/bfd-topo1/r3/peers.json new file mode 100644 index 0000000000..ef38008643 --- /dev/null +++ b/tests/topotests/bfd-topo1/r3/peers.json @@ -0,0 +1,6 @@ +[ + { + "peer": "192.168.1.2", + "status": "up" + } +] diff --git a/tests/topotests/bfd-topo1/r3/zebra.conf b/tests/topotests/bfd-topo1/r3/zebra.conf new file mode 100644 index 0000000000..b4fd80fa10 --- /dev/null +++ b/tests/topotests/bfd-topo1/r3/zebra.conf @@ -0,0 +1,3 @@ +interface r3-eth0 + ip address 192.168.1.1/24 +! diff --git a/tests/topotests/bfd-topo1/r4/bfdd.conf b/tests/topotests/bfd-topo1/r4/bfdd.conf new file mode 100644 index 0000000000..029dfba355 --- /dev/null +++ b/tests/topotests/bfd-topo1/r4/bfdd.conf @@ -0,0 +1,7 @@ +bfd + peer 192.168.2.2 + transmit-interval 2000 + receive-interval 2000 + no shutdown + ! +! diff --git a/tests/topotests/bfd-topo1/r4/bgp_summary.json b/tests/topotests/bfd-topo1/r4/bgp_summary.json new file mode 100644 index 0000000000..7d81784b56 --- /dev/null +++ b/tests/topotests/bfd-topo1/r4/bgp_summary.json @@ -0,0 +1,11 @@ +{ + "ipv4Unicast": { + "as": 104, + "peers": { + "192.168.2.2": { + "remoteAs": 102, + "state": "Established" + } + } + } +} diff --git a/tests/topotests/bfd-topo1/r4/bgpd.conf b/tests/topotests/bfd-topo1/r4/bgpd.conf new file mode 100644 index 0000000000..5f04fbdd7f --- /dev/null +++ b/tests/topotests/bfd-topo1/r4/bgpd.conf @@ -0,0 +1,4 @@ +router bgp 104 + neighbor 192.168.2.2 remote-as 102 + neighbor 192.168.2.2 bfd +! diff --git a/tests/topotests/bfd-topo1/r4/peers.json b/tests/topotests/bfd-topo1/r4/peers.json new file mode 100644 index 0000000000..37140089e1 --- /dev/null +++ b/tests/topotests/bfd-topo1/r4/peers.json @@ -0,0 +1,6 @@ +[ + { + "peer": "192.168.2.2", + "status": "up" + } +] diff --git a/tests/topotests/bfd-topo1/r4/zebra.conf b/tests/topotests/bfd-topo1/r4/zebra.conf new file mode 100644 index 0000000000..afdd44b1ab --- /dev/null +++ b/tests/topotests/bfd-topo1/r4/zebra.conf @@ -0,0 +1,3 @@ +interface r4-eth0 + ip address 192.168.2.1/24 +! diff --git a/tests/topotests/bfd-topo1/test_bfd_topo1.py b/tests/topotests/bfd-topo1/test_bfd_topo1.py new file mode 100644 index 0000000000..6030aff3e3 --- /dev/null +++ b/tests/topotests/bfd-topo1/test_bfd_topo1.py @@ -0,0 +1,157 @@ +#!/usr/bin/env python + +# +# test_bfd_topo1.py +# Part of NetDEF Topology Tests +# +# Copyright (c) 2018 by +# Network Device Education Foundation, Inc. ("NetDEF") +# +# 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_bfd_topo1.py: Test the FRR/Quagga BFD daemon. +""" + +import os +import sys +import json +from functools import partial +import pytest + +# Save the Current Working Directory to find configuration files. +CWD = os.path.dirname(os.path.realpath(__file__)) +sys.path.append(os.path.join(CWD, '../')) + +# pylint: disable=C0413 +# Import topogen and topotest helpers +from lib import topotest +from lib.topogen import Topogen, TopoRouter, get_topogen +from lib.topolog import logger + +# Required to instantiate the topology builder class. +from mininet.topo import Topo + +class BFDTopo(Topo): + "Test topology builder" + def build(self, *_args, **_opts): + "Build function" + tgen = get_topogen(self) + + # Create 4 routers + for routern in range(1, 5): + 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['r2']) + switch.add_link(tgen.gears['r3']) + + switch = tgen.add_switch('s3') + switch.add_link(tgen.gears['r2']) + switch.add_link(tgen.gears['r4']) + + +def setup_module(mod): + "Sets up the pytest environment" + tgen = Topogen(BFDTopo, mod.__name__) + tgen.start_topology() + + router_list = tgen.routers() + for rname, router in router_list.iteritems(): + router.load_config( + TopoRouter.RD_ZEBRA, + os.path.join(CWD, '{}/zebra.conf'.format(rname)) + ) + router.load_config( + TopoRouter.RD_BFD, + os.path.join(CWD, '{}/bfdd.conf'.format(rname)) + ) + router.load_config( + TopoRouter.RD_BGP, + os.path.join(CWD, '{}/bgpd.conf'.format(rname)) + ) + + # Initialize all routers. + tgen.start_router() + + # Verify that we are using the proper version and that the BFD + # daemon exists. + for router in router_list.values(): + # Check for Version + if router.has_version('<', '5.1'): + tgen.set_error('Unsupported FRR version') + break + + +def teardown_module(_mod): + "Teardown the pytest environment" + tgen = get_topogen() + tgen.stop_topology() + + +def test_bfd_connection(): + "Assert that the BFD peers can find themselves." + tgen = get_topogen() + if tgen.routers_have_failure(): + pytest.skip(tgen.errors) + + logger.info('waiting for bfd peers to go up') + + for router in tgen.routers().values(): + json_file = '{}/{}/peers.json'.format(CWD, router.name) + expected = json.loads(open(json_file).read()) + + test_func = partial(topotest.router_json_cmp, + router, 'show bfd peers json', expected) + _, result = topotest.run_and_expect(test_func, None, count=8, wait=0.5) + assertmsg = '"{}" JSON output mismatches'.format(router.name) + assert result is None, assertmsg + + +def test_bgp_convergence(): + "Assert that BGP is converging." + tgen = get_topogen() + if tgen.routers_have_failure(): + pytest.skip(tgen.errors) + + logger.info('waiting for bgp peers to go up') + + for router in tgen.routers().values(): + ref_file = '{}/{}/bgp_summary.json'.format(CWD, router.name) + expected = json.loads(open(ref_file).read()) + test_func = partial(topotest.router_json_cmp, + router, 'show ip bgp summary json', expected) + _, res = topotest.run_and_expect(test_func, None, count=20, wait=0.5) + assertmsg = '{}: bgp did not converge'.format(router.name) + assert res is None, assertmsg + + +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)) -- 2.39.5