summaryrefslogtreecommitdiff
path: root/tests
AgeCommit message (Collapse)Author
2019-05-16Merge pull request #4144 from pguibert6WIND/bfd_cbitRuss White
BFD CBIT
2019-05-16Merge pull request #4340 from qlyoung/hash-key-constRuss White
lib: hashing functions should take const arguments
2019-05-16Merge pull request #4316 from LabNConsulting/working/master/tt-scale-vrfsDonald Sharp
bgp_l3vpn_to_bgp_vrf: test adding/removing 50K*2CEs VRF routes
2019-05-15bgp_l3vpn_to_bgp_vrf: test adding/removing 50K*2CEs VRF routesLou Berger
Only runs when configured with --enable-sharpd Signed-off-by: Lou Berger <lberger@labn.net>
2019-05-15bgp_l3vpn_to_bgp_vrf: disable bgp debug loggingLou Berger
Signed-off-by: Lou Berger <lberger@labn.net>
2019-05-15topotests/lib: support sharpd and other daemons in lutilsLou Berger
needs both <daemon>.conf and <daemon> to be present Signed-off-by: Lou Berger <lberger@labn.net>
2019-05-14lib: hashing functions should take const argumentsQuentin Young
It doesn't make much sense for a hash function to modify its argument, so const the hash input. BGP does it in a couple places, those cast away the const. Not great but not any worse than it was. Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
2019-05-14bfd-bgp-cbit-topo3: add bfd bgp cbit topotest testPhilippe Guibert
this test simulates the case where r1 keeps staled entries of r2, because r2 is a 'non stop forwarding' device. For instance, r2 restarts to be upgraded, then r1 keeps the entries, thus avoiding to flush entries in the dataplane. Here, BFD is used, and the bgp check-control-plane-failure command is used to inform BGP that if BFD goes down, this may be because remote peer is restarting, and this may be necessary to keep the entries of remote. Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
2019-05-14Merge pull request #4112 from pguibert6WIND/bfd_vrfRafael Zalamena
support for BFD VRF
2019-05-10Merge pull request #4275 from ton31337/fix/move_outside_bgp_input_modifierDonald Sharp
bgpd: Move inbound policy check outside bgp_input_modifier()
2019-05-10Merge pull request #4272 from opensourcerouting/isis-prefix-sid-fixDonald Sharp
isisd: fix display of the Extended IPv4 reachability TLV
2019-05-10topotests: Add test to check if eBGP required policy worksDonatas Abraitis
Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
2019-05-10isisd: Don't set subtlv structure if we didn't unpack any subtlvsChristian Franke
This ensures deserialized and serialized TLV representation is consistent.
2019-05-08topotests: ifindex values are not guaranteed to be the sameDonald Sharp
Different versions of linux are assigned different interface ifindexes. Mask that cabbage out. Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
2019-05-07bfd-vrf_topo1: add bfd test on a vrf netnsPhilippe Guibert
some testing is available on testing bfd feature based on a bgp vrf instance. Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
2019-05-02Merge pull request #4247 from mjstapp/fix_topotest_topo_onlyDonald Sharp
topotest: fix pytest deprecation warning
2019-05-02topotest: fix pytest deprecation warningMark Stapp
As of pytest 4.something, a pattern we were using in conftest.py was deprecated. Also make a new-ish test script executable (all the rest appear to be?) Signed-off-by: Mark Stapp <mjs@voltanet.io>
2019-05-01topotests/bgp_rfapi_basic_sanity: cleanup rfapi using non-debug commandLou Berger
Signed-off-by: Lou Berger <lberger@labn.net>
2019-04-30Merge pull request #4232 from mjstapp/fix_topotests_py3Donald Sharp
topotest: fixes to support python3
2019-04-30topotest: fixes to support python3Mark Stapp
Make some small changes to support both python 2 and 3. Signed-off-by: Mark Stapp <mjs@voltanet.io>
2019-04-30Merge pull request #3045 from opensourcerouting/atomsLou Berger
READY: lists/skiplists/rb-trees new API & sequence lock & atomic lists
2019-04-29Revert "lib: remove pqueue_*"David Lamparter
This reverts commit 798ac49d06b6619adb4c5ac765b092397bc50a6c.
2019-04-27lib: remove fifo implementationDavid Lamparter
2019-04-27isisd: replace dict_* with DECLARE_RBTREEDavid Lamparter
Historically, isisd has been carrying around its own red-black tree to manage its LSP DB in. This replaces that with the newly-added DECLARE_RBTREE_*. This allows completely removing the dict_* code. Signed-off-by: David Lamparter <equinox@diac24.net>
2019-04-27lib: remove pqueue_*David Lamparter
All users of the pqueue_* implementations have been migrated to use TYPEDSKIP_* skiplists. Remove. Signed-off-by: David Lamparter <equinox@diac24.net>
2019-04-27tests: exercise the typesafe list wrappersDavid Lamparter
Since all of these list implementations provide almost the same API, we can run and validate them against the same test code. 9 tests for the price of one! Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
2019-04-27lib: atomlist & atomsortDavid Lamparter
These two are lock-free linked list implementations, the plain one is primarily intended for queues while the sorted one is for general data storage. Signed-off-by: David Lamparter <equinox@diac24.net>
2019-04-25Merge pull request #4140 from ↵Russ White
ton31337/fix/do_not_send_notification_again_with_invalid_nlri bgpd: Do not send UPDATE message with maximum-prefix
2019-04-24bgpd: Do not send UPDATE message with maximum-prefixDonatas Abraitis
When using maximum-prefix and count is overflow BGP sends UPDATE message: Apr 15 20:45:06 exit1-debian-9 bgpd[9818]: 192.168.0.2 [Error] Error parsing NLRI Apr 15 20:45:06 exit1-debian-9 bgpd[9818]: %NOTIFICATION: sent to neighbor 192.168.0.2 3/10 (UPDATE Message Error/Invalid Network Field) 0 bytes Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
2019-04-18tests: bgp_l3vpn_to_bgp_vrf were bailing to quicklyDonald Sharp
The tests are not coming up consistently on my test box. Add a bit of wait time to test to allow normal bgp when the first attempt doesn't come up. Especially since bgp timeouts are 120 seconds with non datacenter compiles. Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
2019-04-18lib: add "seqlock" wait/broadcast primitiveDavid Lamparter
Manually tested rather extensively in addition to included unit tests, should work as intended. NB: The OpenBSD futex() code is "future"; it's not actually in OpenBSD (yet?) and thus untested. Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
2019-04-09Merge pull request #3779 from opensourcerouting/fix-mpls-label-topotestsQuentin Young
tests: Remove some inconsistent local labels in check in test_ldp_vpls_topo1
2019-04-04tests: test_ldp_vpls_topo1 Remove some inconsistent labels in checkMartin Winter
Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>
2019-04-04Merge pull request #3899 from ton31337/fix/remove_private_as_with_local_asLou Berger
bgpd: Remove private AS numbers if local-as is defined
2019-04-03lib: remove event_counter.[ch]Quentin Young
goodbye spooky code Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
2019-03-28Merge pull request #4015 from opensourcerouting/topotests-fixesMark Stapp
topotests: fix missing log file and duplicated output folder
2019-03-27topotests: Add a bit more useful debugging to ldp-vpls-topo1Donald Sharp
When something goes wrong with this test, having these turned on will help us debug zebra processing a bit better in the future without having to modify anything else. Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
2019-03-26bgpd: Remove private AS numbers if local-as is definedDonatas Abraitis
When using remove-private-AS together with local-as aspath_remove_private_asns() is called before bgp_packet_attribute(). In this case, private AS will always appear in front of change_local_as. Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
2019-03-25topotests/lib: fix router specific log outputRafael Zalamena
Change the router log output to the previous folder so it doesn't get erased when starting the old API (unbreaks command/output logging on Topogen). Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
2019-03-25topotests/lib: fallback topology logdir if none was configuredRafael Zalamena
`param.get` always evaluates the second argument and it was causing two log directories being created for topologies using Topogen. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
2019-03-20topotests: pim-basic: fix some rough edgesRafael Zalamena
Move daemon configuration to appropriated files and use `json_cmp` to assert the values we expect. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
2019-03-20Add code to test some very basic pim functionalityDonald Sharp
Add code to send a S,G stream and make sure the RP see's it. Add code to send a *,G report and make sure the RP see's it. This is just some *very* basic functionality testing to ensure that we don't break anything basic. Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
2019-03-19tests: point topotests docker img to new locationQuentin Young
The topotests docker image has moved from frrouting/frr to frrouting/topotests. Update accordingly. Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
2019-03-15Merge pull request #3920 from AkhileshSamineni/show_bgp_ipv6_summary_fix_masterDonald Sharp
bgpd: Incorrect number of peers count in "show bgp ipv6 summary" output
2019-03-11topotests: add new bfd topology with IPv6Rafael Zalamena
New BFD topology using IPv6 and multi hop peer to cover more daemon features. This topology also tests BFD integration with BGP, OSPF and OSPF6. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
2019-03-07bgpd: Incorrect number of peers count in "show bgp ipv6 summary outputAkhilesh Samineni
The "show bgp ipv6 summary" output displays incorrect number of peers count. sonic# show bgp ipv6 summary IPv6 Unicast Summary: BGP router identifier 10.1.0.1, local AS number 65100 vrf-id 0 BGP table version 0 RIB entries 0, using 0 bytes of memory Peers 5, using 103 KiB of memory Peer groups 1, using 64 bytes of memory Neighbor V AS MsgRcvd MsgSent TblVer InQ OutQ Up/Down State/PfxRcd 2003::1 4 65099 0 0 0 0 0 never Active 2088::1 4 65100 0 0 0 0 0 never Active 3021::2 4 65100 0 0 0 0 0 never Active Total number of neighbors 3 sonic# In the above output, the peers count displays as 5 but the actual peer count is 3, i.e.. 3 neighbors are activated in ipv6 unicast address family. Displayed peer count (5) is the number of the neighbors activated in a BGP instance. Fix : Now the peers count displays the number of neighbors activated per afi/safi. After Fix: sonic# show bgp ipv6 summary IPv6 Unicast Summary: BGP router identifier 10.1.0.1, local AS number 65100 vrf-id 0 BGP table version 0 RIB entries 0, using 0 bytes of memory Peers 3, using 62 KiB of memory Peer groups 1, using 64 bytes of memory Neighbor V AS MsgRcvd MsgSent TblVer InQ OutQ Up/Down State/PfxRcd 2003::1 4 65099 0 0 0 0 0 never Active 2088::1 4 65100 0 0 0 0 0 never Active 3021::2 4 65100 0 0 0 0 0 never Active Total number of neighbors 3 sonic# Signed-off-by: Akhilesh Samineni <akhilesh.samineni@broadcom.com>
2019-02-25*: use array_size instead of raw divisionQuentin Young
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
2019-02-25*: return bool from boolean functionsQuentin Young
Not 1 or 0. Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
2019-02-25*: Rename backet to bucketTim Bray
Presume typo from original author Signed-off-by: Tim Bray <tim@kooky.org>
2019-02-18Merge pull request #3777 from donaldsharp/topotest_all_routesDavid Lamparter
topotests: Add code to ensure routes are as expected