summaryrefslogtreecommitdiff
path: root/tests/topotests/all_protocol_startup
AgeCommit message (Collapse)Author
2023-10-05tests: locate nhg id from json output in all_proto_startupMark Stapp
Don't hard-code a sharpd nhg id: those values aren't stable if the daemons/protos/route-types change. Use json show output to find the id in the 'resilient' nhg test case in the all_protocol_startup suite. Signed-off-by: Mark Stapp <mjs@labn.net>
2023-08-17zebra: Show NHT resolve via default status on/offDonatas Abraitis
``` donatas-laptop# show ip nht VRF default: Resolve via default: on 192.168.10.123 resolved via connected is directly connected, wlp82s0 (vrf default) Client list: bgp(fd 21) donatas-laptop# show ip nht json { "default":{ "ipv4":{ "resolveViaDefault":true, "192.168.10.123":{ "nhtConnected":false, "clientList":[ { "protocol":"bgp", "socket":21, "protocolFiltered":"none" } ], "nexthops":[ { "flags":3, "fib":true, "directlyConnected":true, "interfaceIndex":3, "interfaceName":"wlp82s0", "vrf":"default", "active":true } ], "resolvedProtocol":"connected" } } } } donatas-laptop# show ip nht vrf all VRF default: Resolve via default: on 192.168.10.123 resolved via connected is directly connected, wlp82s0 (vrf default) Client list: bgp(fd 21) donatas-laptop# show ip nht vrf all json { "default":{ "ipv4":{ "resolveViaDefault":true, "192.168.10.123":{ "nhtConnected":false, "clientList":[ { "protocol":"bgp", "socket":21, "protocolFiltered":"none" } ], "nexthops":[ { "flags":3, "fib":true, "directlyConnected":true, "interfaceIndex":3, "interfaceName":"wlp82s0", "vrf":"default", "active":true } ], "resolvedProtocol":"connected" } } } } donatas-laptop# ``` Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
2023-05-16Merge pull request #12949 from opensourcerouting/ospf-unplanned-grRuss White
OSPF GR for unplanned outages
2023-05-11topotest: all_protocol_startup - respect TOPOTESTS_CHECK_STDERR environ variableLou Berger
Signed-off-by: Lou Berger <lberger@labn.net>
2023-05-08tests: Fix ospfv3 output to include Graceful Restart infoMartin Winter
Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>
2023-04-07tests: don't flush ipv6 addressesChristian Hopps
Signed-off-by: Christian Hopps <chopps@labn.net>
2023-03-29tests: Turn off debug loggingDonald Sharp
Tests are taking up too much space, turn off debug logging for normal runs. Signed-off-by: Donald Sharp <sharpd@nvidia.com>
2023-03-22tests: Add topotests for MGMT daemonnguggarigoud
1. MGMT daemon support in topotests. 2. Sanity tests for MGMTd. Signed-off-by: nguggarigoud <nguggarigoud@vmware.com>
2023-02-28tests: Add test to check hello padding sometimes behaviorDiogo Oliveira
Signed-off-by: Diogo Oliveira <14191454+dorDiogo@users.noreply.github.com>
2023-02-09*: auto-convert to SPDX License IDsDavid Lamparter
Done with a combination of regex'ing and banging my head against a wall. Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
2023-01-24zebra: fix JSON fields for "show ip/ipv6 nht"Pooja Jagadeesh Doijode
1. Renamed "gates" to "nexthops" 2. Displaying afi of the nexthops being dispalyed in place of "nexthops" JSON object in the old JSON output 3. Calling show_route_nexthop_helper() and show_nexthop_json_helper() instead of print_nh() inorder to keeps the fields in "nexthops" JSON object in sync with "nexthops" JSON object of "show nexthop-group rib json". Updated vtysh: r1# show ip nht 192.168.0.2 resolved via connected is directly connected, r1-eth0 (vrf default) Client list: static(fd 28) 192.168.0.4 resolved via connected is directly connected, r1-eth0 (vrf default) Client list: static(fd 28) Updated JSON: r1# show ip nht json { "default":{ "ipv4":{ "192.168.0.2":{ "nhtConnected":false, "clientList":[ { "protocol":"static", "socket":28, "protocolFiltered":"none" } ], "nexthops":[ { "flags":3, "fib":true, "directlyConnected":true, "interfaceIndex":2, "interfaceName":"r1-eth0", "vrf":"default", "active":true } ], "resolvedProtocol":"connected" } } } } Signed-off-by: Pooja Jagadeesh Doijode <pdoijode@nvidia.com>
2022-11-04tests: Test Resilient NHG's are properly created in zebraDonald Sharp
When a Resilient NHG is created, ensure that Zebra notes that it is created and has it as well. Signed-off-by: Donald Sharp <sharp@nvidia.com>
2022-11-04tests: Speedup test_all_protocol_startup.py by 55 secondsDonald Sharp
Just make ospf and ospfv3 converge faster with faster hello timers. Signed-off-by: Donald Sharp <sharpd@nvidia.com>
2022-10-25bgpd: fixed misaligned columns in BGP routes tableWayne Morrison
Column headers in BGP routes table are not aligned with data when RPKI status is available. This was fixed to insert a space at the beginning of the header and at the beginning of lines that do not have RPKI status. This fix requires that several testing templates be adjusted to match the new output. Signed-off-by: Wayne Morrison <wmorrison@netgate.com>
2021-11-07tests: Remove debugs from topotestsDonald Sharp
Debugs take up a significant amount of cpu time as well as increased disk space for storage of results. Reduce test over head by removing the debugs, Hopefully this helps alleviate some of the overloading that we are seeing in our CI systems. Signed-off-by: Donald Sharp <sharpd@nvidia.com>
2021-11-04tests: Fix route replace test in all_protocol_startupDonald Sharp
The route replace test was doing this seq of events: a) Create nhg b) Install route w/ sharpd c) Ensure it worked d) Modify nhg d) Ensure the update group replace worked The problem is that the sharp code is doing this: /* Only send via ID if nhgroup has been successfully installed */ if (nhgid && sharp_nhgroup_id_is_installed(nhgid)) { SET_FLAG(api.message, ZAPI_MESSAGE_NHG); api.nhgid = nhgid; } else { for (ALL_NEXTHOPS_PTR(nhg, nh)) { api_nh = &api.nexthops[i]; zapi_nexthop_from_nexthop(api_nh, nh); i++; } api.nexthop_num = i; } The created nhg has not been successfully installed( or at least sharpd has not read the results yet) when it gets the command to install the routes. As such it passes down the individual nexthops instead. The route replace is never going to work. Modify the code to add a bit of sleep to allow sharpd to get notified when the system is under load. At this point there is no way to query sharpd for whether or not it thinks it's nhg is installed properly or not. This test is failing all over the place for a bunch of people let's get this fixed so people can get running Signed-off-by: Donald Sharp <sharpd@nvidia.com>
2021-11-01tests: All_protocol_startup sporadic failureDonald Sharp
the test_nexthop_groups function is failing occassionally because the test executes 4 in succession sharp install routes commands. When I dumped the rib on a failed test run there were only 2 of the 4 routes in the rib and the two that were in were the last 2 installed. The sharp daemon setups a event process where it installs routes `automatically`. If the previous run is not finished entering a new command to install the routes will mess up the last one from ever happening. It is assumed that the user doesn't do stupid stuff here. In this case I am just adding a small sleep between each installation to just let the test proceed. Signed-off-by: Donald Sharp <sharpd@nvidia.com>
2021-10-22tests: all_protocol_startup needs some tweaks to allow for processingDonald Sharp
The nexthop group code is installing routes and nexthop groups and immediately expecting zebra to have processed the results as a result there is a situation when the CI system is under intense load that the nexthop group might not have been processed. Add a bit of code to allow the test to give FRR some time to finish work before declaring it not working. Signed-off-by: Donald Sharp <sharpd@nvidia.com>
2021-10-14Merge pull request #9761 from mjstapp/fix_topo_debug_cliIgor Ryzhov
tests: remove deprecated debug cli references
2021-10-08bgpd: fix crash when using "show bgp vrf all"Igor Ryzhov
Any command that uses `peer_lookup_in_view` crashes when "vrf all" is used, because bgp is NULL in this case. Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>
2021-10-06tests: clean up all_proto_startup a bitMark Stapp
Remove references to the deprecated "CLI()" function; clean up a couple of string escapes; make one test-case sensitive to previous failures. Signed-off-by: Mark Stapp <mstapp@nvidia.com>
2021-09-09tests: Fix BGP check in all_protocol_startupMartin Winter
Fix issue of topotest failures with BGP status Connect or Idle instead of the expected Active Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>
2021-09-04tests: cleanup: rerun changed files through blackChristian Hopps
Signed-off-by: Christian Hopps <chopps@labn.net>
2021-09-04tests: cleanup - remove unused importsChristian Hopps
Signed-off-by: Christian Hopps <chopps@labn.net>
2021-09-04tests: remove legacy Topo class (fixes many pylint errors)Christian Hopps
Signed-off-by: Christian Hopps <chopps@labn.net>
2021-09-04tests: micronet: adapt testsChristian Hopps
Signed-off-by: Christian Hopps <chopps@labn.net>
2021-06-08bgpd: add terse display option on show bgp summaryLouis Scalbert
Add a terse option to show bgp summary to shorten output. Do not show the following information about the BGP instances: the number of RIB entries, the table version and the used memory. The "terse" option can be used in combination with the "remote-as", "neighbor", "failed" and "established" filters, and with the "wide" option as well. Before patch: ubuntu# show bgp summary remote-as 123456 IPv4 Unicast Summary (VRF default): BGP router identifier X.X.X.X, local AS number XXX vrf-id 0 BGP table version 0 RIB entries 3, using 552 bytes of memory Peers 5, using 3635 KiB of memory Neighbor V AS MsgRcvd MsgSent TblVer InQ OutQ Up/Down State/PfxRcd PfxSnt Desc 10.200.200.2 4 123456 81432 4 0 56092 0 00:00:13 572106 0 N/A Displayed neighbors 1 Total number of neighbors 4 IPv6 Unicast Summary (VRF default): BGP router identifier X.X.X.X, local AS number XXX vrf-id 0 BGP table version 0 RIB entries 3, using 552 bytes of memory Peers 5, using 3635 KiB of memory Neighbor V AS MsgRcvd MsgSent TblVer InQ OutQ Up/Down State/PfxRcd PfxSnt Desc % No matching neighbor Total number of neighbors 5 After patch: ubuntu# show bgp summary remote-as 123456 terse IPv4 Unicast Summary (VRF default): BGP router identifier X.X.X.X, local AS number XXX vrf-id 0 Neighbor V AS MsgRcvd MsgSent TblVer InQ OutQ Up/Down State/PfxRcd PfxSnt Desc 10.200.200.2 4 123456 81432 4 0 56092 0 00:00:13 572106 0 N/A Displayed neighbors 1 Total number of neighbors 4 IPv6 Unicast Summary (VRF default): BGP router identifier X.X.X.X, local AS number XXX vrf-id 1 % No matching neighbor Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>
2021-06-08bgpd: add counter of displayed show bgp summary when filteringLouis Scalbert
When filtering sessions on show bgp summary with failed, established, neighbor and remote-as options, add a counter of displayed neighbors in addition to the total number of neighbor : "Displayed neighbors X" ubuntu# show bgp summary failed remote-as external IPv4 Unicast Summary (VRF default): Neighbor EstdCnt DropCnt ResetTime Reason 10.200.200.2 0 0 never Waiting for NHT 172.16.29.2 0 0 never Waiting for NHT 10.22.1.2 0 0 never Waiting for NHT Displayed neighbors 3 Total number of neighbors 5 Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>
2021-06-08bgpd: display pretty VRF/view name on no such neighborLouis Scalbert
Display on which VRF/view the neighbor was not found. Useful when selecting "vrf all". Before patch: No such neighbor in this view/vrf After patch: No such neighbor in VRF default Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>
2021-06-04bgpd: modify VRF/view display in show bgp summaryLouis Scalbert
Modify VRF/view display in show bgp summary: - to be more concise - to display on which VRF/view no neighbor was found Before patch: ubuntu# show bgp vrf all summary Instance default: IPv4 Unicast Summary: BGP router identifier XX.XX.XX.XX, local AS number XXXX vrf-id 0 (...) IPv6 Unicast Summary: Instance private: IPv4 Unicast Summary: ubuntu# show bgp vrf all ipv4 multicast summary % No BGP neighbors found % No BGP neighbors found After patch: ubuntu# show bgp vrf all summary IPv4 Unicast Summary (VRF default): BGP router identifier XX.XX.XX.XX, local AS number XXXX vrf-id 0 (...) IPv6 Unicast Summary (VRF default): (...) IPv4 Unicast Summary (VRF private): (...) ubuntu# show bgp vrf all ipv4 multicast summary % No BGP neighbors found in VRF default % No BGP neighbors found in VRF private Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>
2021-06-04topotests: remove uneccessary vtysh cmd requestLouis Scalbert
Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>
2021-05-26bgpd: add show bgp summary filter by neighbor or ASLouis Scalbert
Add ability to filter session on show bgp summary by neighbor or remote AS: ubuntu# show bgp summary ? neighbor Show only the specified neighbor session remote-as Show only the specified remote AS session ubuntu# show bgp summary neighbor ? A.B.C.D Neighbor to display information about WORD Neighbor on BGP configured interface X:X::X:X Neighbor to display information about ubuntu# show bgp summary remote-as ? (1-4294967295) AS number external External (eBGP) AS sessions internal Internal (iBGP) AS sessions This patch includes the documentation and the topotest. Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>
2021-05-11tests: Unify directory naming for topotestsDonatas Abraitis
Change every `-` to `_` in directory names. This is to avoid mixing _ and -. Just for consistency and directory sorting properly. Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>