topotests: test MSDP shutdown command

New test step to check that MSDP shutdown command is working.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

pimd: implement MSDP shutdown command

Allow MSDP protocol to be disabled.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

Merge pull request #17477 from donaldsharp/topo_connected_installed

tests: Ensure connected routes are installed before continuing

Merge pull request #17473 from donaldsharp/bgp_bmp_and_its_ilk

Bgp bfd and its ilk

Merge pull request #17340 from nabahr/mapping-agent

PIMD: Implement AutoRP mapping-agent

Merge pull request #17469 from opensourcerouting/msdp-logs

pimd: MSDP logging improvements

Merge pull request #17476 from donaldsharp/support_bundle_isis

Support bundle isis

Merge pull request #17436 from LabNConsulting/aceelindem/ospf-multi-instance-default

ospfd: OSPF multi-instance default origination fixes

tests: Ensure connected routes are installed before continuing

Upon high load the ospf_instance_redistribute test can attempt
to install routes with sharpd before the connected routes have
fully baked themselves into zebra.  Since sharpd intentionally
has no retry mechanism we need to ensure that the test is waiting
a small bit.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

tools: Add pim msdp show commands to support bundle

The support bundle was not gathering any msdp data
for pim at all.  Let's add a bit to allow us to
have more data here when a suppport bundle is generated

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

tools: Add zebra label show commands to support bundle

There were no zebra show commands related to mpls labels
being generated for support bundles.  Let's rectify that
situation.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

tools: Add isis command generation to support bundle

When a support bundle is created no isis commands were
issued to attempt to gather data about what is going
on in isis.  Let's rectify that

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

tools: Add bmp command to support bundle generation

Add a bit of bmp data gathering for bgp support bundle generation

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

topotests: test new log toggle

Test MSDP new log toggle: the MSDP peer connection state and SA events
should be logged.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

pimd,yang: log MSDP SA events

Add new command to log all SA events.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

pimd,yang: log MSDP neighbor events

Move MSDP neighbor events global debug to per PIM instance.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

Merge pull request #17468 from opensourcerouting/msdp-tweaks

pimd: two small improvements

bgpd: Allow bfd to work if peer known but interface address not yet

If bgp is coming up and bgp has not received the interface address yet
but bgp has knowledge about a bfd peering, allow it to set the peering
data appropriately.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

tests: zebra_fec_nexthop_resolution improve

a) timers are really large preventing convergence in 30 seconds
b) The same configuration does not need to be initiated 60 times
when things are not working properly.  Once is enough.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

bgpd: bgp_connect should return an `enum connect_result`

This function when it is run by bgp_start is expected
to return a `enum connect_result`.  But instead
the function returns a variety of values that are
not really being checked for.  Consolidate to a correct
choice.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

Merge pull request #17464 from mjstapp/fix_bfdd_dplane_client_conn

bfdd: retain remote dplane client socket

pimd: constify peer lookup function

Make parameter `const` to document that the variable will not change.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

pimd: remove temporary variables from MSDP peer

Remove from MSDP peer data structure two temporary variables
that should only be used when calling library functions.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

tests: PIM AutoRP tests expanded

Now with a full AutoRP implementation, we can test AutoRP in a full network setup
beginning with candidate RP announcements all the way through discovery and active RP
selection.

Signed-off-by: Nathan Bahr <nbahr@atcorp.com>

pimd: Implement autorp mapping agent

Fully flushed out the AutoRP implementation now with the AutoRP mapping agent.
This touched most of AutoRP in order to have common reuse of containers for each
section of AutoRP operation (Candidate RP announcement, Mapping agent, Discovery).
Many debugs had guards added and many more debug logs added.

Signed-off-by: Nathan Bahr <nbahr@atcorp.com>

pimd: Implement autorp mapping agent CLI

Also exposes BSR cand_addrsel methods for use in AutoRP.

Signed-off-by: Nathan Bahr <nbahr@atcorp.com>

pimd,yang: Add PIMD northbound/yang for AutoRP mapping agent

Reuses the candidate selection logic from BSR configuration

Signed-off-by: Nathan Bahr <nbahr@atcorp.com>

bfdd: retain remote dplane client socket

When using bfd in remote-dataplane client mode, don't close
a new client socket if we're going to try to use it.

Signed-off-by: Mark Stapp <mjs@cisco.com>

Merge pull request #17156 from opensourcerouting/eradicate-strncpy

*: remove remaining `strncpy()` users

Merge pull request #17414 from dmytroshytyi-6WIND/srv6_isisd_fix_display_algorithm

isisd: properly display srv6 algorithm

Merge pull request #17453 from opensourcerouting/fix/raw_python_clippy

tools: Fix syntax raw parsing for make-foobar helper

Merge pull request #17455 from opensourcerouting/erp-cs

pim6d: fix coverity scan warning

Merge pull request #17456 from opensourcerouting/fix/lua_unit_test

Lua casting

Merge pull request #17457 from opensourcerouting/fix/optimize_community_alias_lookup

bgpd: Optimize the way parsing communities if no community alias exists

bgpd: Optimize the way parsing communities if no community alias exists

If at least one community alias is configured, then let's do the work,
otherwise we don't need to spend time on splitting stuff and creating
a new string.

This should improve the performance.

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

Merge pull request #17375 from opensourcerouting/fix/use_pcre2

packaging: Use PCRE2 for .deb/.rpm builds

tests: Add topotest for OSPF multi-instance default origination.

This change adds a topotest to test various case of OSPF multi-instance
origination including cases where the criteria route is from another
instance of OSPF, as well as the same OSPF instance (where a default
should not be originated).

Signed-off-by: Acee Lindem <acee@lindem.com>

bgpd: Use int, not long long for route-map action in Lua script

This is an issue for big-endian architectures, that causes incorrect castings.

lua_tointegerp() uses int*, not long long*.

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

lib: Fix Lua script unit test

When building for big-endian architectures, this is failing because of
long long / int casting issues, let's use a separate integer to get the
results.

This is especially important when building the Docker images for multiple arches.

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

pim6d: fix coverity scan warning

Fix Coverity Scan CID 1601875: use the return value to tell user about
the availability of a next hop to the learned RP (needs debug enabled).

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

tools: Fix syntax raw parsing for make-foobar helper

96 38.93 config.status: creating Makefile
96 39.48 /src/python/makefile.py:55: SyntaxWarning: invalid escape sequence '\s'
96 39.48   "^#\s*include.*_clippy.c",

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

Merge pull request #17451 from opensourcerouting/fix/bgp_optimizations

bgpd: Optimize the outbound path if RFC8212 is applied

ospfd: OSPF multi-instance default origination fixes

When originating a default AS-External LSA in one OSPF instance,
it wasn't working if the criteria route was installed by another OSPF
instance. This required more flexible processing of the OSPF external
route information.

Also fix problem multi-instance display for "show ip ospf
 <instance> database ...".

Signed-off-by: Acee Lindem <acee@lindem.com>

bgpd: Optimize the outbound path if RFC8212 is applied

If we have (default enabled) enabled `bgp ebgp-require-policy`, then first check
it before applying the route-maps.

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

Merge pull request #17445 from cscarpitta/fix/sharp-use-ipv6-max-bitlen

sharpd: Convert numeric 128 into IPV6_MAX_BITLEN for prefixlen

Merge pull request #17444 from cscarpitta/fix/sharpd-cli-help-string

sharpd: Fix a few typos in CLI help messages

sharpd: Use IPV6_MAX_BITLEN in `sharp send opaque notify`

Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>

sharpd: Use IPV6_MAX_BITLEN in `sharp install seg6local-routes`

Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>

sharpd: Fix typo in `sharp install seg6local-routes` help message

Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>

sharpd: Fix typo in `sharp install seg6-routes` help message

Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>

sharpd: Fix typo in `sharp install routes` help message

Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>

Merge pull request #17193 from opensourcerouting/frr-name-use

accords: guidelines/terms for FRRouting trademarks

Merge pull request #17427 from opensourcerouting/fix/more_details_for_ebgp_no_policy

bgpd: Add more details to ebgp requires policy warning

Merge pull request #17435 from opensourcerouting/fix/bgp_allow_0.0.0.0_if_multiple_nlris_exist

bgpd: Validate both nexthop information (NEXTHOP and NLRI)

Merge pull request #16937 from opensourcerouting/embedded-rp

pim6d: support embedded-rp

Merge pull request #17432 from raja-rajasekar/rajasekarr/evpn_pop_items_bp

bgpd : backpressure - Fix to pop items off zebra_announce FIFO for few EVPN triggers

bgpd: Validate both nexthop information (NEXTHOP and NLRI)

If we receive an IPv6 prefix e.g.: 2001:db8:100::/64 with nextop: 0.0.0.0, and
mp_nexthop: fc00::2, we should not treat this with an invalid nexthop because
of 0.0.0.0. We MUST check for MP_REACH attribute also and decide later if we
have at least one a valid nexthop.

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

tests: Check if IPv6 prefix has a valid nexthop if multiple NLRIs exist

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

Merge pull request #17434 from opensourcerouting/fix/bgp_color_parsing_ecomm

bgpd: Fix color extended community parsing

bgpd : backpressure - Fix to pop items off zebra_announce FIFO for few EVPN triggers

In cases such as 'no advertise-all-vni' and L2 VNI DELETE, we need to
pop all the VPN routes present in the bgp_zebra_announce FIFO yet to
be processed regardless of VNI is configured or not.

NOTE: NO need to pop the VPN routes in two cases
 1) In free_vni_entry
   - Called by bgp_free()->bgp_evpn_cleanup().
   - Since bgp_delete is called before bgp_free and we pop all the dest
     pertaining to bgp under delete.
 2) evpn_delete_vni() when user configures "no vni" since the withdraw
    of all routes happen in normal cycle.

Fixes: a07df6f7548f6bd1b92acbb7a10c3823de33fe5f
("bgpd : backpressure - Handle BGP-Zebra(EPVN) Install evt Creation")

Ticket :#4163611

Signed-off-by: Rajasekar Raja <rajasekarr@nvidia.com>

bgpd: Use tmp_as when parsing color extended community

as is not initialized and it's assigned only later.

CID: 1601739

Fixes: 937cf4d ("bgpd:support of color extended community color-only types")
Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

Revert "bgpd: Fix color extended community parsing"

This reverts commit 8e771bda840e189eed2851f63ad80cb1a235f40f.

bgpd: Add more details to ebgp requires policy warning

This will tell explicitly which peer does not have a filter applied.

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

Merge pull request #17433 from donaldsharp/zebra_debug_assert

Zebra debug assert

Merge pull request #17429 from opensourcerouting/fix/unsupported_commands_bgpd

bgpd: Drop unsupported commands

zebra: Put debug guards in zebra_vxlan.c

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

zebra: zebra_vxlan.c assert on dev escape problem

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

zebra: Missed debug guard in zebra_evpn.c

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

bgpd: Depracate no_synchronization_cmd, no_auto_summary_cmd commands

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

Merge pull request #17422 from opensourcerouting/fix/coverity_bgp_color

bgpd: Fix color extended community parsing

Merge pull request #17426 from mjstapp/fix_z_evpn_debug

zebra: fix unguarded debug in evpn code

Merge pull request #17423 from opensourcerouting/fix/show_advertised_routes_plain_with_addpath

bgpd: Show neighbor advertised paths including addpath

zebra: fix unguarded debug in evpn code

Guard a debug in the evpn code.

Signed-off-by: Mark Stapp <mjs@cisco.com>

pimd: prevent invalid memory access

Signalize termination to functions so they can avoid accessing pointers
that might be no longer available.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

pimd: fix check-patch warning

Remove unnecessary braces.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

pim6d: fix channel oil leak on embedded RP

When a multicast route is created with embedded RP the channel oil
never gets decremented when `clear ipv6 mroute` is called.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

doc: document and explain PIM embedded RP

Let user know about PIM embedded RP configuration commands.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

topotests: new embedded RP topology test

Test embedded RP features:
- Embedded RP itself
- Embedded RP group list filtering
- Embedded RP limit

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

pim6d: support embedded-rp

Implement embedded RP support and configuration commands.

Embedded RP is disabled by default and can be globally enabled with the
command `embedded-rp` in the PIMv6 configuration node.

It supports the following options:
- Embedded RP maximum limit
- Embedded RP group filtering

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

bgpd: Reduce the nesting level for show_adj_route()

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

bgpd: Show neighbor advertised paths including addpath

Without the patch only the best path is displayed.

With the patch, display all paths including addpaths, but only for non-JSON
output to avoid breaking existing output.

E.g.:

```
munet> r2 shi vtysh -c 'sh ip bgp nei 192.168.2.3 advertised-routes'
     Network          Next Hop            Metric LocPrf Weight Path
 *>  172.16.16.254/32 192.168.2.3              0             0 65003 ?
 *   172.16.16.254/32 192.168.2.4              0             0 65004 ?
 *>  192.168.2.0/24   192.168.2.3              0             0 65003 ?
 *   192.168.2.0/24   192.168.2.4              0             0 65004 ?
```

Before it was:

```
munet> r2 shi vtysh -c 'sh ip bgp nei 192.168.2.3 advertised-routes'
     Network          Next Hop            Metric LocPrf Weight Path
 *>  172.16.16.254/32 192.168.2.3              0             0 65003 ?
 *>  192.168.2.0/24   192.168.2.3              0             0 65003 ?
```

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

bgpd: Fix color extended community parsing

CID: 1601739

Fixes: 937cf4db1738d768ed7dac8677d16f7585bb75de ("bgpd:support of color extended community color-only types")
Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

Merge pull request #17331 from Jafaral/ospf-instance

tests: add support for ospf instances  with unified configs

Merge pull request #17205 from piotrjurkiewicz/patch-1

debian: Add missing libprotobuf-dev to grpc profile

Merge pull request #17297 from mjstapp/mjs_ifp_table

zebra, lib: use internal rbtree for per-NS tree of ifps

tests: add support for ospf instances  with unified configs

Signed-off-by: Jafar Al-Gharaibeh <jafar@atcorp.com>

Merge pull request #17368 from louis-6wind/bgp_vpnv4_asbr-clarify

tests: clarify bgp_vpnv4_asbr

Merge pull request #17410 from opensourcerouting/fix/bgpd_ebgp_multihop_set_unset

BGP BFD session things

Merge pull request #17374 from opensourcerouting/fix/rfc_9687

Add two RFCs for BGP to the list

Merge pull request #17366 from baozhen-H3C/202411061447

isisd: fix crash when switching P2P after shutdowning LAN circuit

Merge pull request #17344 from opensourcerouting/fix/reset_bgp_session_only_if_bfd_is_properly_working

bgpd: Reset BGP session only if it was a real BFD DOWN event

isisd: properly display srv6 algorithm

When the segment-routing ipv6 is configured, the SPF
algoritm shows S-SPF is used:

> rt1# show isis segment-routing srv6 node
> Area 1:
> IS-IS L1 SRv6-Nodes:
>
> System ID Algorithm SRH Max SL SRH Max End Pop SRH Max H.encaps SRH Max End D
> -----------------------------------------------------------------------------------------
> 0000.0000.0001 S-SPF 3 3 2 5

Actually, the segment-routing ipv6 algo capabilities (SPF) are rewritten.
Don't rewrite the algo[0].

> rt1# show isis segment-routing srv6 node
> Area 1:
> IS-IS L1 SRv6-Nodes:
>
> System ID Algorithm SRH Max SL SRH Max End Pop SRH Max H.encaps SRH Max End D
> -----------------------------------------------------------------------------------------
> 0000.0000.0001 SPF 3 3 2 5

Fixes: ("78774bbcd51b") isisd: add isis flex-algo lsp advertisement
Signed-off-by: Dmytro Shytyi <dmytro.shytyi@6wind.com>

tests: Check if BFD session is created with update-source (interface)

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

bgpd: Update source address for BFD session

If BFD is down, we should try to detect the source automatically from the given
interface.

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

Merge pull request #17409 from krishna-samy/show_nexthop_json_fix

bgpd: fix resolvedPrefix in show nexthop json output

tests: Check if BGP BFD session is created correctly with multihop/update-source

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

bgpd: Do not try to uninstall BFD session if the peer is not established

Having something like:

```
 neighbor 192.168.1.222 ebgp-multihop 32
 neighbor 192.168.1.222 update-source 192.168.1.5
 neighbor 192.168.1.222 bfd
```

Won't work and the result is (empty):

```
$ show bfd peers
BFD Peers:
```

bgp_stop() is called in BGP FSM multiple times (even at startup) that
causes intermediate session interruption when update-source/ebgp-multihop
is triggered.

With this fix, the ordering does not matter and the BFD session's parameters
are updated correctly.

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

bgpd: Do not use an existing peer pointer for ALL_LIST_ELEMENTS()

Use a separate `member` in this case.

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

bgpd: fix resolvedPrefix in show nexthop json output

While populating  the nexthop info for "show bgp nexthop json", resolvedPrefix
is added in parent json object instead of json_nexthop object.
This results in displaying wrong resolvedPrefix for nexthops.
Fixing the same by adding resolvedPrefix to json_nexthop object, so that
the proper resolvedPrefix would be displayed for the respective nexthop

Signed-off-by: Krishnasamy R <krishnasamyr@nvidia.com>

Merge pull request #17362 from raja-rajasekar/rajasekarr/src_proto_for_redist_cmd

bgpd: Fix for match source-protocol in route-map for redistribute cmd