ldp-topo1: fix ip mpls route output format

Remove the different file as it is no longer needed.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

ospf6-topo1: make ip-route comparison more robust

Signed-off-by: Christian Franke <nobody@nowhere.ws>

ldp-topo1: fix parsing for iproute from Ubuntu 18.04

Signed-off-by: Christian Franke <nobody@nowhere.ws>

lib: bgprib - get human readable verson of json commands for log

Signed-off-by: Lou Berger <lberger@labn.net>

lib: allow all rw access to /tmp/topotests (for package support and debug)

Signed-off-by: Lou Berger <lberger@labn.net>

lib: use per node log directory when topotest is used without topogen

Signed-off-by: Lou Berger <lberger@labn.net>

lib: log to start/local (per node) dir, log commands

Signed-off-by: Lou Berger <lberger@labn.net>

GUIDELINES.md: update to reflect per node log dirs

Signed-off-by: Lou Berger <lberger@labn.net>

*: convert startup logs to use per node directories also have startup and running log files match

Signed-off-by: Lou Berger <lberger@labn.net>

lib: limit checkRouterCores output to once per router start

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_l3vpn_to_bgp_vrf: Fixup topo test to write log files to same location at start of run

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

lib: add check for core and memory issues after routerstop

Signed-off-by: Lou Berger <lberger@labn.net>

lib: convert to directories per node

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_l3vpn_to_bgp_vrf: more pings...

bgp_l3vpn_to_bgp_vrf: fix wrong table entries

lib: lutil - fix test numbering, add test number to command logging

bgp_l3vpn_to_bgp_vrf: remote static routes left over from old test versions

bgp_multiview_topo1: more fixes for header changes of PR 2045

For this test, vrf id in headers is unset

Signed-off-by: G. Paul Ziemba <paulz@labn.net>

bgp: tolerate route-table-show header changes of PR 2045

The table header has changed slightly for "bgp show" commands. Change
all-protocol-startup (test_bgp_ipv4() and test_bgp_ipv6()) and
bgp_multiview_topo1 (test_bgp_routingTable())  to run diffs against multiple
templates (representing the various valid outputs for different versions
of the FRR code) and mark the test "passed" if any one template matches.

Signed-off-by: G. Paul Ziemba <paulz@labn.net>

bgp_l3vpn_to_bgp_vrf: use FRR PR #2053

Check CE routes from CE in another VRF attached to same PE router. Relies
on FRR bug fix to not require labeled nexthop for paths that go only
through PE router and not mpls core.

Signed-off-by: G. Paul Ziemba <paulz@labn.net>

bgp_l3vpn_to_bgp_vrf: check_linux_mpls: delete obsolete code; formatting

Signed-off-by: G. Paul Ziemba <paulz@labn.net>

lib: fix path the led to group_nl_converted being referenced when not set

Signed-off-by: Lou Berger <lberger@labn.net>

lib/bgprib.py: add copyright notice

Signed-off-by: G. Paul Ziemba <paulz@labn.net>

lib: lutil matching without changing newlines

Prior behavior of luCommand was to convert newlines to spaces in DUT
output before pattern matching. New method operating in parallel uses
re.DOTALL to mimic same behavior and allow preserving original DUT output.
The original output is needed for some scripts that parse line-by-line.

There is also some test code to compare match results using the
old way and new way and log a message if they are different. After
some short time we can develop confidence that using this new method
will not break any existing tests.

Signed-off-by: G. Paul Ziemba <paulz@labn.net>

bgp_l3vpn_to_bgp_vrf: require frr 4.1

Signed-off-by: G. Paul Ziemba <paulz@labn.net>

bgp_l3vpn_to_bgp_vrf: update to current vpn-vrf leak syntax and behavior

Signed-off-by: G. Paul Ziemba <paulz@labn.net>

lib/bgprib.py: handle empty json results gracefully

Signed-off-by: G. Paul Ziemba <paulz@labn.net>

lib/ltemplate.py: add KeepGoing parameter to ltemplateTest

Signed-off-by: G. Paul Ziemba <paulz@labn.net>

test_bgp_l3vpn_to_bgp_vrf: match new nexthop=0 code

Signed-off-by: G. Paul Ziemba <paulz@labn.net>

test_bgp_l3vpn_to_bgp_vrf: small config comment cleanup

Signed-off-by: G. Paul Ziemba <paulz@labn.net>

ltemplate.py: start isisd when isisd.conf present

Signed-off-by: G. Paul Ziemba <paulz@labn.net>

ltemplate.py: remove dependency on hard-coded router name r2

Signed-off-by: G. Paul Ziemba <paulz@labn.net>

bgp_l3vpn_to_bgp_vrf: assign different RDs to customers on r4

Signed-off-by: G. Paul Ziemba <paulz@labn.net>

bgp_l3vpn_to_bgp_vrf: comment out route-add commands no longer needed

Signed-off-by: G. Paul Ziemba <paulz@labn.net>

bgp_l3vpn_to_bgp_vrf: check_routes test now uses bgprib for exact route checking

Signed-off-by: G. Paul Ziemba <paulz@labn.net>

lib/bgprib.py naming rib->bgprib is more accurate

Signed-off-by: G. Paul Ziemba <paulz@labn.net>

bgp_l3vpn_to_bgp_vrf: updated config files to current vpn-policy syntax

Signed-off-by: G. Paul Ziemba <paulz@labn.net>

lutil: expose result method

Signed-off-by: G. Paul Ziemba <paulz@labn.net>

lib: bgprib utility to compare rib against list of required routes

Signed-off-by: G. Paul Ziemba <paulz@labn.net>

bgp vpn-vrf leaking: partial changes toward using new functionality

Signed-off-by: G. Paul Ziemba <paulz@labn.net>

bgp_vrf_netns: revert changes from topotest 77

revert
https://github.com/FRRouting/topotests/pull/77
bgp_vrf_netns: do not run test on 32 bit linux machines

Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

bgp_vrf_netns: increase bgp convergence timer

BGP convergence timer is increased from 90 seconds to 120 seconds.

Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

bgp_vrf_netns: do not run test on 32 bit linux machines

It seems vrf netns is not working on 32 bit platforsm.
For that, the test should be cancelled.

Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

bgp_vrf_netns: attempt cleanup of previous test abnormal exit. skip test if setup fails

Signed-off-by: Lou Berger <lberger@labn.net>

lib: TopoExaBGP.start - report unexpected exabgp output

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_vrf_netns: allow more time for adjacency to come up, report unexpected output

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_vrf_netns: add basic test to support netns backend with netns

The test consist in starting a BGP router on a NETNS, and interacting
with an exaBGP instance. The test automatically relies on VRF with netns
backend but can be easily configured so as to perform basic BGP VRF
testing with VRF with VRF-lite backend.

Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

lib: zebra support for vrfwnetns parameter

topotest library is being added a new parameter when preparing the
configurationof each daemon. This parameter will be used to call the
daemon with some extra parameters. The -n parameter for zebra is taken
into account.
Also, when a extra parameter is given for calling zebra, a check is done to
see if it is possible to run zebra daemon with that option. This is the
case for vrfwnetns option. If not available, an error message is sent
back.

Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

bgp_l3vpn_to_bgp_vrf: fix route set on non-vrf interface (thanks ben!)

Signed-off-by: Lou Berger <lberger@labn.net>

lib: lutil - fix test counting, add returnJson flag

Signed-off-by: Lou Berger <lberger@labn.net>

ldp-vpls-topo1: support new OSPF JSON format

Support the new OSPF JSON output format for `show ip ospf neighbor
json` introduced in FRR's PR 1723.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

Revert previous 2 commits (pushed by mistake)

Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>

Update OSPF-SR tests to handle various version

In order to avoid failed test on unsupported version:
 - Test that version is greater than 4
 - Check that Segment Routing CLI are available

Signed-off-by: Olivier Dugeon <olivier.dugeon@orange.com>

Add OSPF Segment Routing tests

 - Build a topology made of 4 routers: 3 PE connected to a P
 - Segment Routing is activated on all 4 routers with various setting
for the MSD, Global Block and Prefix (with or without NO-PHP flag)
 - 2 tests have been elaborated: the first one check the consistency of the
Segment Routing database on each router compared to a reference and the second
one check that the MPLS FIB has been correctly configured on each router,
again compared to a reference

Signed-off-by: Olivier Dugeon <olivier.dugeon@orange.com>

ldp-topo1: Support implicit-null and old format on "show_mpls_table"

Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>

bgp_l3vpn_to_bgp_direct: use new ltemplate common functions

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_l3vpn_to_bgp_vrf: use new ltemplate common functions

Signed-off-by: Lou Berger <lberger@labn.net>

lib: ltemplate add common ltemplateRtrCmd and ltemplateVersionCheck

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_l3vpn_to_bgp_vrf: centralize iproute2 check, add checks to appropriate tests

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_l3vpn_to_bgp_vrf: required 4.11 kernel on arm processors

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_l3vpn_to_bgp_vrf: add test to check linux vrf info

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_l3vpn_to_bgp_vrf: prepend rtr name to vrf name, per Philippe

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_l3vpn_to_bgp_vrf: skip mpls vrf ping tests when have old iproute2

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_l3vpn_to_bgp_vrf: add testing for manually setup MPLS path (until have dynamic)

Signed-off-by: Lou Berger <lberger@labn.net>

isis-topo1: allow up to 90 sec for convergence

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_l3vpn_to_bgp_direct: use tgen.hasmpls flag

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_l3vpn_to_bgp_vrf: catch linux setup failures

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_l3vpn_to_bgp_vrf: collect iproute2 package into

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_l3vpn_to_bgp_vrf: add kernel check (>=4.9)

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_l3vpn_to_bgp_vrf: use tgen.hasmpls flag

Signed-off-by: Lou Berger <lberger@labn.net>

lib: add tgen level mpls init and hasmpls flag

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_l3vpn_to_bgp_vrf: 2nd try at skipping tests when MPLS not available

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_l3vpn_to_bgp_vrf: skip tests when mpls not available

Signed-off-by: Lou Berger <lberger@labn.net>

lib: add script name to summary output

Signed-off-by: Lou Berger <lberger@labn.net>

lib: add luLast to get last found luCommand regex match

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_l3vpn_to_bgp_direct: allow for some ping loss

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_l3vpn_to_bgp_vrf: allow for some ping loss

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_l3vpn_to_bgp_vrf: add basic data path checks prior to adjacency check

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_l3vpn_to_bgp_direct: add basic data path checks prior to adjacency check

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_l3vpn_to_bgp_xxx: match ltemplate start/stop changes

Signed-off-by: Lou Berger <lberger@labn.net>

lib: ltemplate simplify start/stop

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_l3vpn_to_bgp_xxx: avoid collision of customize.py match lutil and ltemplate changes

Signed-off-by: Lou Berger <lberger@labn.net>

lib: scope life of Lutil and _lt to a single test run reorg to support regression environment

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_l3vpn_to_bgp_vrf: just test adjacencies now - for dev/integration

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_l3vpn_to_bgp_vrf: don't set vrf-policy, wait for new format

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_l3vpn_to_bgp_vrf: add ce4 off of r4 for inter-vrf route leaking

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_l3vpn_to_bgp_vrf: in progress version Adjacenies up, MPLS configured on interfaces -- right now provides testbed for code changes

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_l3vpn_to_bgp_vrf: clone bgp_l3vpn_to_bgp_direct as starting point

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_l3vpn_to_bgp_direct: 2nd try at skipping tests when MPLS not available

Signed-off-by: Lou Berger <lberger@labn.net>

bgp_l3vpn_to_bgp_direct: skip tests when mpls not available

Signed-off-by: Lou Berger <lberger@labn.net>

lib: add router.has_mpls check

Signed-off-by: Lou Berger <lberger@labn.net>

topogen: fix assertion on `routers_have_failure`

`errors != ''` is never going to be false so the assert was never being
triggered.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

topogen: import pytest assert function

Use the pytest assert function to be able to report to pytest failures
that happened inside the API.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

isis-topo1: check IPv6 linux route install

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

isis-topo1: simplify dictionary key removal code

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

isis-topo1: check ipv6 zebra route installation

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

isis-topo1: skip test on FRR 2.0

It was noticed that this topology doesn't converge when there are IPv6
configurations, so this commit makes the tests to be skipped when the
version is 2.X.Y.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org

isis-topo1: add IPv6 ISIS topology test

Configure IPv6 addresses and ISIS distribution, then test the topology
for the newly configured addresses.

This commit also adds support for parsing IPv6 data from the ISIS
topology output while updating the routers topology JSON files.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

isis-topo1: add support for stable versions

Allow isis-topo1 tests to run on stable versions (2.0.2 and 3.0.2).

There are two differences between the stable versions and master:
1. ISIS nexthops don't show interfaceName nor interfaceIndex
   (on 'show ip route json')
2. New FRR version install ISIS routes in Linux with a different proto
   type identification.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

isis-topo1: check ISIS kernel route installation

Check if the ISIS learned routes are being installed in the underlying
OS.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>