summaryrefslogtreecommitdiffstats
path: root/tests/topotests (follow)
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #16945 from ↵Russ White2024-10-084-0/+134
|\ | | | | | | | | opensourcerouting/fix/bgp_received-routes_with_soft_inbound bgpd: Show unmodified version of received-routes per neighbor
| * tests: Check if we can see unmodified received-routes with soft inboundDonatas Abraitis2024-09-274-0/+134
| | | | | | | | Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
* | Merge pull request #16941 from opensourcerouting/fix/issue_16877Russ White2024-10-086-289/+170
|\ \ | | | | | | bgpd: Relax the same prefix and nexthop to be valid
| * | tests: Check if loopback routes are considered valid for nexthop trackingDonatas Abraitis2024-10-015-0/+170
| | | | | | | | | | | | Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
| * | tests: Drop test_bgp_with_loopback_with_same_subnet_p1Donatas Abraitis2024-10-011-289/+0
| | | | | | | | | | | | | | | | | | | | | | | | It's replaced and simplified by c3fd1e9520c619babb3004cea6df622ca67b0dfa. JSON topo is just horrible to debug. Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
* | | Merge pull request #15903 from y-bharath14/srib-topotests-v1Russ White2024-10-081-1/+10
|\ \ \ | | | | | | | | tests: catch exception during switch shutdown
| * | | tests: catch exception during switch shutdownY Bharath2024-09-251-1/+10
| | |/ | |/| | | | | | | Signed-off-by: y-bharath14 <y.bharath@samsung.com>
* | | Merge pull request #16800 from donaldsharp/nhg_reuse_intf_down_upRuss White2024-10-042-8/+2
|\ \ \ | | | | | | | | Nhg reuse intf down up
| * | | tests: When finding nexthops ensure that they are activeDonald Sharp2024-09-162-8/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Do not accept a nexthop as valid unless it is marked as being active. Signed-off-by: Donald Sharp <sharpd@nvidia.com>
* | | | tests: Clean up some logging in test_bgp_default_originate_2links.pyDonald Sharp2024-10-011-5/+6
| |_|/ |/| | | | | | | | | | | | | | Test was confusing. Add some useful data and clean up some debugs Signed-off-by: Donald Sharp <sharpd@nvidia.com>
* | | tests: enhance autorp topotestBarry A. Trent2024-09-271-10/+144
| |/ |/| | | | | Signed-off-by: Barry A. Trent <barry.trent@atcorp.com>
* | tests: Addition of AutoRP Discovery uncovered broken PIM testNathan Bahr2024-09-241-4/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With AutoRP discovery running by default, that adds a new IGMP group that needs to be accounted for in IGMP output. For pim.py The clear IGMP interfaces function is in a broken state. It was already ignoring any errors and returned true always, but with the addition of the AutoRP discovery group, you could end up with a different group order in the json which would cause a key error making the test fail. For now I just added a check to avoid the key error. Signed-off-by: Nathan Bahr <nbahr@atcorp.com>
* | tests: Update existing tests for changes introduced by AutoRP DiscoveryNathan Bahr2024-09-245-6/+3
| | | | | | | | | | | | | | | | | | | | With AutoRP discovery running by default, that adds a new IGMP group that needs to be accounted for in IGMP output. For multicast_pim_sm_topo3: Ignore the total group number as it is unnecessary for the test. Signed-off-by: Nathan Bahr <nbahr@atcorp.com>
* | tests: Added tests for new AutoRP functionalityNathan Bahr2024-09-245-0/+324
| | | | | | | | | | | | | | | | Uses hardcoded sample AutoRP packets injected in to test message parsing and proper application of AutoRP learned RP info. Tests mix of AutoRP and static RP's. Signed-off-by: Nathan Bahr <nbahr@atcorp.com>
* | Merge pull request #16861 from btrent98/igmp-proxy2Donald Sharp2024-09-246-1/+460
|\ \ | | | | | | Add igmp proxy support
| * | tests: add topotest for igmp proxyBarry A. Trent2024-09-236-1/+460
| | | | | | | | | | | | Signed-off-by: Barry A. Trent <barry.trent@atcorp.com>
* | | Merge pull request #16906 from opensourcerouting/fix/match_peer_selfRuss White2024-09-246-0/+167
|\ \ \ | | | | | | | | bgpd: Return NOMATCH only if both conf_if/group does not exist
| * | | tests: Add a basic test to verify `match peer ...`Donatas Abraitis2024-09-246-0/+167
| | | | | | | | | | | | | | | | Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
* | | | Merge pull request #16904 from louis-6wind/fix-zebra-rib-rangeRuss White2024-09-241-1/+1
|\ \ \ \ | | | | | | | | | | tests: fix zebra_rib range
| * | | | tests: fix zebra_rib rangeLouis Scalbert2024-09-241-1/+1
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Range is wrong. We want values 1 and 2 but we only test 1. > >>> for i in range(1, 2): > ... print(i) > ... > 1 Fixes: abd2a1ff3f ("tests: Test some basic kernel <-> zebra interactions") Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>
* | | | Merge pull request #16895 from opensourcerouting/fix/dual_as_topotestRuss White2024-09-242-33/+1
|\ \ \ \ | |/ / / |/| | | tests: Simplify BGP dual-as topotest
| * | | tests: Simplify BGP dual-as topotestDonatas Abraitis2024-09-232-33/+1
| | | | | | | | | | | | | | | | | | | | | | | | Test if we can establish a session with a global ASN directly. Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
* | | | tests: Add v4/v6 forwarding off/onDonald Sharp2024-09-234-0/+152
|/ / / | | | | | | | | | | | | | | | | | | | | | There are no tests that ensured that turning off then on v4 and v6 forwarding actually worked. This does so. This was found via looking at the code coverage. Signed-off-by: Donald Sharp <sharpd@nvidia.com>
* | | tests: Add testcase for SRv6 End.DT6Carmine Scarpitta2024-09-221-0/+24
| | | | | | | | | | | | Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>
* | | tests: Add testcase for SRv6 End.DT4Carmine Scarpitta2024-09-221-0/+25
| | | | | | | | | | | | Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>
* | | tests: Add testcase for SRv6 End.DX6Carmine Scarpitta2024-09-221-0/+24
| | | | | | | | | | | | Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>
* | | tests: Fix unused variable warningCarmine Scarpitta2024-09-221-1/+1
| | | | | | | | | | | | | | | | | | Fix warning unused variable `router_list`. Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>
* | | Merge pull request #16889 from cscarpitta/fix/remove-duplicate-import-toopotestJafar Al-Gharaibeh2024-09-211-1/+1
|\ \ \ | | | | | | | | tests: Remove duplicate `check_ping` import
| * | | tests: Remove duplicate `check_ping` importCarmine Scarpitta2024-09-211-1/+1
| | | | | | | | | | | | | | | | Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>
* | | | Merge pull request #16888 from cscarpitta/fix/remove-log-monitor-cliJafar Al-Gharaibeh2024-09-213-3/+0
|\ \ \ \ | | | | | | | | | | tests: Remove `log monitor XX ` from topotests
| * | | | tests: Remove `log monitor XX` from `srv6_sid_manager` testCarmine Scarpitta2024-09-211-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `log monitor XX` command is deprecated and does nothing. Let's remove it. Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>
| * | | | tests: Remove `log monitor XX` from `srv6_sid_manager` testCarmine Scarpitta2024-09-211-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `log monitor XX` command is deprecated and does nothing. Let's remove it. Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>
| * | | | tests: Remove `log monitor XX` from `srv6_encap_src_addr` testCarmine Scarpitta2024-09-211-1/+0
| |/ / / | | | | | | | | | | | | | | | | | | | | `log monitor XX` command is deprecated and does nothing. Let's remove it. Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>
* / / / tests: Add a very basic `dump bgp PATH..` topotestDonald Sharp2024-09-203-0/+131
|/ / / | | | | | | | | | | | | | | | | | | | | | None of the bgp dump code was even tested. Add a bit of basic stuff that it at least generates a dump file. This can be extended at a future time. Signed-off-by: Donald Sharp <sharpd@nvidia.com>
* / / tests: adjust topotest after bgp_redistribute_add() fixEnke Chen2024-09-194-7/+7
|/ / | | | | | | | | | | | | | | In ospf_multi_vrf_bgp_route_leak, the admin distance for the redistributed ospf route should be 110, and should remain as 110 after it's imported into another vrf, and then downloaded to zebra. Signed-off-by: Enke Chen <enchen@paloaltonetworks.com>
* | Merge pull request #16857 from donaldsharp/logger_warningMark Stapp2024-09-182-2/+2
|\ \ | | | | | | tests: use logger.warning not logger.warn
| * | tests: use logger.warning not logger.warnDonald Sharp2024-09-182-2/+2
| | | | | | | | | | | | | | | | | | | | | Apparently logger.warn is being deprecated. So let's switch over to logger.warning. Clearly it's better Signed-off-by: Donald Sharp <sharpd@nvidia.com>
* | | Merge pull request #16816 from opensourcerouting/feature/bgp_dual_asDonald Sharp2024-09-184-0/+145
|\ \ \ | |/ / |/| | bgpd: Implement BGP dual-as feature
| * | tests: Test `neighbor X no-prepend replace-as dual-as`Donatas Abraitis2024-09-134-0/+145
| |/ | | | | | | Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
* | tests: add test for new ietf-yang-library supportChristian Hopps2024-09-182-0/+55
| | | | | | | | Signed-off-by: Christian Hopps <chopps@labn.net>
* | Merge pull request #16828 from enkechen-panw/print-rnhDonald Sharp2024-09-162-15/+15
|\ \ | | | | | | zebra: include the prefix in nht show command
| * | zebra: include the prefix in nht show commandEnke Chen2024-09-152-15/+15
| |/ | | | | | | | | | | Include the prefix in "show ip nht" and "show ipv6 nht". Signed-off-by: Enke Chen <enchen@paloaltonetworks.com>
* | tests: fix spawning shells/vtysh on error in xdist modeChristian Hopps2024-09-141-11/+27
| | | | | | | | | | | | - Also fix the above and CLI when running in munet native mode Signed-off-by: Christian Hopps <chopps@labn.net>
* | tests: update munet 0.14.14, 0.14.15Christian Hopps2024-09-141-5/+33
| | | | | | | | | | | | | | - improvement to remote CLI - 0.14.15 fixed a small bug in 0.14.14 Signed-off-by: Christian Hopps <chopps@labn.net>
* | tests: update munet 0.14.13Christian Hopps2024-09-141-1/+21
| | | | | | | | | | | | - Improve remote CLI operation [improves xdist mode] Signed-off-by: Christian Hopps <chopps@labn.net>
* | tests: update munet 0.14.12Christian Hopps2024-09-142-11/+19
| | | | | | | | | | | | | | - Adds retry_sleep (i.e., interval) parameter to native @retry decorator - Fix --stdout and --stderr munet CLI args Signed-off-by: Christian Hopps <chopps@labn.net>
* | Merge pull request #16438 from Jafaral/pim-10.1-cand-rpDonald Sharp2024-09-149-1/+480
|\ \ | | | | | | PIM candidate BSR/RP support
| * | tests: don't set oif to a list if it is NoneJafar Al-Gharaibeh2024-09-101-1/+1
| | | | | | | | | | | | Signed-off-by: Jafar Al-Gharaibeh <jafar@atcorp.com>
| * | tests: add candidate BSR/RP topotestJafar Al-Gharaibeh2024-09-108-0/+479
| | | | | | | | | | | | Signed-off-by: Jafar Al-Gharaibeh <jafar@atcorp.com>
* | | Merge pull request #16815 from ↵Donald Sharp2024-09-131-5/+13
|\ \ \ | | | | | | | | | | | | | | | | opensourcerouting/fix/adjust_wait_count_default_timers_if_too_low tests: Adjust minimum wait/count timers for run_and_expect() if they are too low
generated by cgit v1.2.3 (git 2.39.1) at 2024-12-02 05:24:32 +0100