summaryrefslogtreecommitdiffstats
path: root/tests (follow)
Commit message (Collapse)AuthorAgeFilesLines
* ospfd: show ip ospf interface only shows BDR info, Added DR info.rgirada2021-04-291-0/+2
| | | | | | | | Description: DR information is missing under "show ip ospf interface [json]". Added DR infomation to display in "show ip ospf interface". Signed-off-by: Rajesh Girada <rgirada@vmware.com>
* Merge pull request #8546 from ton31337/fix/add_markers_for_recent_bgpdQuentin Young2021-04-282-0/+4
|\
| * tests: Add pytest bgpd markers for recent BGP testsDonatas Abraitis2021-04-272-0/+4
| | | | | | | | Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
* | Merge pull request #8322 from qlyoung/topotestsJafar Al-Gharaibeh2021-04-282-55/+6
|\ \ | | | | | | Assorted topotests cleanup
| * | tests: fix use of undefined variable in pim libQuentin Young2021-04-071-2/+1
| | | | | | | | | | | | Signed-off-by: Quentin Young <qlyoung@nvidia.com>
| * | tests: fix improper format strings in pim libQuentin Young2021-04-071-8/+3
| | | | | | | | | | | | Signed-off-by: Quentin Young <qlyoung@nvidia.com>
| * | tests: remove duplicate pim test functionQuentin Young2021-04-071-45/+1
| | | | | | | | | | | | | | | | | | Twice defined Signed-off-by: Quentin Young <qlyoung@nvidia.com>
| * | tests: provide sane default for --topology-onlyQuentin Young2021-04-071-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: /usr/lib/python3.9/site-packages/_pytest/config/__init__.py:1463: in getoption val = getattr(self.option, name) E AttributeError: 'Namespace' object has no attribute 'topology_only' The above exception was the direct cause of the following exception: /usr/lib/python3.9/site-packages/pluggy/manager.py:127: in register hook._maybe_apply_history(hookimpl) /usr/lib/python3.9/site-packages/pluggy/hooks.py:333: in _maybe_apply_history res = self._hookexec(self, [method], kwargs) /usr/lib/python3.9/site-packages/pluggy/manager.py:93: in _hookexec return self._inner_hookexec(hook, methods, kwargs) /usr/lib/python3.9/site-packages/pluggy/manager.py:84: in <lambda> self._inner_hookexec = lambda hook, methods, kwargs: hook.multicall( tests/topotests/conftest.py:62: in pytest_configure if config.getoption("--topology-only"): /usr/lib/python3.9/site-packages/_pytest/config/__init__.py:1474: in getoption raise ValueError(f"no option named {name!r}") from e E ValueError: no option named 'topology_only' Signed-off-by: Quentin Young <qlyoung@nvidia.com>
* | | tests: fix topotest polling logIgor Ryzhov2021-04-281-2/+2
| |/ |/| | | | | | | | | | | | | | | | | The current log prints maximum wait time which is not actually correct, because it doesn't include the command execution time. We usually have "failed after X seconds" log with X being far longer than this maximum. Let's print the maximum number of tries instead. Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>
* | Merge pull request #8540 from donaldsharp/kill_them_allRafael Zalamena2021-04-266-38/+0
|\ \ | | | | | | tests: Remove kill_mininet_router_process
| * | tests: Remove kill_mininet_router_processDonald Sharp2021-04-226-38/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | This function kills all processes that happen to have the same name to frr processes and it was only ever used in the setup. Setup should not be used to kill old runs. That should be a separate process. Signed-off-by: Donald Sharp <sharpd@nvidia.com>
* | | Merge pull request #8555 from idryzhov/fix-bfd-testRafael Zalamena2021-04-261-3/+3
|\ \ \ | | | | | | | | tests: fix bfd-bgp-cbit-topo3 test
| * | | tests: fix bfd-bgp-cbit-topo3 testIgor Ryzhov2021-04-241-3/+3
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This test is completely incorrect on test_bfd_loss_intermediate step. It shuts down the interface and then "waiting" for the BGP session to fail. But instead of the actual wait it compares the output of "show bfd peers" with the "up" state. As it does this comparison right after the interface shutdown, the BFD session has not yet failed and the comparison is always successful except very rare cases when the command takes a lot of time to execute (due to the heavy load on CI system I suppose). Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>
* | | Merge pull request #8495 from opensourcerouting/ospfv3-bfd-reworkIgor Ryzhov2021-04-235-5/+13
|\ \ \ | | | | | | | | ospf6d: rework BFD integration and add BFD profiles support
| * | | topotests: test OSPFv3 BFD profilesRafael Zalamena2021-04-224-5/+9
| | | | | | | | | | | | | | | | | | | | | | | | See if the BFD sessions created by OSPFv3 are using the configured BFD profiles. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
| * | | topotests: converge OSPFv3 fasterRafael Zalamena2021-04-222-0/+4
| |/ / | | | | | | | | | | | | | | | Tweak OSPFv3 interface configuration to converge faster. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
* | | topotests: test PIM BFD profile integrationRafael Zalamena2021-04-233-2/+38
| | | | | | | | | | | | | | | | | | Test that PIM BFD profile integration works. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
* | | topotests: add new basic PIM topology testRafael Zalamena2021-04-2316-0/+329
|/ / | | | | | | | | | | Test for PIM (re)convergence and BFD usage. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
* | build: properly split CFLAGS from AC_CFLAGSDavid Lamparter2021-04-211-0/+1
| | | | | | | | | | | | | | | | `CFLAGS` is a "user variable", not intended to be controlled by configure itself. Let's put all the "important" stuff in AC_CFLAGS and only leave debug/optimization controls in CFLAGS. Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
* | Merge pull request #8434 from donaldsharp/python_bullshitMartin Winter2021-04-165-493/+18
|\ \ | | | | | | Python pylint stuff
| * | tests: test_bgp_vrf_dynamic_route_leak_topo1.py was missing a variableDonald Sharp2021-04-151-0/+1
| | | | | | | | | | | | Signed-off-by: Donald Sharp <sharpd@nvidia.com>
| * | tests: Fix test_bgp_multi_vrf_topo2.py string formattingDonald Sharp2021-04-151-20/+9
| | | | | | | | | | | | Signed-off-by: Donald Sharp <sharpd@nvidia.com>
| * | tests: test_bgp_multi_vrf_topo1.py fix formatting issues pointed out by pylintDonald Sharp2021-04-151-13/+6
| | | | | | | | | | | | Signed-off-by: Donald Sharp <sharpd@nvidia.com>
| * | tests: Remove unnecessary formating for stringsDonald Sharp2021-04-151-2/+2
| | | | | | | | | | | | Signed-off-by: Donald Sharp <sharpd@nvidia.com>
| * | tests: Remove duplicate functions in python codeDonald Sharp2021-04-151-458/+0
| | | | | | | | | | | | | | | | | | | | | We have basically two identical functions in test_bgp_gr_functionality_topo1.py Fix. Signed-off-by: Donald Sharp <sharpd@nvidia.com>
* | | Merge pull request #8437 from donaldsharp/verify_bgp_community_no_sleepMartin Winter2021-04-151-1/+0
|\ \ \ | |/ / |/| | tests: Do not sleep(5) when verifying bgp communities
| * | tests: Do not sleep(5) when verifying bgp communitiesDonald Sharp2021-04-091-1/+0
| | | | | | | | | | | | | | | | | | | | | There are better ways of ensuring that the remote side has your change instead of sleeping Signed-off-by: Donald Sharp <sharpd@nvidia.com>
* | | Merge pull request #8448 from ↵Sri Mohana Singamsetty2021-04-1411-0/+11
|\ \ \ | | | | | | | | | | | | | | | | ton31337/feature/show_rpki_state_in_global_show_bgp bgpd: Show RPKI short state in `show bgp <afi> <safi>`
| * | | tests: Add BGP RPKI show header to outputsDonatas Abraitis2021-04-1211-0/+11
| | | | | | | | | | | | | | | | Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
* | | | Merge pull request #8145 from pguibert6WIND/nhrp_use_zebraMark Stapp2021-04-1311-0/+457
|\ \ \ \ | |/ / / |/| | | nhrp: use zebra
| * | | topotests: add standard nhrp test between spoke and hubPhilippe Guibert2021-04-0911-0/+457
| |/ / | | | | | | | | | | | | | | | | | | This test establishes a binding between nbma ip of a spoke and its protocol address. This information is pushed to hub. Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
* | | tests: More black fixupsDonald Sharp2021-04-0935-872/+1078
| | | | | | | | | | | | | | | | | | Just another round of fixups found by running black on the code Signed-off-by: Donald Sharp <sharpd@nvidia.com>
* | | tests: Add some more pytest marks for bgpdDonald Sharp2021-04-0912-1/+41
|/ / | | | | | | | | | | | | Just another round of trying to add pytest.mark.bgpd. Not finished yet just what I could stand doing for a few minutes. Signed-off-by: Donald Sharp <sharpd@nvidia.com>
* | Revert "bgpd: BGP session not established for ipv6 link local address with ↵Donatas Abraitis2021-04-096-148/+0
| | | | | | | | | | | | vrf config" This reverts commit 0b6d35324ef3fd2b4d5997103795cad73eb72954.
* | Merge pull request #8402 from opensourcerouting/debian10-topotest-fixPatrick Ruddy2021-04-087-80/+71
|\ \ | | | | | | tests: fix for l3mdev topotests on kernel 4.19
| * | tests: fix python3 backported iproute2 vrf checkDavid Schweizer2021-04-051-2/+1
| | | | | | | | | | | | Signed-off-by: David Schweizer <dschweizer@opensourcerouting.org>
| * | tests: fix l3mdev config in isis-topo1-vrf topotestDavid Schweizer2021-04-051-25/+6
| | | | | | | | | | | | Signed-off-by: David Schweizer <dschweizer@opensourcerouting.org>
| * | tests: fix l3mdev config in bgp_vrf_lite_ipv6_rtadv topotestDavid Schweizer2021-04-051-25/+3
| | | | | | | | | | | | Signed-off-by: David Schweizer <dschweizer@opensourcerouting.org>
| * | tests: fix l3mdev config in bgp_l3vpn_to_bgp_vrf topotestDavid Schweizer2021-04-052-18/+9
| | | | | | | | | | | | Signed-off-by: David Schweizer <dschweizer@opensourcerouting.org>
| * | tests: fix l3mdev config in bgp_evpn_rt5 topotestDavid Schweizer2021-04-051-10/+2
| | | | | | | | | | | | Signed-off-by: David Schweizer <dschweizer@opensourcerouting.org>
| * | tests: add l3mdev config helpers for VRF topotestsDavid Schweizer2021-04-051-0/+50
| | | | | | | | | | | | Signed-off-by: David Schweizer <dschweizer@opensourcerouting.org>
* | | Merge pull request #7434 from sudhanshukumar22/bgp-link-local-addressDonatas Abraitis2021-04-086-0/+148
|\ \ \ | | | | | | | | bgpd: BGP session not established for ipv6 link local address with vrf config
| * | | bgpd: BGP session not established for ipv6 link local address with vrf configsudhanshukumar222021-04-016-0/+148
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Description: BGP session not established for ipv6 link local address with vrf config Problem Description/Summary : BGP session not established for ipv6 link local address with vrf configyy 1.Configure ipv6 link-local address fe80::1234/64 on dut1 and fe80::4567/64 on dut2 2.Configure BGP neighbors for ipv6 link-local on both dut1 and dut2 3.Verify BGP session is UP over link-local ipv6 address 4.Observed that bgp session not established for ipv6 link local address Expected Behavior : BGP session should be established for ipv6 link local address with vrf config Signed-off-by: sudhanshukumar22 <sudhanshu.kumar@broadcom.com>
* | | | Merge pull request #8365 from ↵Mark Stapp2021-04-076-338/+358
|\ \ \ \ | |_|_|/ |/| | | | | | | | | | | aldobrrrr/fix_parse_topology_function_in_isis_topo1_vrf_test topotests: fix 'parse_topology' function in test_isis_topo1_vrf.py
| * | | topotests: fix 'parse_topology' function in test_isis_topo1_vrf.pyEmanuele Altomare2021-03-296-338/+358
| | | | | | | | | | | | | | | | | | | | | | | | Now the parse_topology function handles well the vertex type identifier. Signed-off-by: Emanuele Altomare <emanuele@common-net.org>
* | | | Merge pull request #8407 from ↵Russ White2021-04-068-0/+157
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | ton31337/fix/nexthop_should_not_be_changed_if_peer_shares_common_subnet tests: Nexthop should not be changed if peer X shares a common subnet
| * | | | tests: Nexthop should not be changed if peer X shares a common subnetDonatas Abraitis2021-04-068-0/+157
| | |_|/ | |/| | | | | | | | | | Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
* | | | Merge pull request #8295 from opensourcerouting/ospf6-topo-stabilizeMark Stapp2021-04-0610-94/+113
|\ \ \ \ | | | | | | | | | | topotest: stabilize OSPFv3 topology
| * | | | topotests: improve OSPF(v2|v3) convergence testRafael Zalamena2021-03-231-0/+72
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Wait until all routers are actually `Full` instead of just looking at routes. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
| * | | | topotests: remove old configuration filesRafael Zalamena2021-03-235-39/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We version the tests with FRR so it is no longer necessary to support old versions. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>