summaryrefslogtreecommitdiffstats
path: root/tests (follow)
Commit message (Collapse)AuthorAgeFilesLines
* tests: fix check for nhid in ipv6 table outputMartin Winter2020-03-201-6/+6
| | | | | | | Check based on actual output instead of assuming output based on kernel version Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>
* tests: Make topotest working on different localeMartin Winter2020-03-201-2/+2
| | | | | | | | "sort" as used in all-protocol-startup used sort which causes different sort order based on locale settings. Specify the correct one to make output matching our expected result Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>
* Merge pull request #5755 from slankdev/bgpd-fix-prefix-sid-parse-errorQuentin Young2020-03-199-7/+421
|\ | | | | bgpd: fix Prefix-SID parse error
| * tests: refactor Prefix-SID binary syntaxHiroki Shirokura2020-02-141-6/+13
| | | | | | | | Signed-off-by: Hiroki Shirokura <slank.dev@gmail.com>
| * bgpd: refactor func prototype arround Prefix-SIDHiroki Shirokura2020-02-141-1/+1
| | | | | | | | | | | | mp_update value isn't used by the function arround Prefix-SID. Signed-off-by: Hiroki Shirokura <slank.dev@gmail.com>
| * topotests: add bgp_prefix_sidHiroki Shirokura2020-02-148-0/+407
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit add behavior test for BGP Prefix-SID path attribute generically. In this time, there are only 1-test for Prefix-SID type-1 Label-Index TLV. There are 3 nodes r1(FRR), peer1(exabgp) and peer2(exabgp) on this topotest. And it perform following: * peer1 advertise Prefix-SID to r1 * r1 is received Prefix-SID from peer1 * bgpd on r1 check the path attribute and parse correctly. * user can check information from type-1 information via show cli * bgpd on r1 advertise Prefix-SID to peer2 * peer2 is received Prefix-SID from r1 * peer2 check the path attribute and parse correctly. This test uses exabgp's generic path attribute feature of exabgp is used to advertise Prefix-SID path attribute to bgpd. generic path attribute feature enable exabgp users to specify binary format path attribute. we can send valious binary pattern (but overflow test doesn't can be performed). The reason why this commit uses generic attribute feature is that exabgp v3 doesn't support Prefix-SID path attribute and topotest support exabgp only v3. (fyr. exabgp v4 supports it). Thus this test includes little complicated binary format, so I wrote full binary desection and explanation. If topotest support exabgp v4, this test should be rewrite with non generic attribute feature. Signed-off-by: Hiroki Shirokura <slank.dev@gmail.com>
* | tests: Increase some wait time in testsDonald Sharp2020-03-191-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The bfd-bgp-cbit-topo3 test is testing bfd timers with some timers that only wait 4 seconds. The CI system is failing in various places due to bfd not converging properly. Upon logging into a CI system and running tests with intensive disk i/o I was able to make the tests fail repeatedly in a couple of different places. Add some additional time to allow the system to converge on our CI systems that are running in vm's and may not always have complete control of cpu's. Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
* | Merge pull request #5849 from donaldsharp/pim_register_prefix_listRuss White2020-03-176-1/+31
|\ \ | | | | | | Pim register prefix list
| * | tests: Add a test for the `ip pim register-accept-list PLIST` commandDonald Sharp2020-03-066-1/+31
| | | | | | | | | | | | | | | | | | | | | Add a test to the pim-basic to ensure that the prefix-list works as expected. Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
* | | Merge pull request #5954 from ton31337/feature/rfc7607Russ White2020-03-172-0/+31
|\ \ \ | | | | | | | | bgpd: Proscribe the use of AS 0 (zero)
| * | | tests: Add a test case for BGP_AS_ZERODonatas Abraitis2020-03-102-0/+31
| | | | | | | | | | | | | | | | Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
* | | | Merge pull request #6009 from kuldeepkash/bgp_basic_functionalityDonald Sharp2020-03-172-6/+7
|\ \ \ \ | | | | | | | | | | tests: Optimize bgp-basic-functionality-topo1 test suite
| * | | | tests: Optimize bgp-basic-functionality-topo1 test suiteKuldeep Kashyap2020-03-162-6/+7
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | 1. Used aggresive values to verify keepalive and holddown timers functionality 2. Modified variable name in lib/bgp.py Signed-off-by: Kuldeep Kashyap <kashyapk@vmware.com>
* / / / tests/isisd: Update expected output to match corrected spellingChristian Franke2020-03-161-0/+0
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The update was performed using wuschl [1] like this: $ wuschl rebuild tests/isisd/test_fuzz_isis_tlv $ gzip -9 tests/isisd/test_fuzz_isis_tlv_tests.h [1] https://pypi.org/project/wuschl/ Signed-off-by: Christian Franke <chris@opensourcerouting.org>
* / / tests: fix build with GCC 10Ruben Kerkhof2020-03-081-1/+1
|/ / | | | | | | Signed-off-by: Ruben Kerkhof <ruben@rubenkerkhof.com>
* | *: Replace __PRETTY_FUNCTION__/__FUNCTION__ to __func__Donatas Abraitis2020-03-051-2/+2
| | | | | | | | | | | | Just keep the code cool. Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
* | Merge pull request #5855 from ↵Sri Mohana Singamsetty2020-03-028-0/+194
|\ \ | | | | | | | | | | | | ton31337/fix/allow_using_add_sub_for_local-preference bgpd: Allow using add/subtract for local-preference in route-maps
| * | tests: Add a test case for `set local-preference +/-LOCAL_PREF`Donatas Abraitis2020-02-288-0/+194
| | | | | | | | | | | | Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
* | | tests: Modified scripts to optimize execution time for route-map test suiteKuldeep Kashyap2020-02-263-637/+652
| | | | | | | | | | | | | | | | | | | | | 1. Modified scripts to optimize execution time for route-map test suite 2. Removed unnecessary print statements from /lib/bgp.py Signed-off-by: Kuldeep Kashyap <kashyapk@vmware.com>
* | | tests: Reduce rip[ng]-topo1 run timeDonald Sharp2020-02-1914-18/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | Reduce rip topo1 run time from ~100 seconds down to ~45 seconds. 1) Reduce very very long sleeps 2) Modify rip timers to be more aggressive in sending data. Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
* | | tests: bgp_multiview_topo1 was taking too longDonald Sharp2020-02-191-7/+5
|/ / | | | | | | | | | | | | | | | | | | | | | | The bgp_multiview_topo1 test had two issues a) An abbundance of sleeps that are not necessary b) The convergence time was too short. The upshoot of this is that the test goes from 72 -> 33 seconds run time on my machine. Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
* | Merge pull request #5802 from ton31337/feature/aggregate-address_origin_overrideDonald Sharp2020-02-196-0/+154
|\ \ | | | | | | bgpd: Allow overriding ORIGIN for aggregate-address
| * | tests: Add test case for `aggregate-address <prefix> origin <origin>`Donatas Abraitis2020-02-136-0/+154
| |/ | | | | | | Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
* | Merge pull request #5651 from AnuradhaKaruppiah/evpn-pim-fixesPhilippe Guibert2020-02-1818-0/+402
|\ \ | | | | | | EVPN-PIM: complete anycast (MLAG) VTEP support
| * | tests: Add a evpn-pim test caseAnuradha Karuppiah2020-02-1418-0/+402
| |/ | | | | | | | | | | | | | | | | This is a basic setup and test of evpn-pim. Create a vxlan device ensure that pim notices this and setups the appropriate groups and sends them to the RP. Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
* / bgpd: Add a couple more spaces for output on MsgRcvd and MsgSentDonald Sharp2020-02-172-8/+8
|/ | | | | | | | | | | | | | | | | | | | | annie# show bgp ipv4 uni summ BGP router identifier 192.168.201.136, local AS number 64539 vrf-id 0 BGP table version 22458946 RIB entries 1458006, using 178 MiB of memory Peers 4, using 68 KiB of memory Neighbor V AS MsgRcvd MsgSent TblVer InQ OutQ Up/Down State/PfxRcd 45.33.5.119 4 0 0 0 0 0 0 never Active 65.19.134.122 4 15096 4611832 108292 0 0 0 6d22h55m 800670 107.13.46.23 4 0 0 0 0 0 0 never Connect robot(192.168.201.139) 4 64540 11159975 11365599 0 0 0 05w2d05h Connect Total number of neighbors 4 On very busy systems The column output for MsgRcvd and MsgSent can quickly move past 7 columns. Add a couple more to allow for even display. Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
* Merge pull request #5744 from ton31337/fix/thread-as-withdraw_attributesRuss White2020-02-111-1/+1
|\ | | | | bgpd: Update some attributes how they are handled if malformed
| * bgpd: Update some attributes how they are handled if malformedDonatas Abraitis2020-02-051-1/+1
| | | | | | | | | | | | | | According to https://tools.ietf.org/html/rfc7606 some of the attributes MUST be handled as "treat-as-withdraw" approach. Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
* | Merge pull request #5776 from volta-networks/fix_ldp_topol_testRenato Westphal2020-02-101-14/+14
|\ \ | | | | | | ldpd: Fix LDP topotest to validate MPLS LDP CLI output
| * | tests: Fix LDP topotest to validate MPLS LDP CLI outputKaren Schoener2020-02-101-14/+14
| | | | | | | | | | | | | | | | | | Fixing existing ldp-topol test to validate MPLS LDP CLI output. Signed-off-by: Karen Schoener <karen@voltanet.io>
* | | topotest: bgp_l3vpn_to_bgp_vrf - report unexpected BGP notificationsLou Berger2020-02-082-0/+45
| | | | | | | | | | | | Signed-off-by: Lou Berger <lberger@labn.net>
* | | Merge pull request #5722 from donaldsharp/kernel_routesRuss White2020-02-065-0/+296
|\ \ \ | | | | | | | | Kernel routes
| * | | tests: Test some basic kernel <-> zebra interactionsDonald Sharp2020-01-245-0/+296
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Install some kernel routes with different admin distances ensuring that they are installed correctly and characterized properly in the rib. Then add a static route to override the kernel and then remove it again to ensure that we account for it properly still. Signed-off-by: Donald Sharp <sharpd@cumulusnetorks.com>
* | | | Merge pull request #5761 from qlyoung/fix-bgp-gr-cruftDonatas Abraitis2020-02-061-1/+1
|\ \ \ \ | |_|_|/ |/| | | Fix bgp gr style
| * | | bgpd, lib: fix style from BGP GR codeQuentin Young2020-02-041-1/+1
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch fixes the noncompliant style for the following commit range: 4a6e80fbf 2ba1fe695 efcb2ebbb 8c48b3b69 dc95985fe 0f0444fbd 85ef4179a eb451ee58 2d3dd828d 9e3b51a7f d6e3c15b6 34aa74486 6102cb7fe d7b3cda6f 2bb5d39b1 5f9c1aa29 5cce3f054 3a75afa4b f009ff269 cfd47646b 2986cac29 055679e91 034e185dc 794b37d52 b0965c44e 949b0f24f 63696f1d8 Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
* / | *: don't null after XFREE; XFREE does this itselfQuentin Young2020-02-031-1/+0
|/ / | | | | | | Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
* | Merge pull request #5210 from bisdhdh/masterRuss White2020-01-281-1/+19
|\ \ | |/ |/| bgpd:BGP Graceful Restart Per Neighbor(BGPN) Feature.
| * bgpd: Adding Selection Deferral Timer handler changes.bisdhdh2020-01-231-1/+19
| | | | | | | | | | | | | | | | | | | | * Selection Deferral Timer for Graceful Restart. * Added selection deferral timer handling function. * Route marking as selection defer when update message is received. * Staggered processing of routes which are pending best selection. * Fix for multi-path test case. Signed-off-by: Biswajit Sadhu <sadhub@vmware.com>
* | tests: Test ospf unnumbered behavior and ensure that it worksDonald Sharp2020-01-229-0/+411
|/ | | | | | | | | | | Recent commit: 0eb97b860dc94329cf9add9f8f3d3a2c7f539568 Broke onlink behavior and as a result ospf unnumbered failed to work. This commit adds a small test to create 2 ospf routers, connect them through ospf unlinked behavior and then ensure that the routes are installed into the kernel as expected. Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
* topotest: log results summary at end of lutil runLou Berger2020-01-212-0/+9
| | | | Signed-off-by: Lou Berger <lberger@labn.net>
* topotest: bgp_l3vpn_to_bgp_vrf - catch case of sharpd not being compiledLou Berger2020-01-211-5/+6
| | | | Signed-off-by: Lou Berger <lberger@labn.net>
* Merge pull request #5707 from donaldsharp/nexthop_encodingDonatas Abraitis2020-01-189-3/+112
|\ | | | | Nexthop encoding
| * tests: Add another router to the basic pim testsDonald Sharp2020-01-189-3/+112
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add an additional router to the basic pim tests. 1) This test will add a link between r1 and a new rp 2) This test will ensure that r1 and rp have the expected who is the rp. 3) This test will ensure that the rp has received the upstream data for the multicast stream that is started. Ostensibly commit 68a02e06e5f103048d947262c08c569056f74d1c is the first bad commit commit 68a02e06e5f103048d947262c08c569056f74d1c Author: Mark Stapp <mjs@voltanet.io> Date: Wed Nov 13 16:06:06 2019 -0500 *: revise zapi nexthop encoding Use a per-nexthop flag to indicate the presence of labels; add some utility zapi encode/decode apis for nexthops; use the zapi apis more consistently. Signed-off-by: Mark Stapp <mjs@voltanet.io> Sparked this commit in that it broke nexthop reporting to upper level protocols. Ensure that this expectation stays working in the future. Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
* | tests: Add test for `neighbor <X.X.X.X> maximum-prefix-out <Y>`Donatas Abraitis2020-01-176-0/+142
|/ | | | Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
* tests: add basic nexthop group functionality testStephen Worley2020-01-153-0/+38
| | | | | | | | | | Add a very basic nexthop group functionality test. This test creates a 2-way ecmp group and installs a route with it using sharpd. Then we check to see that the nexthop groups are marked valid/installed in zebra. Signed-off-by: Stephen Worley <sworley@cumulusnetworks.com>
* tests: ospf6-topo1 did not work properly with nhg's and 5.3 kernelsDonald Sharp2020-01-102-2/+16
| | | | | | | Fix the ospf6-topo1 test to understand nhg's and what happens when they are installed into a kernel that can support them. Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
* Merge pull request #5410 from ↵Donald Sharp2020-01-096-0/+161
|\ | | | | | | | | ton31337/feature/bgp_default-route_with_route-map_set bgpd: An ability to set attributes for default-originate via route-map
| * tests: Add a test for 'neighbor <neighbor> default-originate route-map <rm>'Donatas Abraitis2020-01-076-0/+161
| | | | | | | | Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
* | bgpd: fix missing bounds checks for psid attrQuentin Young2020-01-031-1/+1
|/ | | | | | | | | | | Guess what - for a bounds check to work, it has to happen *before* you read the data. We were trusting the attribute field received in a prefix SID attribute and then checking if it was correct afterwards, but if was wrong we'd crash before that. This fixes the problem, and adds additional paranoid bounds checks. Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
* tests: Test if eBGP policy is not applied to iBGP peersDonatas Abraitis2019-12-195-22/+75
| | | | Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
generated by cgit v1.2.3 (git 2.39.1) at 2024-12-04 23:47:52 +0100