| Age | Commit message (Collapse) | Author |
|---|
|
|
|
Add a northbound unit-test binary product to .gitignore
Signed-off-by: Mark Stapp <mjs@cisco.com>
|
|
Use the expected - little-endian - byte-order for a param
to one of the libyang apis; tests fail on LE architectures
otherwise.
Signed-off-by: Mark Stapp <mjs@cisco.com>
|
|
Introducing do_ipv6_nexthop to static_simple topotest. The test
configures IPv4 routes with IPv6 nexthop and validates that via
inet6 is visible in the Linux Kernel
Signed-off-by: Christopher Dziomba <christopher.dziomba@telekom.de>
|
|
Looking at the gcov of the rpki code, I noticed
that there was some functionality that is not
covered in our test suites. Add the functionality.
Signed-off-by: Donald Sharp <donaldsharp72@gmail.com>
|
|
tests: Shadowing the built-in function
|
|
tests: Fix potential issues in mcast-tester.py
|
|
Fix potential issues in mcast-tester.py
Signed-off-by: y-bharath14 <y.bharath@samsung.com>
|
|
This fix add tests to verify routes/path are getting
deleted properly, when the advertising neighbor is shutdown
Signed-off-by: Soumya Roy <souroy@nvidia.com>
|
|
Clean up -Wshadow warnings in unit-tests
Signed-off-by: Mark Stapp <mjs@cisco.com>
|
|
Currently, when the locator block and sid block differs, staticd would
still go ahead and request zebra to allocate the SID which it does if
there is atleast one match (from any locators).
Only when staticd tries to install the route, it sees that the locator
block and sid block are different and avoids installing the route.
Fix:
Check if the locator block and sid block match before even requesting
Zebra to allocate one.
Signed-off-by: Rajasekar Raja <rajasekarr@nvidia.com>
|
|
Shadowing the built-in function
Signed-off-by: y-bharath14 <y.bharath@samsung.com>
|
|
opensourcerouting/fix/syntax_error_bgp_gr_notification
tests: Fix typo when configuring delayopen timer
|
|
bgpd: rfapi: track outstanding rib and import timers, free mem at exit
|
|
Add BGP redistribution in SRv6 BGP
|
|
`"` was accidentally added, and random tests failures happening.
Fixes: a4f61b78dd382c438ff4fec2fda7450ecc890edf ("tests: Check if routes are marked as stale and retained with N-bit for GR")
Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
|
|
tests: Resource leaks in test_all_protocol_startup
|
|
Bgp packet reads conversion to a FIFO
|
|
Fix resource leaks in test_all_protocol_startup.py
Signed-off-by: y-bharath14 <y.bharath@samsung.com>
|
|
While here, also make "VPN SAFI clear" test wait for clear result
(tests/topotests/bgp_rfapi_basic_sanity{,_config2})
Original RFAPI code relied on the frr timer system to remember
various allocations that were supposed to be freed at future times
rather than manage a parallel database. However, if bgpd is terminated
before the times expire, those pending allocations are marked as
memory leaks, even though they wouldn't be leaks under normal operation.
This change adds some hash tables to track these outstanding
allocations that are associated with pending timers, and uses
those tables to free the allocations when bgpd exits.
Signed-off-by: G. Paul Ziemba <paulz@labn.net>
|
|
Irrelevant code in lutil.py
Signed-off-by: y-bharath14 <y.bharath@samsung.com>
|
|
Fix potential issues at send_bsr_packet.py
Signed-off-by: y-bharath14 <y.bharath@samsung.com>
|
|
bgpd: Retain the routes if we do a clear with N-bit set for Graceful-Restart
|
|
tests: Modify simple_snmp_test to use frr.conf
|
|
bgpd: Set the label for MP_UNREACH_NLRI 0x800000 instead of 0x000000
|
|
The simple_snmp_test was not properly testing
the rip snmp code because of weirdness w/ mgmtd
and non-integrated configs. Modify the whole
test to use a integrated config and voila
ripd is talking snmp again in the test.
Signed-off-by: Donald Sharp <sharpd@nvidia.com>
|
|
tests: Add ripng aggregate address testing
|
|
Related-to: 94e2aadf7187d7d695babce21033b5bc8e454f25 ("bgpd: Set the label for MP_UNREACH_NLRI 0x800000 instead of 0x000000")
Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
|
|
Increase wait times to at least the minimum wait time accepted by
topotest.run_and_expect(). Also change poll interval to 1s, no point in
doings this more frequently.
Finally, slightly improve the topology diagram to also include area numbers.
Signed-off-by: Martin Buck <mb-tmp-tvguho.pbz@gromit.dyndns.org>
|
|
tests: Handling potential errors gracefully
|
|
Looking at gcov and noticed that ripngd does not
test any aggregate address addition/deletion
to ensure that it works.
Signed-off-by: Donald Sharp <sharpd@nvidia.com>
|
|
Related-to: b7c657d4e065f310fcf6454abae1a963c208c3b8 ("bgpd: Retain the routes if we do a clear with N-bit set for Graceful-Restart")
Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
|
|
Many useless commande are still persistent in the
bgp_srv6l3vpn_to_bgp_vrf tests. Remove the useless commands.
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
|
|
The hold timer is 5/20. At load with a very very
large number of routes, the tests are experiencing
some issues with this. Let's just give ourselves
some headroom associated with the receiving
of packets
Signed-off-by: Donald Sharp <sharpd@nvidia.com>
|
|
Bgp clear batch
|
|
Add more control on the expected outputs, by using an exact json
comparison.
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
|
|
Handling potential errors gracefully at exa-receive.py
Signed-off-by: y-bharath14 <y.bharath@samsung.com>
|
|
staticd: Fix a crash that occurs when modifying an SRv6 SID
|
|
When srv6 is disabled due to misconfiguration, exported VPN prefixes
are invalidated, except for the ones that have their nexthop modified
with the 'nexthop vpn export' command. The previous commit also
invalidates those vpn prefixes.
Apply the changes to the test by not considering some prefixes as
selected. Enforce the expected route count.
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
|
|
This test ensures route redistribution across an srv6 VPN network
is well taken into account.
Signed-off-by: Jonathan Voss <jvoss@onvox.net>
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
|
|
Add a BGP update in CE1 for redistribution. The expectation is that this
BGP update will be leaked to the L3VPN. Reversely, if the locator is
unset, the L3VPN prefix will be invalidated.
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
|
|
Use experimental AS values to play the test.
Add BGP peering on CEs, and use the default-originate functionality on
each PE facing CPEs.
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
|
|
The aspath value has no need to be controlled. Unset the bgp capability
to send aspath information to zebra.
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
|
|
Use the unified configuration for bgp_srv6l3vpn_to_bgp_vrf test.
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
|
|
The high_ecmp test was creating 2 update groups, where
513 of the neighbors are in 1 and the remaining is in
another. They should just all be in 1 update group.
Modify the test creation such that interfaces r1-eth514
and r2-eth514 have v4 and v6 addresses.
Signed-off-by: Donald Sharp <donaldsharp72@gmail.com>
|
|
This commit adds a test case that modifies a SID and verifies that the
RIB is as expected.
Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>
|
|
bgpd: fix `set evpn gateway-ip ipv[46]` route-map
|
|
tests: Corrected typo at path_attributes.py
|
|
tests: add bfd_static_vrf
|
|
Currently the tests appear to do this:
a) Start the neighbors
b) Start the bmp server connection
c) Look for the neighbors up
d) Look for the neighbor up messages in the bmp log
This is not great from a testing perspective in that
even though we started a) first it may not happen
until after b) happens. Or even worse if it is
partially up ( 1 of the 2 peers ) then the dump
will have the neighbor connecting after parts
of the table. This doesn't work too well because
the SEQ number is something that is kept and compared
to to make sure only new data is being looked at.
Let's modify the startup configuration to start
the bmp server first and then have a delayopen
on the bgp neighbor statements so that the bmp
peering can come up first.
Signed-off-by: Donald Sharp <sharpd@nvidia.com>
|
