Merge pull request #9199 from LabNConsulting/chopps/micronet-prime

tests: cleanup: rerun changed files through black

Signed-off-by: Christian Hopps <chopps@labn.net>

tests: cleanup - remove unused imports

Signed-off-by: Christian Hopps <chopps@labn.net>

tests: Disable test which fails under micronet

Signed-off-by: Christian Hopps <chopps@labn.net>

tests: use common exa-receive.py script

New generic script uses a new default node specific log dir to avoid
collisions when running in parallel.

Signed-off-by: Christian Hopps <chopps@labn.net>

tests: add generic exa-receive.py script

Signed-off-by: Christian Hopps <chopps@labn.net>

tests: Make bgp_multiview_topo1 predictable

Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>

tests: remove unneeded mcast group kernel routes and sysctl

- The PIM tests do not need kernel routes to help them bind joins and
sources to specific interfaces. They should do that themselves directly.
Also do not change system wide "rp_filter" sysctl away from the value
required by everyone else.

Signed-off-by: Christian Hopps <chopps@labn.net>

tests: add back a 10 second delay to see if this fixes the failures

Signed-off-by: Christian Hopps <chopps@labn.net>

tools: move frr-reload.py to python3 explicitly

We already, reasonably, require python3 elsewhere. Do so here, and reap some
benefit.

Signed-off-by: Christian Hopps <chopps@labn.net>

tests: keep revisions of configs

Signed-off-by: Christian Hopps <chopps@labn.net>

tests: use new helper object for mcast-tester and iperf

- Decrease igmp query interval to fix pim test run times

Signed-off-by: Christian Hopps <chopps@labn.net>

tests: add helper object for mcast-tester and iperf tool.

Signed-off-by: Christian Hopps <chopps@labn.net>

tests: remove legacy Topo class from micronet

Signed-off-by: Christian Hopps <chopps@labn.net>

tests: remove legacy Topo class from infra

Signed-off-by: Christian Hopps <chopps@labn.net>

tests: remove legacy Topo class (fixes many pylint errors)

Signed-off-by: Christian Hopps <chopps@labn.net>

tests: fix pylint test errors

Signed-off-by: Christian Hopps <chopps@labn.net>

tests: fix pylint infra errors

Signed-off-by: Christian Hopps <chopps@labn.net>

docker: update with micronet changes

Signed-off-by: Christian Hopps <chopps@labn.net>

doc: minor update based on mininet removal

Signed-off-by: Christian Hopps <chopps@labn.net>

tests: micronet: update defaults for results+logging

Signed-off-by: Christian Hopps <chopps@labn.net>

tests: micronet: adapt tests

Signed-off-by: Christian Hopps <chopps@labn.net>

tests: micronet: update infra

Signed-off-by: Christian Hopps <chopps@labn.net>

tests: NEW micronet replacement for mininet

Signed-off-by: Christian Hopps <chopps@labn.net>

Merge pull request #9541 from donaldsharp/evpn_cleanup

typedef cleanup to proper usage of `enum X` or `struct X`

tests: improve vxlan test determinism

Signed-off-by: Christian Hopps <chopps@labn.net>

tests: summarize XML test results

Signed-off-by: Christian Hopps <chopps@labn.net>

Merge pull request #9545 from ton31337/feature/disable-addpath-rx

bgpd: Add disable-addpath-rx knob

Merge pull request #9542 from pguibert6WIND/ssh_rpki_source

bgpd: add rpki source address configuration

Merge pull request #9549 from mobash-rasool/ospfv2-bug-fixes-02

ospfd: Memory Leak seen at show_ip_ospf_neighbor_all_common.

Merge pull request #9077 from idryzhov/ospf-memory-alloc

ospfd: remove unnecessary memory allocation

doc: Add `neighbor PEER disable-addpath-rx` command

Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>

tests: Check if `disable-addpath-rx` knob works

Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>

bgpd: Add `disable-addpath-rx` knob

The idea is to disable addpath-rx capability to avoid unnecessary additional
routes installed.

Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>

ospfd: Memory Leak seen at show_ip_ospf_neighbor_all_common.

Problem Statement:
==================
Memory Leak seen at  show_ip_ospf_neighbor_all_common (ospf_vty.c:4635)

RCA:
=================
In function show_ip_ospf_neighbor_all_common, one child json object is not
 added to the parent child object when there is no nbma neighbor. Hence
 the memory leak.

Fix:
=================
Add the child object to the parent json object.

Fixes: #9548
Signed-off-by: Mobashshera Rasool <mrasool@vmware.com>

Merge pull request #9538 from donaldsharp/bgp_view_not_working

Bgp view not working

bgpd: add rpki source address configuration

Add the ability to configure the source address of rpki
connection. Proposed vty command is to add below parameter:

rpki cache <address> source <bindaddr> <port> preference <pref>
rpki cache <address> source <bindaddr> <port> <usernamessh> ...

This works for both tcp and ssh connections. In case the source
address is not available yet, the rpki retry interval will retry
in a defined amount of time.

Rtrlib library is the library in charge of the binding of the
tcp/ssh connection, and applies the getaddrinfo() and bind()
operations to the passed parameter bindaddr to the respective
tcp_config/ssh_config structures.

Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

zebra: Convert to `enum zebra_slave_iftype` as per our internal standard

We do not use typedef's to talk about structures as per our standard.
Fixing.

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

zebra: Convert to `enum zebra_iftype` as per our internal standard

We do not use typedef's to talk about structures as per our standard.
Fixing.

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

zebra: Convert to `struct zebra_fec` as per our internal standard

We do not use typedef's to talk about structures as per our standard.
Fixing.

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

zebra: Convert to `struct zebra_lsp` as per our internal standard

We do not use typedef's to talk about structures as per our standard.
Fixing.

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

zebra: Convert to `struct zebra_nhlfe` as per our internal standard

We do not use typedef's to talk about structures as per our standard.
Fixing.

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

zebra: Convert to `struct zebra_ile` as per our internal standard

We do not use typedef's to talk about structures as per our standard.
Fixing.

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

zebra: Convert to `struct zebra_neigh` as per our internal standard

We do not use typedef's to talk about structures as per our standard.
Fixing.

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

zebra: Convert to `struct zebra_l3nvi` as per our internal standard

We do not use typedef's to talk about structures as per our standard.
Fixing.

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

zebra: Convert to `struct zebra_vxlan_sg` as per our internal standard

We do not use typedef's to talk about structures as per our standard.
Fixing.

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

zebra: Convert to `struct zebra_mac` as per our internal standard

We do not use typedef's to talk about structures as per our standard.
Fixing.

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

zebra: Convert to `struct zebra_vtep` as per our internal standard

We do not use typedef's to talk about structures as per our standard.
Fixing.

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

zebra: Convert to `struct zebra_evpn` as per our internal standard

We do not use typedef's to talk about structures as per our standard.
Fixing.

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

bgpd: Add some debug events for when things go wrong

As it stands there are cases where FRR is silently handling
error events and not giving any log output to say what is
going wrong.  This should be fixed.

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

lib: Remove unused function vrf_generate_id

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

bgpd: fix bgp_get_bound_name to handle views better

The vrf socket code needs a interface/vrf name to be passed
in, in order for it to properly bind to the correct vrf.
In the case where bgp is using a view based instance
the bgp_get_bound_name should handle views better and
not return anything to be bound to.

Fixes #9519.
Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>

bgpd: Do not randomly generate a vrf id for -Z

When FRR added the -Z parameter the bgp daemon was setting
a vrf identifier based upon a number starting at 1.  This
caused issues when we upgraded the code to the outgoing
sockets to use vrf_bind always.

FRR should never just randomly select a vrf identifier.
Let's just use VRF_DEFAULT when we are in a -Z environment.
It's a safe bet.

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

Merge pull request #9106 from opensourcerouting/clippy-elf-bsd-cond

lib/clippy: fix forgotten conditional

Merge pull request #9340 from stefreak/patch-1

tools: fix frr pathspace folder permissions

Merge pull request #9516 from qlyoung/fix-alpine-docker-images

docker: build libyang2 along with FRR

Merge pull request #9443 from opensourcerouting/topo-multiview-fix

tests: Make bgp_multiview_topo1 predictable

Merge pull request #9469 from ton31337/fix/extcommunity_bandwidth_floating_to_hex

bgpd: Use IEEE-754 Floating Point for storing extcommunity bandwidth

Merge pull request #9439 from ton31337/feature/set_ext_community_to_none

bgpd: Route-map `set extcommunity none`

Merge pull request #9534 from donaldsharp/ospf6_crashes

ospf6d: Prevent crash of show ipv6 ospf data adv-router 0.0.0.0 links…

ospf6d: Prevent crash of show ipv6 ospf data adv-router 0.0.0.0 linkstate-id 0.0.0.0

With this sequence of events:
eva# conf
eva(config)# router ospf6
eva(config-ospf6)# end
eva# show ipv6 ospf data adv-router 0.0.0.0 linkstate-id 0.0.0.0
OSPF6: Received signal 11 at 1630442431 (si_addr 0x0, PC 0x559dcfa3a656); aborting...
OSPF6: zlog_signal+0x18c                  7fd2cc8229f7     7fff606775d0 /lib/libfrr.so.0 (mapped at 0x7fd2cc770000)
OSPF6: core_handler+0xe3                  7fd2cc8616ad     7fff606776f0 /lib/libfrr.so.0 (mapped at 0x7fd2cc770000)
OSPF6: funlockfile+0x50                   7fd2cc74f140     7fff60677840 /lib/x86_64-linux-gnu/libpthread.so.0 (mapped at 0x7fd2cc73b000)
OSPF6:     ---- signal ----
OSPF6: ospf6_lsdb_type_show_wrapper+0x5d     559dcfa3a656     7fff60677dd0 /usr/lib/frr/ospf6d (mapped at 0x559dcf9a5000)
OSPF6: show_ipv6_ospf6_database_adv_router_linkstate_id+0x1f9     559dcfa3c24a     7fff60677e50 /usr/lib/frr/ospf6d (mapped at 0x559dcf9a5000)

OSPF6 crashes.  Fix.

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

Merge pull request #9531 from LabNConsulting/chopps/bgp-fast-reconnect

tests: add global BGP json config for [re]connect timer

tests: add global BGP json config for [re]connect timer

- Fix the syntax.

Signed-off-by: Christian Hopps <chopps@labn.net>

Merge pull request #9318 from Prerana-GB/ibgp_knob

bgp: BGP knob for faster convergence of bgp sessions

Merge pull request #9530 from ton31337/fix/doc_connect_timers_bgp

doc: Add missing `timers` for connect timer in BGP

doc: Add missing `timers` for connect timer in BGP

Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>

Merge pull request #9397 from tkms1122/master

bgpd: minimum-holdtime knob to prevent session establishment with BGP peer with low holdtime.

Merge pull request #9526 from LabNConsulting/chopps/bgp-test-defaults

tests: log bgp nbr changes, and allow config of connect timer

Merge pull request #9525 from idryzhov/ospfd-crash

ospfd: correctly cleanup spf data

Merge pull request #9522 from mobash-rasool/ospfv3-bug-fixes

ospfd: add dead-interval 40 if configured in show running

tests: log bgp nbr changes, and allow config of connect timer

- Allow tests to also change the connect timer as they can for the
keep-alive and holddown timers.

Signed-off-by: Christian Hopps <chopps@labn.net>

ospfd: correctly cleanup spf data

ospf_spf_cleanup frees the data so we need to reset the stale pointers.

Fixes #9523.

Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>

Merge pull request #9518 from ton31337/fix/redundand_interface_name_for_peer

bgpd: Drop redundand `WORD` for no_neighbor_interface_cmd

tests: Test if IEEE floating-point encoding for bw works with older format

Just check if backward-compatibility works fine between uint32 / IEEE encodings.

Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>

ospfd: add dead-interval 40 if configured in show running

Problem Statement:
==================
When hello-interval is configured as 5, automatically dead interval becomes
4 times of hello i.e 20 seconds. But user wants the dead interval as
40 seconds and hello as 5 seconds. Therefore user configures it.
Now "ip ospf dead-interval 40" is not shown in "show running-config"
Therefore when user restarts the daemon, the dead interval goes back to
20 seconds and the neighbors are down.

Fix:
==================
If user configures dead-interval as 40, show it in show running config.

Fixes: #9401
Signed-off-by: Mobashshera Rasool <mrasool@vmware.com>

Merge pull request #9502 from idryzhov/cmd-vtysh-cleanup

lib, vtysh: make node-processing code more generic

alpine: fix path for daemons file install

The install statement was putting the daemons file at "/etc/frr" instead
of making a new directory "/etc/frr" and putting it inside there.

Signed-off-by: Quentin Young <qlyoung@nvidia.com>

docker: add expected log functions to alpine docker-start

Some basic functions are required by frrcommon.sh

Signed-off-by: Quentin Young <qlyoung@nvidia.com>

bgpd: Drop redundand `WORD` for no_neighbor_interface_cmd

Just to be consistent with neighbor_interface_cmd.

Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>

docker: build libyang2 along with FRR

Alpine images have been broken for some time because libyang2 is not
available in Alpine. This patch updates our Dockerfile to build a
libyang2 APK and install it into the image to satisfy FRR's libyang2
dependency.

Unfortunately, libyang2 erroneously includes an internal header from
glibc, making it dependent on glibc to build. FRR's official Docker
images are based on Alpine, which only offers musl libc. Until libyang2
fixes this problem, the libyang2 source that is installed in this image
is a patched version that is compatible with musl libc and not an
official version.

Signed-off-by: Quentin Young <qlyoung@nvidia.com>

Merge pull request #8964 from LabNConsulting/chopps/config-faster

tests: configure/reset routers in parallel

bgpd: Add `neighbor PEER link-bw-encoding-ieee`

This is to avoid breaking changes between existing deployments of
extended community for bandwidth encoding. By default FRR uses uint32
to encode bandwidth, which is not as the draft requires (IEEE floating-point).

This switch enables the required encoding per-peer.

Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>

doc: Add `neighbor PEER link-bw-encoding-ieee`

Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>

Merge pull request #9484 from anlancs/fix-doc-bgp-desc

doc: add a missing field for bgp summary's display

Merge pull request #9511 from LabNConsulting/chopps/ospf-asbr-test-fix

tests: fix ospf ASBR summary test

doc: add a missing field for bgp summary's display

Signed-off-by: anlan_cs <anlan_cs@tom.com>

bgpd: Use IEEE-754 Floating Point for storing extcommunity bandwidth

https://datatracker.ietf.org/doc/html/draft-ietf-idr-link-bandwidth-07 says:

The bandwidth of the link is expressed as 4
   octets in IEEE floating point format, units being bytes (not bits!)
   per second.  It is carried in the Local Administrator subfield of the
   Value Field.

Before:

```
  Extended Community (16), length: 8, Flags [OT]:
    unknown extd community typecode (0x0004), Flags [none]
      0x0000:  0004 fdeb 0001 e848
    0x0000:  0004 fdeb 0001 e848
  Updated routes:
    172.16.16.1/32
```

0001 e848 - means 125000 (1Mbps), which is encoded incorrect.

After:

```
  Extended Community (16), length: 8, Flags [OT]:
    unknown extd community typecode (0x0004), Flags [none]
      0x0000:  0004 fdeb 47f4 2400
    0x0000:  0004 fdeb 47f4 2400
  Updated routes:
    172.16.16.1/32
```

47f4 2400 - means the same, but in floating point format.

Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>

Merge pull request #9503 from opensourcerouting/ospf6d-redistribute-metrics

ospf6d: extend the "redistribute" command with more options

lib/clippy: fix forgotten conditional

This fixes some spurious warnings on *BSD, where `elffile_add_dynreloc`
isn't used since `elf_getdata_rawchunk` is not available.

Signed-off-by: David Lamparter <equinox@opensourcerouting.org>

Merge pull request #9509 from LabNConsulting/chopps/dirlocals

tests: Update dir-locals

Merge pull request #9499 from gsol10/bogus_lsp

isisd: Fix sending of LSP with null seqno

Merge pull request #9488 from pguibert6WIND/fix_nhrp_neigh_state

Fix nhrp neigh state

Merge pull request #9504 from donaldsharp/cbit_is_needed

lib: Send c-bit when not using bfdd as daemon of choice

Merge pull request #9473 from ton31337/fix/BGP_STR_unified

bgpd: Use BGP_STR for CLI definitions everywhere for BGP CLI

tests: fix ospf ASBR summary test

Previously the check did nothing as the input_dict parameter was wrong.

Signed-off-by: Christian Hopps <chopps@labn.net>

tests: Update dir-locals

- format with black in emacs
- wrap text at 88 (black)

Signed-off-by: Christian Hopps <chopps@labn.net>

lib: Send c-bit when not using bfdd as daemon of choice

Commit: 4b983eef2cb5c7306a8303f002d0e053ebeabdca

Modified the zapi send receive of the c-bit to only
be under the HAVE_BFDD.  If you are using ptm-bfd
then the decoder function still expects this to be
sent down.  This commit puts this behavior back

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

Merge pull request #9480 from SaiGomathiN/pim-coverity

pimd: Fixing coverity issues

Merge pull request #9496 from idryzhov/vrf-cmd-init-unused-arg

lib: remove unused argument from vrf_cmd_init

tests: configure routers in parallel

Signed-off-by: Christian Hopps <chopps@labn.net>