]> git.puffer.fish Git - mirror/frr.git/commitdiff
tests: Our tests are for FRR 7154/head
authorDonald Sharp <sharpd@nvidia.com>
Sat, 19 Sep 2020 01:07:20 +0000 (21:07 -0400)
committerDonald Sharp <sharpd@nvidia.com>
Tue, 22 Sep 2020 14:11:58 +0000 (10:11 -0400)
These tests at this point only work for FRR.  Let's cut to the chase
and admit it.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>`
22 files changed:
tests/topotests/all-protocol-startup/test_all_protocol_startup.py
tests/topotests/bfd-bgp-cbit-topo3/test_bfd_bgp_cbit_topo3.py
tests/topotests/bfd-topo1/test_bfd_topo1.py
tests/topotests/bfd-topo2/test_bfd_topo2.py
tests/topotests/bfd-vrf-topo1/test_bfd_vrf_topo1.py
tests/topotests/bgp_evpn_rt5/test_bgp_evpn.py
tests/topotests/bgp_ipv6_rtadv/test_bgp_ipv6_rtadv.py
tests/topotests/bgp_l3vpn_to_bgp_direct/customize.py
tests/topotests/bgp_l3vpn_to_bgp_vrf/customize.py
tests/topotests/bgp_multiview_topo1/test_bgp_multiview_topo1.py
tests/topotests/bgp_rfapi_basic_sanity/customize.py
tests/topotests/bgp_vrf_lite_ipv6_rtadv/test_bgp_vrf_lite_ipv6_rtadv.py
tests/topotests/ldp-oc-acl-topo1/test_ldp_oc_acl_topo1.py
tests/topotests/ldp-oc-topo1/test_ldp_oc_topo1.py
tests/topotests/ldp-topo1/test_ldp_topo1.py
tests/topotests/lib/topogen.py
tests/topotests/lib/topotest.py
tests/topotests/ospf-topo1-vrf/test_ospf_topo1_vrf.py
tests/topotests/ospf-topo1/test_ospf_topo1.py
tests/topotests/pytest.ini
tests/topotests/rip-topo1/test_rip_topo1.py
tests/topotests/ripng-topo1/test_ripng_topo1.py

index 8525838d96a78cca55623573d25db0b4c870031a..2a00398e285fc9adf49b46e35bd979059f326ed8 100755 (executable)
@@ -126,7 +126,7 @@ def setup_module(module):
         net['r%s' % i].loadConf('pbrd', '%s/r%s/pbrd.conf' % (thisDir, i))
         net['r%s' % i].startRouter()
 
-    # For debugging after starting Quagga/FRR daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -148,7 +148,7 @@ def test_router_running():
     if (fatal_error != ""):
         pytest.skip(fatal_error)
 
-    print("\n\n** Check if FRR/Quagga is running on each Router node")
+    print("\n\n** Check if FRR is running on each Router node")
     print("******************************************\n")
     sleep(5)
 
@@ -157,7 +157,7 @@ def test_router_running():
         fatal_error = net['r%s' % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -213,7 +213,7 @@ def test_error_messages_vtysh():
         fatal_error = net['r%s' % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -295,7 +295,7 @@ def test_error_messages_daemons():
 
     assert error_logs == "", "Daemons report errors to StdErr"
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -363,7 +363,7 @@ def test_converge_protocols():
 
         assert failures == 0, "IPv6 Routing table failed for r%s\n%s" % (i, diff)
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     ## CLI(net)
 
 def route_get_nhg_id(route_str):
@@ -527,7 +527,7 @@ def test_rip_status():
         fatal_error = net['r%s' % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -582,7 +582,7 @@ def test_ripng_status():
         fatal_error = net['r%s' % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -646,7 +646,7 @@ def test_ospfv2_interfaces():
         fatal_error = net['r%s' % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -701,7 +701,7 @@ def test_isis_interfaces():
         fatal_error = net['r%s' % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -782,7 +782,7 @@ def test_bgp_summary():
         fatal_error = net['r%s' % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -864,7 +864,7 @@ def test_bgp_ipv6_summary():
         fatal_error = net['r%s' % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -927,7 +927,7 @@ def test_bgp_ipv4():
         fatal_error = net['r%s' % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -989,7 +989,7 @@ def test_bgp_ipv6():
         fatal_error = net['r%s' % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 def test_route_map():
@@ -1155,7 +1155,7 @@ def test_mpls_interfaces():
         fatal_error = net['r%s' % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
index 7d1521c8b2b4f0806ce018fca2a5a8ed3c1895f3..595132214bfdf8dac7a5e9d667269a5e720f58a5 100644 (file)
@@ -21,7 +21,7 @@
 #
 
 """
-test_bfd_bgp_cbit_topo3.py: Test the FRR/Quagga BFD daemon with multihop and BGP
+test_bfd_bgp_cbit_topo3.py: Test the FRR BFD daemon with multihop and BGP
 unnumbered.
 """
 
index 5306fdf3532eb27e9fc56a398e0eec0474724f94..6e589d55eb0bea193a62b450c6aa4c6156a2daba 100644 (file)
@@ -23,7 +23,7 @@
 #
 
 """
-test_bfd_topo1.py: Test the FRR/Quagga BFD daemon.
+test_bfd_topo1.py: Test the FRR BFD daemon.
 """
 
 import os
index 2c5ce3e4c32c68d09393041900fbf6ee659cc2f6..feb4576bd305a888c00baa04fd6ab31520bf322f 100644 (file)
@@ -23,7 +23,7 @@
 #
 
 """
-test_bfd_topo2.py: Test the FRR/Quagga BFD daemon with multihop and BGP
+test_bfd_topo2.py: Test the FRR BFD daemon with multihop and BGP
 unnumbered.
 """
 
index 95595ecba461172a933ac78a4e7a6765396ef6b3..5fed135f8d3bfb7da38fd36ddbc7749d1a562ffa 100644 (file)
@@ -24,7 +24,7 @@
 #
 
 """
-test_bfd_vrf_topo1.py: Test the FRR/Quagga BFD daemon.
+test_bfd_vrf_topo1.py: Test the FRR BFD daemon.
 """
 
 import os
index 69ef7e9faeca1d5a820930fe409065491b69a866..3ad989c6019aff23e2a425516b9eb53752c24333 100755 (executable)
@@ -22,7 +22,7 @@
 #
 
 """
- test_bgp_evpn.py: Test the FRR/Quagga BGP daemon with BGP IPv6 interface
+ test_bgp_evpn.py: Test the FRR BGP daemon with BGP IPv6 interface
  with route advertisements on a separate netns.
 """
 
index 0acf8d2dbc7996679dc64480bae3fc04aedadcdd..783e746418ef21768f99db3bb5d3dbf3f331cb92 100644 (file)
@@ -22,7 +22,7 @@
 #
 
 """
- test_bgp_ipv6_rtadv.py: Test the FRR/Quagga BGP daemon with BGP IPv6 interface
+ test_bgp_ipv6_rtadv.py: Test the FRR BGP daemon with BGP IPv6 interface
  with route advertisements on a separate netns.
 """
 
index 05db9ab14b611bdaacb616520510ea562ead7086..752e37f5f880f522fb713647a6579fc0365c039d 100644 (file)
@@ -22,7 +22,7 @@
 #
 
 """
-customize.py: Simple FRR/Quagga MPLS L3VPN test topology
+customize.py: Simple FRR MPLS L3VPN test topology
 
                   |
              +----+----+
index fb919f02d0dee13c8ff86a3b13f0d23d12b49c1f..1c3c51f68e0200233e224bcbd3c00e87ac182b44 100644 (file)
@@ -22,7 +22,7 @@
 #
 
 """
-customize.py: Simple FRR/Quagga MPLS L3VPN test topology
+customize.py: Simple FRR MPLS L3VPN test topology
 
                   |
              +----+----+
index a2020ffa557d65357225ab81a68aacbc8ffb8794..6344f7bb406453297865e3d7ff91bab007228fd7 100755 (executable)
@@ -23,7 +23,7 @@
 #
 
 """
-test_bgp_multiview_topo1.py: Simple Quagga/FRR Route-Server Test
+test_bgp_multiview_topo1.py: Simple FRR Route-Server Test
 
 +----------+ +----------+ +----------+ +----------+ +----------+
 |  peer1   | |  peer2   | |  peer3   | |  peer4   | |  peer5   |
@@ -165,7 +165,7 @@ def setup_module(module):
         print("peer%s" % i),
     print("")
 
-    # For debugging after starting Quagga/FRR daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -193,7 +193,7 @@ def test_router_running():
     if fatal_error != "":
         pytest.skip(fatal_error)
 
-    print("\n\n** Check if FRR/Quagga is running on each Router node")
+    print("\n\n** Check if FRR is running on each Router node")
     print("******************************************\n")
 
     # Starting Routers
@@ -201,7 +201,7 @@ def test_router_running():
         fatal_error = net["r%s" % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -262,7 +262,7 @@ def test_bgp_converge():
         fatal_error = net["r%s" % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting Quagga/FRR daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -300,7 +300,7 @@ def test_bgp_routingTable():
                         .rstrip()
                     )
 
-                    # Fix inconsitent spaces between 0.99.24 and newer versions of Quagga...
+                    # Fix inconsitent spaces between 0.99.24 and newer versions
                     actual = re.sub("0             0", "0              0", actual)
                     actual = re.sub(
                         r"([0-9])         32768", r"\1          32768", actual
@@ -347,7 +347,7 @@ def test_bgp_routingTable():
         fatal_error = net["r%s" % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
index ea548a73378a4bc004c09ad4fab43e3a1c313ae3..2c85cf6e9da6e5595f52e266e6f334e5851acaae 100644 (file)
@@ -23,7 +23,7 @@
 #
 
 """
-customize.py: Simple FRR/Quagga MPLS L3VPN test topology
+customize.py: Simple FRR MPLS L3VPN test topology
 
              +---------+
              |    r1   |
index 50b9b092d6d29014e5427369d8e9dded34980a42..97d98415db297b0f37d11b9694d91c515f5bc817 100644 (file)
@@ -22,7 +22,7 @@
 #
 
 """
- test_bgp_ipv6_rtadv.py: Test the FRR/Quagga BGP daemon with BGP IPv6 interface
+ test_bgp_ipv6_rtadv.py: Test the FRR BGP daemon with BGP IPv6 interface
  with route advertisements on a separate netns.
 """
 
index dadb2065e6a9cb36fc139a809b15ab9265e508a5..3b3ed3383c75cb4b7a059d53b24830901b494d52 100644 (file)
@@ -22,7 +22,7 @@
 #
 
 """
-test_ldp_oc_acl_topo1.py: Simple FRR/Quagga LDP Test
+test_ldp_oc_acl_topo1.py: Simple FRR LDP Test
 
              +---------+
              |    r1   |
index ea449e4aba79f5349a08a58da53bd45b50cc753d..ead991b18390b562bf762bed86a6c41549d028ee 100644 (file)
@@ -22,7 +22,7 @@
 #
 
 """
-test_ldp_oc_topo1.py: Simple FRR/Quagga LDP Test
+test_ldp_oc_topo1.py: Simple FRR LDP Test
 
              +---------+
              |    r1   |
index cef4d6587ed2f0230f4bf22f81a17e05f4c7b5c7..31adeafbf67cc2b1e486769bee048592a632ca05 100755 (executable)
@@ -23,7 +23,7 @@
 #
 
 """
-test_ldp_topo1.py: Simple FRR/Quagga LDP Test
+test_ldp_topo1.py: Simple FRR LDP Test
 
              +---------+
              |    r1   |
@@ -186,7 +186,7 @@ def setup_module(module):
         if fatal_error != "":
             break
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -208,7 +208,7 @@ def test_router_running():
     if fatal_error != "":
         pytest.skip(fatal_error)
 
-    print("\n\n** Check if FRR/Quagga is running on each Router node")
+    print("\n\n** Check if FRR is running on each Router node")
     print("******************************************\n")
     sleep(5)
 
@@ -217,7 +217,7 @@ def test_router_running():
         fatal_error = net["r%s" % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -283,7 +283,7 @@ def test_mpls_interfaces():
         fatal_error = net["r%s" % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -420,7 +420,7 @@ def test_mpls_ldp_discovery():
         fatal_error = net["r%s" % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -490,7 +490,7 @@ def test_mpls_ldp_neighbor():
         fatal_error = net["r%s" % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -582,7 +582,7 @@ def test_mpls_ldp_binding():
         fatal_error = net["r%s" % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -612,7 +612,7 @@ def test_zebra_ipv4_routingTable():
                 .cmd('vtysh -c "show ip route" 2> /dev/null | grep "^O"')
                 .rstrip()
             )
-            # Drop timers on end of line (older Quagga Versions)
+            # Drop timers on end of line
             actual = re.sub(r", [0-2][0-9]:[0-5][0-9]:[0-5][0-9]", "", actual)
 
             # Mask out label - all LDP labels should be >= 10 (2-digit)
@@ -657,7 +657,7 @@ def test_zebra_ipv4_routingTable():
         fatal_error = net["r%s" % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -736,7 +736,7 @@ def test_mpls_table():
         fatal_error = net["r%s" % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -820,7 +820,7 @@ def test_linux_mpls_routes():
         fatal_error = net["r%s" % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
index b9f82877e22ca0185ce21f43344892b46fc8b367..171a894b358f5faf51f356b787f767702536248c 100644 (file)
@@ -96,7 +96,6 @@ def set_topogen(tgen):
 tgen_defaults = {
     "verbosity": "info",
     "frrdir": "/usr/lib/frr",
-    "quaggadir": "/usr/lib/quagga",
     "routertype": "frr",
     "memleak_path": None,
 }
@@ -173,7 +172,7 @@ class Topogen(object):
         options:
         * `name`: (optional) select the router name
         * `daemondir`: (optional) custom daemon binary directory
-        * `routertype`: (optional) `quagga` or `frr`
+        * `routertype`: (optional) `frr`
         Returns a TopoRouter.
         """
         if name is None:
@@ -182,7 +181,6 @@ class Topogen(object):
             raise KeyError("router already exists")
 
         params["frrdir"] = self.config.get(self.CONFIG_SECTION, "frrdir")
-        params["quaggadir"] = self.config.get(self.CONFIG_SECTION, "quaggadir")
         params["memleak_path"] = self.config.get(self.CONFIG_SECTION, "memleak_path")
         if not params.has_key("routertype"):
             params["routertype"] = self.config.get(self.CONFIG_SECTION, "routertype")
@@ -529,12 +527,10 @@ class TopoRouter(TopoGear):
     Router abstraction.
     """
 
-    # The default required directories by Quagga/FRR
+    # The default required directories by FRR
     PRIVATE_DIRS = [
         "/etc/frr",
-        "/etc/quagga",
         "/var/run/frr",
-        "/var/run/quagga",
         "/var/log",
     ]
 
@@ -581,7 +577,7 @@ class TopoRouter(TopoGear):
         * cls: router class that will be used to instantiate
         * name: router name
         * daemondir: daemon binary directory
-        * routertype: 'quagga' or 'frr'
+        * routertype: 'frr'
         """
         super(TopoRouter, self).__init__()
         self.tgen = tgen
@@ -626,7 +622,7 @@ class TopoRouter(TopoGear):
         except OSError:
             pass
 
-        # Allow unprivileged daemon user (frr/quagga) to create log files
+        # Allow unprivileged daemon user (frr) to create log files
         try:
             # Only allow group, if it exist.
             gid = grp.getgrnam(self.routertype)[2]
@@ -675,7 +671,7 @@ class TopoRouter(TopoGear):
         * Load modules
         * Clean up files
         * Configure interfaces
-        * Start daemons (e.g. FRR/Quagga)
+        * Start daemons (e.g. FRR)
         * Configure daemon logging files
         """
         self.logger.debug("starting")
@@ -724,7 +720,7 @@ class TopoRouter(TopoGear):
     def startDaemons(self, daemons):
         """
         Start Daemons: to start specific daemon(user defined daemon only)
-        * Start daemons (e.g. FRR/Quagga)
+        * Start daemons (e.g. FRR)
         * Configure daemon logging files
         """
         self.logger.debug('starting')
@@ -1041,12 +1037,10 @@ def diagnose_env_linux():
 
     # Assert that FRR utilities exist
     frrdir = config.get("topogen", "frrdir")
-    hasfrr = False
     if not os.path.isdir(frrdir):
         logger.error("could not find {} directory".format(frrdir))
         ret = False
     else:
-        hasfrr = True
         try:
             pwd.getpwnam("frr")[2]
         except KeyError:
@@ -1095,56 +1089,6 @@ def diagnose_env_linux():
 
                 os.system("{} -v 2>&1 >/tmp/topotests/frr_zebra.txt".format(path))
 
-    # Assert that Quagga utilities exist
-    quaggadir = config.get("topogen", "quaggadir")
-    if hasfrr:
-        # if we have frr, don't check for quagga
-        pass
-    elif not os.path.isdir(quaggadir):
-        logger.info(
-            "could not find {} directory (quagga tests will not run)".format(quaggadir)
-        )
-    else:
-        ret = True
-        try:
-            pwd.getpwnam("quagga")[2]
-        except KeyError:
-            logger.info('could not find "quagga" user')
-
-        try:
-            grp.getgrnam("quagga")[2]
-        except KeyError:
-            logger.info('could not find "quagga" group')
-
-        try:
-            if "quagga" not in grp.getgrnam("quaggavty").gr_mem:
-                logger.error(
-                    '"quagga" user and group exist, but user is not under "quaggavty"'
-                )
-        except KeyError:
-            logger.warning('could not find "quaggavty" group')
-
-        for fname in [
-            "zebra",
-            "ospfd",
-            "ospf6d",
-            "bgpd",
-            "ripd",
-            "ripngd",
-            "isisd",
-            "pimd",
-            "pbrd"
-        ]:
-            path = os.path.join(quaggadir, fname)
-            if not os.path.isfile(path):
-                logger.warning("could not find {} in {}".format(fname, quaggadir))
-                ret = False
-            else:
-                if fname != "zebra":
-                    continue
-
-                os.system("{} -v 2>&1 >/tmp/topotests/quagga_zebra.txt".format(path))
-
     # Test MPLS availability
     krel = platform.release()
     if topotest.version_cmp(krel, "4.5") < 0:
index e34d1cf0be3605388d3f93c229957c05ba38afcf..f8843b0f195b8bdd92d988471281e52120932d31 100644 (file)
@@ -947,13 +947,11 @@ def checkAddressSanitizerError(output, router, component):
 
 
 def addRouter(topo, name):
-    "Adding a FRRouter (or Quagga) to Topology"
+    "Adding a FRRouter to Topology"
 
     MyPrivateDirs = [
         "/etc/frr",
-        "/etc/quagga",
         "/var/run/frr",
-        "/var/run/quagga",
         "/var/log",
     ]
     if sys.platform.startswith("linux"):
@@ -985,7 +983,7 @@ def assert_sysctl(node, sysctl, value):
 
 
 class Router(Node):
-    "A Node with IPv4/IPv6 forwarding enabled and Quagga as Routing Engine"
+    "A Node with IPv4/IPv6 forwarding enabled"
 
     def __init__(self, name, **params):
         super(Router, self).__init__(name, **params)
@@ -997,7 +995,6 @@ class Router(Node):
             {
                 "verbosity": "info",
                 "frrdir": "/usr/lib/frr",
-                "quaggadir": "/usr/lib/quagga",
                 "routertype": "frr",
                 "memleak_path": None,
             }
@@ -1055,18 +1052,6 @@ class Router(Node):
         if not os.path.isfile(zebra_path):
             raise Exception("FRR zebra binary doesn't exist at {}".format(zebra_path))
 
-    def _config_quagga(self, **params):
-        "Configure Quagga binaries"
-        self.daemondir = params.get("quaggadir")
-        if self.daemondir is None:
-            self.daemondir = self.config_defaults.get("topogen", "quaggadir")
-
-        zebra_path = os.path.join(self.daemondir, "zebra")
-        if not os.path.isfile(zebra_path):
-            raise Exception(
-                "Quagga zebra binary doesn't exist at {}".format(zebra_path)
-            )
-
     # pylint: disable=W0221
     # Some params are only meaningful for the parent class.
     def config(self, **params):
@@ -1078,10 +1063,7 @@ class Router(Node):
             self.routertype = params.get(
                 "routertype", self.config_defaults.get("topogen", "routertype")
             )
-            if self.routertype == "quagga":
-                self._config_quagga(**params)
-            else:
-                self._config_frr(**params)
+            self._config_frr(**params)
         else:
             # Test the provided path
             zpath = os.path.join(self.daemondir, "zebra")
@@ -1406,7 +1388,7 @@ class Router(Node):
     def killRouterDaemons(
         self, daemons, wait=True, assertOnError=True, minErrorVersion="5.1"
     ):
-        # Kill Running Quagga or FRR specific
+        # Kill Running FRR
         # Daemons(user specified daemon only) using SIGKILL
         rundaemons = self.cmd("ls -1 /var/run/%s/*.pid" % self.routertype)
         errors = ""
@@ -1668,7 +1650,7 @@ class Router(Node):
         return True
 
     def get_routertype(self):
-        "Return the type of Router (frr or quagga)"
+        "Return the type of Router (frr)"
 
         return self.routertype
 
index 4ec09b10d3166af3a867492b9964a9dbe270eab0..6d44d02e5e4f20ebff985cd682872ca6a63b9734 100644 (file)
@@ -23,7 +23,7 @@
 #
 
 """
-test_ospf_topo1.py: Test the FRR/Quagga OSPF routing daemon.
+test_ospf_topo1.py: Test the FRR OSPF routing daemon.
 """
 
 import os
index 3af60fd48f0cc53507cabeabc96e8dc4282f29d4..95193afb2aae49aaad4e17f6c9ab28d5dc54fff7 100644 (file)
@@ -23,7 +23,7 @@
 #
 
 """
-test_ospf_topo1.py: Test the FRR/Quagga OSPF routing daemon.
+test_ospf_topo1.py: Test the FRR OSPF routing daemon.
 """
 
 import os
index 77b8c2b47810fc4ae5161b4651530af592f3e3e8..2e9c4901bc4e7ae1e81de95750d23912576cd5e8 100644 (file)
@@ -19,10 +19,9 @@ norecursedirs = .git example-test example-topojson-test lib docker evpn_type5_te
 
 # Default daemons binaries path.
 #frrdir = /usr/lib/frr
-#quaggadir = /usr/lib/quagga
 
 # Default router type to use. Possible values are:
-# 'frr' and 'quagga'.
+# 'frr'
 #routertype = frr
 
 # Memory leak test reports path
index 3098812a24c4269d9abb5d9e408d246db537e4b3..7ff18ba524b8bdb7cf76c215f2e01d32d0a1bb68 100755 (executable)
@@ -127,7 +127,7 @@ def setup_module(module):
         net["r%s" % i].loadConf("ripd", "%s/r%s/ripd.conf" % (thisDir, i))
         net["r%s" % i].startRouter()
 
-    # For debugging after starting Quagga/FRR daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -149,7 +149,7 @@ def test_router_running():
     if fatal_error != "":
         pytest.skip(fatal_error)
 
-    print("\n\n** Check if FRR/Quagga is running on each Router node")
+    print("\n\n** Check if FRR is running on each Router node")
     print("******************************************\n")
 
     # Make sure that all daemons are running
@@ -157,7 +157,7 @@ def test_router_running():
         fatal_error = net["r%s" % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -182,7 +182,7 @@ def test_converge_protocols():
         fatal_error = net["r%s" % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -243,7 +243,7 @@ def test_rip_status():
         fatal_error = net["r%s" % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -298,7 +298,7 @@ def test_rip_routes():
         fatal_error = net["r%s" % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -330,7 +330,7 @@ def test_zebra_ipv4_routingTable():
                 .cmd('vtysh -c "show ip route" 2> /dev/null | grep "^R"')
                 .rstrip()
             )
-            # Drop timers on end of line (older Quagga Versions)
+            # Drop timers on end of line
             actual = re.sub(r", [0-2][0-9]:[0-5][0-9]:[0-5][0-9]", "", actual)
             # Fix newlines (make them all the same)
             actual = ("\n".join(actual.splitlines()) + "\n").splitlines(1)
@@ -362,7 +362,7 @@ def test_zebra_ipv4_routingTable():
         fatal_error = net["r%s" % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
index 23e689235c50e5940dd44b83702cc1a82a9454c2..2976cdefe48629ed23f39d4b3edbbb0e48bd3c43 100755 (executable)
@@ -127,7 +127,7 @@ def setup_module(module):
         net["r%s" % i].loadConf("ripngd", "%s/r%s/ripngd.conf" % (thisDir, i))
         net["r%s" % i].startRouter()
 
-    # For debugging after starting Quagga/FRR daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -149,7 +149,7 @@ def test_router_running():
     if fatal_error != "":
         pytest.skip(fatal_error)
 
-    print("\n\n** Check if FRR/Quagga is running on each Router node")
+    print("\n\n** Check if FRR is running on each Router node")
     print("******************************************\n")
 
     # Starting Routers
@@ -157,7 +157,7 @@ def test_router_running():
         fatal_error = net["r%s" % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -182,7 +182,7 @@ def test_converge_protocols():
         fatal_error = net["r%s" % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -250,7 +250,7 @@ def test_ripng_status():
         fatal_error = net["r%s" % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -317,7 +317,7 @@ def test_ripng_routes():
         fatal_error = net["r%s" % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)
 
 
@@ -351,7 +351,7 @@ def test_zebra_ipv6_routingTable():
             )
             # Mask out Link-Local mac address portion. They are random...
             actual = re.sub(r" fe80::[0-9a-f:]+", " fe80::XXXX:XXXX:XXXX:XXXX", actual)
-            # Drop timers on end of line (older Quagga Versions)
+            # Drop timers on end of line
             actual = re.sub(r", [0-2][0-9]:[0-5][0-9]:[0-5][0-9]", "", actual)
             # Fix newlines (make them all the same)
             actual = ("\n".join(actual.splitlines()) + "\n").splitlines(1)
@@ -383,7 +383,7 @@ def test_zebra_ipv6_routingTable():
         fatal_error = net["r%s" % i].checkRouterRunning()
         assert fatal_error == "", fatal_error
 
-    # For debugging after starting FRR/Quagga daemons, uncomment the next line
+    # For debugging after starting FRR daemons, uncomment the next line
     # CLI(net)