summaryrefslogtreecommitdiffstats
path: root/tests (follow)
Commit message (Collapse)AuthorAgeFilesLines
* tests: update munet to 0.15.4Christian Hopps2025-01-125-14/+353
| | | | | | | | | - add readline and waitline functions for use with popen objects - other non-topotest (munet native) run changes - vm/qemu support booting cloud images (rocky, ubuntu, debian) - native topology init commands Signed-off-by: Christian Hopps <chopps@labn.net>
* tests: avoid nondeterministic routeJafar Al-Gharaibeh2025-01-094-36/+15
| | | | Signed-off-by: Jafar Al-Gharaibeh <jafar@atcorp.com>
* tests: bgp_srv6l3vpn_to_bgp_vrf3 needs more timeDonald Sharp2025-01-091-3/+3
| | | | | | | | | The test starts with checking for rib insertion of routes that may take some time after system startup to come up. Under heavy load this may cause this test to just fail. Give it more time. Signed-off-by: Donald Sharp <sharpd@nvidia.com>
* tests: bgp_srv6_sid_reachability should give more timeDonald Sharp2025-01-091-1/+1
| | | | | | | | | The test starts right in on check_pings with a 10 second time out. Any type of delay on startup is going to cause problems. Give the first check_ping significant time for the test to be fully brought up. Signed-off-by: Donald Sharp <sharpd@nvidia.com>
* Merge pull request #17707 from gromit1811/pr16811_rebasedRuss White2025-01-076-54/+448
|\ | | | | tests: cleanup ospf6 ecmp inter area
| * tests: ospf6_ecmp_inter_area, no shutdown r7/r8 eth3Martin Buck2025-01-072-6/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Drop eth3 shutdown from ospf6d.conf - it doesn't do anything there. And it actually shouldn't do anything: eth3 on r7/r8 are used as loopback-like interfaces to inject the address on eth2 into OSPFv3. So they need to be up for eth2 to work as expected. Based on original PR#16811 commit: eth3 shutdown is not applied because it is ospf6d.conf. Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com> Signed-off-by: Martin Buck <mb-tmp-tvguho.pbz@gromit.dyndns.org>
| * tests: ospf6_ecmp_inter_area, use router_json_cmpMartin Buck2025-01-074-48/+448
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use router_json_cmp when checking for proper nexthops to get better diagnostics in case of mismatches. Add step() messages to see which check failed. Based on original PR#16811 commit: expect_num_nexthops() errors are not understandable. Use router_json_cmp. Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com> Signed-off-by: Martin Buck <mb-tmp-tvguho.pbz@gromit.dyndns.org>
* | Merge pull request #17781 from LabNConsulting/chopps/fix-keyless-list-queryDonald Sharp2025-01-072-0/+46
|\ \ | | | | | | fix xpath query on keyless list with positional predicate
| * | tests: add unit test case for keyless list xpath queriesChristian Hopps2025-01-072-0/+46
| | | | | | | | | | | | Signed-off-by: Christian Hopps <chopps@labn.net>
* | | Merge pull request #17782 from LabNConsulting/chopps/new-notify-msg-fmtDonald Sharp2025-01-071-5/+37
|\ \ \ | | | | | | | | New YANG notify msg fmt
| * | | tests: fe-client: only pick mgmtd FE notify selectors onceChristian Hopps2025-01-071-5/+37
| |/ / | | | | | | | | | | | | | | | - adapt fe_client to new notification message format Signed-off-by: Christian Hopps <chopps@labn.net>
* | | Merge pull request #17783 from LabNConsulting/chopps/new-oper-get-callbackDonald Sharp2025-01-072-4/+49
|\ \ \ | | | | | | | | Add new oper state get callback
| * | | tests: add test for new northbound get functionChristian Hopps2025-01-072-4/+49
| |/ / | | | | | | | | | Signed-off-by: Christian Hopps <chopps@labn.net>
* | | Merge pull request #17733 from pguibert6WIND/bmp_event_changesRuss White2025-01-073-8/+192
|\ \ \ | | | | | | | | BMP handling of BGP configuration changes
| * | | topotests: bgp_bmp, add peer message test when RD changesPhilippe Guibert2024-12-302-0/+79
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a test to control the peer up and down message sent when the route distinguisher is updated. Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
| * | | topotests: bmp_collector, fix display peer distinguisher IP:AS2BPhilippe Guibert2024-12-301-3/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The peer distinguisher format was badly displayed when a per peer header is sent with an IP:AS2B format. > {"peer_type": "loc-rib instance", "is_filtered": false, "policy": "loc-rib", > "peer_distinguisher": "None:2", ... The IP header is incorrectly read. Fix it. > {"peer_type": "loc-rib instance", "is_filtered": false, "policy": "loc-rib", > "peer_distinguisher": "3.3.3.5:2", ... Fixes: 875511c46676 ("topotests: add basic bmp collector") Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
| * | | topotests: bgp_bmp, add peer up message test when router-id changesPhilippe Guibert2024-12-302-1/+58
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a test to control the value of the peer bgp id of loc-rib peer up messages when the bgp router-id is reconfigured. Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
| * | | topotests: bgp_bmp, add a test to check for bgp vrf peer loc-rib messagePhilippe Guibert2024-12-302-4/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a test where, when the vrf interface is flapping, a peer down and a peer up message are sent. This test, when used with ASAN, detects the memory leak of the open_tx and open_rx messages of the loc-rib. Refresh the method of updating the SEQ value when reading the peer messages: only update to the last matching SEQ value. Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
* | | | Merge pull request #17773 from LabNConsulting/chopps/more-oper-test-fixRuss White2025-01-072-2/+24
|\ \ \ \ | | | | | | | | | | tests: improve test reliability
| * | | | tests: improve test reliabilityChristian Hopps2025-01-062-2/+24
| | |/ / | |/| | | | | | | | | | | | | | | | | | Test for presence of IPv6 kernel routes too. Signed-off-by: Christian Hopps <chopps@labn.net>
* | | | Merge pull request #17474 from sougata-github-nvidia/rib_ip_protocol_cleanupRuss White2025-01-071-0/+14
|\ \ \ \ | | | | | | | | | | zebra: Fix ip protocol route-map issue.
| * | | | zebra: Fix ip protocol route-map issue.Sougata Barik2025-01-061-0/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "ip/ipv6 protocol any route-map <route map>" cli is setting wrong route type ( ZEBRA_ROUTE_MAX ), It should set route type ZEBRA_ROUTE_ALL. Ticket: #4101560 Signed-off-by: Sougata Barik <sougatab@nvidia.com>
* | | | | Merge pull request #17431 from krishna-samy/bgpd_json_commitsRuss White2025-01-071-0/+15
|\ \ \ \ \ | |_|/ / / |/| | | | bgpd: show json output changes to optimize various show commands
| * | | | bgpd: add topotest for show bgp router json commandKrishnasamy R2024-11-191-0/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit has changes to validate show bgp router json command Signed-off-by: Krishnasamy R <krishnasamyr@nvidia.com>
* | | | | Merge pull request #17728 from dmytroshytyi-6WIND/bgp_rpki_stateDonatas Abraitis2025-01-012-0/+99
|\ \ \ \ \ | |_|_|/ / |/| | | | bgpd: add rpki current state
| * | | | tests: add bgp rpki topo1 rpkiValidationStateDmytro Shytyi2024-12-302-0/+99
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Topotest to verify the next key-value "rpkiValidationState": "valid" Signed-off-by: Dmytro Shytyi <dmytro.shytyi@6wind.com>
* | | | | tests: Check prefix statistics for path-attribute discard/withdrawDonatas Abraitis2024-12-302-0/+42
|/ / / / | | | | | | | | | | | | Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
* | | | Merge pull request #17718 from cscarpitta/fix/fix-srv6-static-route-pytestmarkDonatas Abraitis2024-12-261-1/+1
|\ \ \ \ | | | | | | | | | | tests: Fix markers in `srv6_static_route` topotest
| * | | | tests: Fix markers in `srv6_static_route` topotestCarmine Scarpitta2024-12-241-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `srv6_static_route` is a pure staticd topotest. It does not have any dependency on bgpd and sharpd. Let's fix the pytestmark to include only staticd. Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>
* | | | | Merge pull request #17647 from LabNConsulting/fix-oper-testJafar Al-Gharaibeh2024-12-2421-168/+614
|\ \ \ \ \ | |/ / / / |/| | | | tests: enable test failure detection and fix resulting failures
| * | | | tests: enable failure and fix broken testsChristian Hopps2024-12-2421-168/+614
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This test was not actually failing when the results didn't match. Fail now. Also fix the tests that are now found to be failing incorrectly (wrong expected result). Signed-off-by: Christian Hopps <chopps@labn.net>
* | | | | Merge pull request #17580 from varuntumbe/dev/label_pool_release_fixDonatas Abraitis2024-12-235-0/+428
|\ \ \ \ \ | | | | | | | | | | | | BGP Labelpool : Releasing the label in labelpool when VPN session gets removed
| * | | | | topotests: Adding a test to control release of BGP labelsvaruntumbe2024-12-165-0/+428
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adding the topotest which verifies whether label beloning to corresponding chunk has been released properly or not once we remove the vpn session Signed-off-by: Varun Hegde <varuntumbe1@gmail.com>
* | | | | | tests: Check if enforce-first-as is working with peer-groupsDonatas Abraitis2024-12-211-4/+6
| |_|_|_|/ |/| | | | | | | | | | | | | | Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
* | | | | Merge pull request #17674 from ↵Jafar Al-Gharaibeh2024-12-207-52/+194
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | opensourcerouting/fix/bgp_show_advertised_routes_detail bgpd: Fix show neighbor X advertised-routes detail
| * | | | | tests: Ignore totalRoutes, totalPaths when checking strict outputDonatas Abraitis2024-12-191-3/+13
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
| * | | | | tests: Check if we received a DECENT amount of prefixes/pathsDonatas Abraitis2024-12-191-49/+40
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
| * | | | | tests: Check if we can see advertised routes with route-map appliedDonatas Abraitis2024-12-195-0/+141
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
* | | | | | test: fix label ordering on error diff reportChristian Hopps2024-12-191-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Christian Hopps <chopps@labn.net>
* | | | | | Merge pull request #17671 from Jafaral/static-propagationDonald Sharp2024-12-193-0/+74
|\ \ \ \ \ \ | | | | | | | | | | | | | | tests: add a test case for static route propagation
| * | | | | | tests: add a test case for static route propagationJafar Al-Gharaibeh2024-12-183-0/+74
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Jafar Al-Gharaibeh <jafar@atcorp.com>
* | | | | | | Merge pull request #17599 from ↵Jafar Al-Gharaibeh2024-12-181-1/+1
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | | | | | | | | | | | | | | | | opensourcerouting/fix/reduce_default_connect_timer bgpd: Connect retry timer backoff
| * | | | | | tests: Wait 30 seconds for a notification check (minimum hold-time)Donatas Abraitis2024-12-111-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
* | | | | | | Merge pull request #17544 from anlancs/zebra/fix-plug-interfaceRuss White2024-12-173-0/+88
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | zebra: fix wrong nexthop status for kernel routes
| * | | | | | | tests: add nexthop/interface's down/up topo for kernel routesanlan_cs2024-12-173-0/+88
| | |_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | Signed-off-by: anlan_cs <anlan_cs@126.com>
* | | | | | | Merge pull request #17526 from ↵Russ White2024-12-172-1/+26
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | raja-rajasekar/rajasekarr/evpn_bp_and_optimizations_3864372_FINAL_upstream EVPN L2VNI/L3VNI Optimize inline Global walk for remote route installations
| * | | | | | | bgpd: backpressure - Optimize EVPN L3VNI remote routes processingRajasekar Raja2024-12-092-1/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Anytime BGP gets a L3 VNI ADD/DEL from zebra, - Walking the entire global routing table per L3VNI is very expensive. - The next read (say of another VNI ADD/DEL) from the socket does not proceed unless this walk is complete. So for triggers where a bulk of L3VNI's are flapped, this results in huge output buffer FIFO growth spiking up the memory in zebra since bgp is slow/busy processing the first message. To avoid this, idea is to hookup the BGP-VRF off the struct bgp_master and maintain a struct bgp FIFO list which is processed later on, where we walk a chunk of BGP-VRFs and do the remote route install/uninstall. Ticket :#3864372 Signed-off-by: Rajasekar Raja <rajasekarr@nvidia.com>
* | | | | | | | Merge pull request #17252 from nabahr/mcast-modeRafael Zalamena2024-12-1627-42/+468
|\ \ \ \ \ \ \ \ | |_|/ / / / / / |/| | | | | | | Fix PIMD RPF lookup mode and nexthop tracking
| * | | | | | | tests: Add new pim mrib testsNathan Bahr2024-12-136-0/+441
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Test mrib overrides and rpf lookup mode changes. Signed-off-by: Nathan Bahr <nbahr@atcorp.com>
| * | | | | | | test: Fix tests for zebra changesNathan Bahr2024-12-1221-42/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove use of `ip multicast rpf-lookup-mode` from unrelated tests. Looks like this test was just unlucky enough to pick that command as an example for use here. Just changed it to something less likely to be removed in the future. Update route table output to include AFI SAFI output. Signed-off-by: Nathan Bahr <nbahr@atcorp.com>
generated by cgit v1.2.3 (git 2.39.1) at 2025-01-30 10:04:46 +0100