]> git.puffer.fish Git - mirror/frr.git/commitdiff
ospfd: add topotest for suppress-fa feature
authorckishimo <carles.kishimoto@gmail.com>
Thu, 5 Nov 2020 13:32:03 +0000 (05:32 -0800)
committerCarles Kishimoto Bisbe <carles.kishimoto@cern.ch>
Wed, 24 Mar 2021 15:00:35 +0000 (16:00 +0100)
Signed-off-by: ckishimo <carles.kishimoto@gmail.com>
tests/topotests/ospf_suppress_fa/__init__.py [new file with mode: 0755]
tests/topotests/ospf_suppress_fa/r1/ospfd.conf [new file with mode: 0644]
tests/topotests/ospf_suppress_fa/r1/zebra.conf [new file with mode: 0644]
tests/topotests/ospf_suppress_fa/r2/ospfd.conf [new file with mode: 0644]
tests/topotests/ospf_suppress_fa/r2/zebra.conf [new file with mode: 0644]
tests/topotests/ospf_suppress_fa/r3/ospfd.conf [new file with mode: 0644]
tests/topotests/ospf_suppress_fa/r3/zebra.conf [new file with mode: 0644]
tests/topotests/ospf_suppress_fa/test_ospf_suppress_fa.dot [new file with mode: 0644]
tests/topotests/ospf_suppress_fa/test_ospf_suppress_fa.dot.jpg [new file with mode: 0644]
tests/topotests/ospf_suppress_fa/test_ospf_suppress_fa.py [new file with mode: 0644]

diff --git a/tests/topotests/ospf_suppress_fa/__init__.py b/tests/topotests/ospf_suppress_fa/__init__.py
new file mode 100755 (executable)
index 0000000..e69de29
diff --git a/tests/topotests/ospf_suppress_fa/r1/ospfd.conf b/tests/topotests/ospf_suppress_fa/r1/ospfd.conf
new file mode 100644 (file)
index 0000000..c02be35
--- /dev/null
@@ -0,0 +1,9 @@
+!
+interface r1-eth0
+ ip ospf network point-to-point
+ ip ospf hello-interval 2
+ ip ospf dead-interval 10
+!
+router ospf
+ network 10.0.12.0/24 area 0
+!
diff --git a/tests/topotests/ospf_suppress_fa/r1/zebra.conf b/tests/topotests/ospf_suppress_fa/r1/zebra.conf
new file mode 100644 (file)
index 0000000..c1e31fb
--- /dev/null
@@ -0,0 +1,4 @@
+!
+interface r1-eth0
+ ip address 10.0.12.1/24
+!
diff --git a/tests/topotests/ospf_suppress_fa/r2/ospfd.conf b/tests/topotests/ospf_suppress_fa/r2/ospfd.conf
new file mode 100644 (file)
index 0000000..ebc7d25
--- /dev/null
@@ -0,0 +1,16 @@
+!
+interface r2-eth0
+ ip ospf network point-to-point
+ ip ospf hello-interval 2
+ ip ospf dead-interval 10
+!
+interface r2-eth1
+ ip ospf network point-to-point
+ ip ospf hello-interval 2
+ ip ospf dead-interval 10
+!
+router ospf
+ network 10.0.12.0/24 area 0
+ network 10.0.23.0/24 area 1
+ area 1 nssa 
+!
diff --git a/tests/topotests/ospf_suppress_fa/r2/zebra.conf b/tests/topotests/ospf_suppress_fa/r2/zebra.conf
new file mode 100644 (file)
index 0000000..9f1a263
--- /dev/null
@@ -0,0 +1,7 @@
+!
+interface r2-eth0
+ ip address 10.0.12.2/24
+!
+interface r2-eth1
+ ip address 10.0.23.2/24
+!
diff --git a/tests/topotests/ospf_suppress_fa/r3/ospfd.conf b/tests/topotests/ospf_suppress_fa/r3/ospfd.conf
new file mode 100644 (file)
index 0000000..08be11a
--- /dev/null
@@ -0,0 +1,11 @@
+!
+interface r3-eth0
+ ip ospf network point-to-point
+ ip ospf hello-interval 2
+ ip ospf dead-interval 10
+!
+router ospf
+ redistribute static
+ network 10.0.23.0/24 area 1
+ area 1 nssa
+!
diff --git a/tests/topotests/ospf_suppress_fa/r3/zebra.conf b/tests/topotests/ospf_suppress_fa/r3/zebra.conf
new file mode 100644 (file)
index 0000000..f76cbf7
--- /dev/null
@@ -0,0 +1,8 @@
+!
+ip route 3.3.1.1/32 Null0
+ip route 3.3.2.2/32 Null0
+ip route 3.3.3.3/32 Null0
+!
+interface r3-eth0
+ ip address 10.0.23.3/24
+!
diff --git a/tests/topotests/ospf_suppress_fa/test_ospf_suppress_fa.dot b/tests/topotests/ospf_suppress_fa/test_ospf_suppress_fa.dot
new file mode 100644 (file)
index 0000000..1036658
--- /dev/null
@@ -0,0 +1,66 @@
+## Color coding:
+#########################
+##  Main FRR: #f08080  red
+##  Switches: #d0e0d0  gray
+##  RIP:      #19e3d9  Cyan
+##  RIPng:    #fcb314  dark yellow
+##  OSPFv2:   #32b835  Green
+##  OSPFv3:   #19e3d9  Cyan
+##  ISIS IPv4 #fcb314  dark yellow
+##  ISIS IPv6 #9a81ec  purple
+##  BGP IPv4  #eee3d3  beige
+##  BGP IPv6  #fdff00  yellow
+##### Colors (see http://www.color-hex.com/)
+
+graph ospf_topo1 {
+       label="ospf suppress-fa";
+
+       # Routers
+       r1 [
+               label="r1\nrtr-id 10.0.12.1",
+               shape=doubleoctagon,
+               fillcolor="#f08080",
+               style=filled,
+       ];
+       r2 [
+               label="r2 (ABR)\nrtr-id 10.0.23.2",
+               shape=doubleoctagon,
+               fillcolor="#f08080",
+               style=filled,
+       ];
+       r3 [
+               label="r3 (ASBR)\nrtr-id 10.0.23.3",
+               shape=doubleoctagon,
+               fillcolor="#f08080",
+               style=filled,
+       ];
+
+       # Switches
+       s1 [
+               label="s1\n10.0.12.0/24",
+               shape=oval,
+               fillcolor="#d0e0d0",
+               style=filled,
+       ];
+       s2 [
+               label="s2\n10.0.23.0/24",
+               shape=oval,
+               fillcolor="#d0e0d0",
+               style=filled,
+       ];
+
+       # Connections
+    subgraph cluster0 {
+        label="area 0"
+           r1 -- s1 [label="eth1\n.1"];
+           r2 -- s1 [label="eth1\n.2"];
+    }
+
+    subgraph cluster1 {
+        label="area 1\nNSSA"
+           r2 -- s2 [label="eth2\n.2"];
+           r3 -- s2 [label="eth1\n.3"];
+    }
+
+    { rank=same; r1; r2; r3; }
+}
diff --git a/tests/topotests/ospf_suppress_fa/test_ospf_suppress_fa.dot.jpg b/tests/topotests/ospf_suppress_fa/test_ospf_suppress_fa.dot.jpg
new file mode 100644 (file)
index 0000000..2907d79
Binary files /dev/null and b/tests/topotests/ospf_suppress_fa/test_ospf_suppress_fa.dot.jpg differ
diff --git a/tests/topotests/ospf_suppress_fa/test_ospf_suppress_fa.py b/tests/topotests/ospf_suppress_fa/test_ospf_suppress_fa.py
new file mode 100644 (file)
index 0000000..74d609c
--- /dev/null
@@ -0,0 +1,181 @@
+#!/usr/bin/env python
+
+#
+# test_ospf_suppres_fa.py
+# Carles Kishimoto
+#
+# 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_ospf_suppres_fa.py: Test OSPF suppress-fa feature
+- Topology: r1 --- R2 (ABR) --- R3 (redistribute static)
+
+test_ospf_set_suppress_fa()
+    1) R1: Get a dict[LSA_ID] = fwd_addr for all type 5 LSA
+    2) R2: Configure: area 1 nssa suppress-fa
+    3) R1: Get a dict[LSA_ID] and compare fwd_address with 0.0.0.0
+
+test_ospf_unset_suppress_fa()
+    4) R2: Configure: no area 1 nssa suppress-fa
+    5) R1: Get a dict[LSA_ID] = fwd_addr and compare it with the dict obtained in 1)
+"""
+
+import os
+import sys
+import re
+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
+
+# Required to instantiate the topology builder class.
+from mininet.topo import Topo
+
+
+class NetworkTopo(Topo):
+    "OSPF topology builder"
+
+    def build(self, *_args, **_opts):
+        "Build function"
+
+        tgen = get_topogen(self)
+
+        # Create routers
+        for router in range(1, 4):
+            tgen.add_router("r{}".format(router))
+
+        # R1-R2 backbone area
+        switch = tgen.add_switch("s1")
+        switch.add_link(tgen.gears["r1"])
+        switch.add_link(tgen.gears["r2"])
+
+        # R2-R3 NSSA area
+        switch = tgen.add_switch("s2")
+        switch.add_link(tgen.gears["r2"])
+        switch.add_link(tgen.gears["r3"])
+
+
+def setup_module(mod):
+    "Sets up the pytest environment"
+
+    tgen = Topogen(NetworkTopo, mod.__name__)
+    tgen.start_topology()
+
+    # This is a sample of configuration loading.
+    router_list = tgen.routers()
+
+    # For all registred routers, load the zebra and ospf configuration file
+    for rname, router in router_list.items():
+        router.load_config(
+            TopoRouter.RD_ZEBRA, os.path.join(CWD, "{}/zebra.conf".format(rname))
+        )
+        router.load_config(
+            TopoRouter.RD_OSPF, os.path.join(CWD, "{}/ospfd.conf".format(rname))
+        )
+
+    tgen.start_router()
+
+def teardown_module(_mod):
+    "Teardown the pytest environment"
+
+    tgen = get_topogen()
+    tgen.stop_topology()
+
+def test_converge_protocols():
+    "Wait for protocol convergence"
+
+    tgen = get_topogen()
+    # Don't run this test if we have any failure.
+    if tgen.routers_have_failure():
+        pytest.skip(tgen.errors)
+
+    topotest.sleep(10, "Waiting for OSPF convergence")
+
+def ospf_configure_suppress_fa(router_name, area):
+    "Configure OSPF suppress-fa in router_name"
+
+    tgen = get_topogen()
+    router = tgen.gears[router_name]
+    router.vtysh_cmd("conf t\nrouter ospf\narea {} nssa suppress-fa\nexit\n".format(area))
+
+def ospf_unconfigure_suppress_fa(router_name, area):
+    "Remove OSPF suppress-fa in router_name"
+
+    tgen = get_topogen()
+    router = tgen.gears[router_name]
+    router.vtysh_cmd("conf t\nrouter ospf\nno area {} nssa suppress-fa\nexit\n".format(area))
+
+def ospf_get_lsa_type5(router_name):
+    "Return a dict with link state id as key and forwarding addresses as value"
+
+    result = dict()
+    tgen = get_topogen()
+    router = tgen.gears[router_name]
+    cmd = "show ip ospf database external\n"
+    output = topotest.normalize_text(router.vtysh_cmd(cmd))
+    for line in output.splitlines():
+        re0 = re.match(r"\s+Link State ID: (\S+) \(External Network Number\)", line)
+        if re0:
+            lsa = re0.group(1)
+        re1 = re.match(r"\s+Forward Address: (\S+)", line)
+        if re1:
+            result[lsa] = re1.group(1)
+    return result
+
+@pytest.fixture(scope='module', name='original')
+def test_ospf_set_suppress_fa():
+    "Test OSPF area [x] nssa suppress-fa"
+
+    # Get current forwarding address for each LSA type-5 in r1
+    initial = ospf_get_lsa_type5("r1")
+
+    # Configure suppres-fa in r2 area 1
+    ospf_configure_suppress_fa("r2", "1")
+    topotest.sleep(10, "Waiting for OSPF convergence")
+
+    # Check forwarding address on r1 for all statics is 0.0.0.0
+    assertmsg = "Forwarding address is not 0.0.0.0 after enabling OSPF suppress-fa"
+    suppress = ospf_get_lsa_type5("r1")
+    for prefix in suppress:
+        assert suppress[prefix] == "0.0.0.0", assertmsg
+
+    # Return the original forwarding addresses so we can compare them
+    # in the test_ospf_unset_supress_fa
+    return initial
+
+def test_ospf_unset_supress_fa(original):
+    "Test OSPF no area [x] nssa suppress-fa"
+
+    # Remove suppress-fa in r2 area 1
+    ospf_unconfigure_suppress_fa("r2", "1")
+    topotest.sleep(10, "Waiting for OSPF convergence")
+
+    # Check forwarding address is the original value on r1 for all statics
+    assertmsg = "Forwarding address is not correct after removing OSPF suppress-fa"
+    restore = ospf_get_lsa_type5("r1")
+    for prefix in restore:
+        assert restore[prefix] == original[prefix], assertmsg
+
+
+if __name__ == "__main__":
+    args = ["-s"] + sys.argv[1:]
+    sys.exit(pytest.main(args))