bgpd: Simplify the checking local path

Replace the checking local path code in `update_evpn_type5_route_entry()`
with generic function.

Signed-off-by: anlan_cs <vic.lan@pica8.com>

Merge pull request #13177 from mjstapp/fix_ospf_supoort_typo

ospf,ospf6: fix json key typo supoort

Merge pull request #13273 from donaldsharp/metaq_not_making_me_meta_happy

zebra: Actually free up memory associated with the mq list

Merge pull request #13193 from Keelan10/link_state-memory-leak

lib: link state leak fix

Merge pull request #13269 from Keelan10/ospf_lsa-memory-leak

ospfd: Fix ospf_lsa memory leak

Merge pull request #13261 from LabNConsulting/ziemba/rfapi-memleak-cleanup-12478-2

bgpd: rfapi memleak fixes

Merge pull request #13249 from Pdoijode/connected-route-install-fix

zebra: Mark connected route as installed after interface flap event

zebra: Actually free up memory associated with the mq list

Free up the link list data structures as well as properly
account for data sizes.

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

Merge pull request #13242 from LabNConsulting/chopps/no_rip_in_lib

Convert if_rmap to use YANG northbound

ospfd: Fix ospf_lsa memory leak

Free translated LSA when LSA installation fails

Signed-off-by: Keelan Cannoo <keelan.cannoo@icloud.com>

Merge pull request #12797 from jvidalallende/ubi8_minimal_dockerfile

docker: reduce ubi8 images size by using ubi8-minimal as base

Merge pull request #13122 from rgirada/mgmtd_codecov

mgmtd: Fixing coverity issues and style warnings in the code

Merge pull request #13159 from mjstapp/ospf_sockets

ospfd: write socket per interface

Merge pull request #13183 from Pdoijode/pdoijode/pim-json-changes

pimd: Option to get IGMP groups and sources for a particular interface

Merge pull request #12959 from leonshaw/fix/zif-link-nsid

zebra: Add link_nsid to zebra interface

Merge pull request #13258 from donaldsharp/evpn_ping

Evpn ping

Merge pull request #13260 from donaldsharp/bgp_l3vpn_to_vrf_scale_up

tests: Ensure that the scale_up.py script tests for actual sharp inst…

Merge pull request #13090 from pguibert6WIND/explicit_null

bgpd: add cli command to control explicit-null label usage

bgpd: rfapi memleak fixes

Signed-off-by: G. Paul Ziemba <paulz@labn.net>

ripngd: adapt to new YANG NB if_rmap conversion

- uses YANG grouping and calls if_rmap library code to implement.

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

ripd: adapt to new YANG NB if_rmap conversion

- uses YANG grouping and calls if_rmap library code to implement.

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

lib: convert if_rmap to YANG northbound

- nice correspondence between new YANG grouping and shared library code.
- fixes bug with RIPNG use, certainly didn't work before.
- removes rip header from shared library code
- still has uses RIP_NODE/RIPNG_NODE as required by CLI foo.

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

lib: fix formatting

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

Merge pull request #13213 from mjstapp/fix_dplane_shutdown_event

zebra: fix race during shutdown

pimd: Option to get IGMP groups and sources for a particular interface

1. Added interface name, group address and detail option to existing
   "show ip igmp groups" so that user can retrieve all the groups
   or a particular group for an interface. Detail option shows the source
   information for the group. With that, the show command
   looks like:

   "show ip igmp [vrf NAME$vrf_name] groups [INTERFACE$ifname [GROUP$grp_str]] [detail$detail] [json$json]"

2. Changed pim_cmd_lookup_vrf() to return empty JSON if VRF is not present

3. Changed "detail" option to print non pretty JSON

4. Added interface name and group address to existing
   "show ip igmp sources" so that user can retrieve all the sources for
   all the groups or, all the sorces for a particular group for an
   interface. With that, the show command looks like:

   "show ip igmp [vrf NAME$vrf_name] sourcess [INTERFACE$ifname [GROUP$grp_str]] [json$json]"

Signed-off-by: Pooja Jagadeesh Doijode <pdoijode@nvidia.com>

Merge pull request #13250 from chiragshah6/fdev1

bgpd:add local_es_df_preference into l2vpn-evpn es json output

tests: Ensure that the scale_up.py script tests for actual sharp installed routes first

The scale_up.py script used by several tests installs 50k routes into the rib from
sharpd.  It is first looking for the results in the bgp database.  Let's ensure
that the routes are actually installed into the rib first before looking in
the bgp tables.  This should help situations where the system is under extreme
load.

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

Merge pull request #13254 from manojvn/remove_startupjson_doc

mgmtd: remove the frr_startup.json related documentation.

doc: add doc for ospf per-interface write socket

Add a doc entry for the per-interface write socket config.

Signed-off-by: Mark Stapp <mjs@labn.net>

ospfd: support write socket per interface

Add support for a write socket per interface, enabled by
default at the ospf instance level. An ospf instance-level
config allows this to be disabled, reverting to the older
behavior where a single per-instance socket is used for
sending and receiving packets.

Signed-off-by: Mark Stapp <mjs@labn.net>

topotests: add topotest to check bgp lu explicit-null service

The test ensures that the incoming prefixes are received with
the appropriate label value, and that connectivity is ensured
between prefixes.

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

bgpd: add cli command to control explicit-null label usage

In BGP labeled unicast address-family, it is not possible to
send explicit-null label values with redistributed or network
declared prefixes.
A new CLI command is introduced:

  > [no] bgp labeled-unicast explicit-null

When used, the explicit-null value for IPv4 ('0' value) or
IPv6 ('2' value) will be used.
It is necessary to reconfigure the networks or the
redistribution in order to inherit this new behaviour.

Add the documentation.

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

Merge pull request #13163 from isabelladeleon12/isis_log_drops

isisd: Add support for log-pdu-drops

Merge pull request #12698 from Orange-OpenSource/isisd

Isisd/Lib: Add new printfrr format facility for Intermediate System ID

tests: Cleanup formatting in a couple of test scripts

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

tests: Fix evpn tests to ensure mac addresses will show up

Locally, the bgp_evpn_vxlan_svd_topo1 and bgp_evpn_vxlan_topo1
tests are failing for me.  Upon inspection the test is looking
for the mac addresses of the interfaces participating in the
evpn bridging on the hosts.  For some reason on my machine
these mac addresses are not in the l2 tables at all on
PE1 or PE2.  Adding quick pings solves the problems.

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

Merge pull request #13227 from mjstapp/ospf_sock_bufsizes

ospfd: support configuration of socket buffer sizes

Merge pull request #13226 from anlancs/fix/ripd-leak

ripd: Fix memory leak for ripd's route-map

Merge pull request #13240 from chiragshah6/fdev2

bgpd: EVPN-MH ESI is not active suppress EAD-ES Type-1 route

Merge pull request #13245 from anlancs/cleanup/ripd-1

ripd: Cosmetic changes for ripd

mgmtd: remove the frr_startup.json related documentation.

Since mgmtd no longer supports the frr_startup.json, removing the
documentation related to that. Proper documentation will be added
when/if the frr_startup.json is ever supported.

Signed-off-by: Manoj Naragund <mnaragund@vmware.com>

Merge pull request #13247 from donaldsharp/evpn_crashes_and_other_ilk

Evpn crashes and other ilk

bgpd:add local_es_df_preference into l2vpn-evpn es json output

Added "Local ES DF preference" into bgp l2vpn evpn es json output.

 - show bgp l2vpn evpn es <es-id> json
 - show  bgp l2vpn evpn es detail json

Before Fix:-
```
cumulus@torm-11:mgmt:~$ sudo vtysh -c "show bgp l2vpn evpn es
03:44:38:39:ff:ff:01:00:00:01 json"
{
  "esi":"03:44:38:39:ff:ff:01:00:00:01",
  "rd":"27.0.0.21:15",
  "type":[
    "local",
    "remote"
  ],
  "vteps":[
    {
      "vtep_ip":"27.0.0.22",
      "flags":[
        "esr",
        "active"
      ],
      "dfPreference":32767,
      "dfAlgorithm":32767
    },
    {
      "vtep_ip":"27.0.0.23",
      "flags":[
        "esr",
        "active"
      ],
      "dfPreference":32767,
      "dfAlgorithm":32767
    }
  ],
  "vniCount":10,
  "flags":[
    "up",
    "advertiseEVI"
  ],
  "originator_ip":"27.0.0.21",
  "remoteVniCount":10,
  "vrfCount":3,
  "macipPathCount":33,
  "macipGlobalPathCount":132,
  "inconsistentVniVtepCount":0,
  "fragments":[
    {
      "rd":"27.0.0.21:15",
      "eviCount":10
    }
  ]
}
cumulus@torm-11:mgmt:~$
```

After Fix:-
```
cumulus@torm-11:mgmt:~$
cumulus@torm-11:mgmt:~$ sudo vtysh -c "show bgp l2vpn evpn es
03:44:38:39:ff:ff:01:00:00:01 json"
{
  "esi":"03:44:38:39:ff:ff:01:00:00:01",
  "rd":"27.0.0.21:4",
  "type":[
    "local",
    "remote"
  ],
  "vteps":[
    {
      "vtep_ip":"27.0.0.22",
      "flags":[
        "esr",
        "active"
      ],
      "dfPreference":32767,
      "dfAlgorithm":32767
    },
    {
      "vtep_ip":"27.0.0.23",
      "flags":[
        "esr",
        "active"
      ],
      "dfPreference":32767,
      "dfAlgorithm":32767
    }
  ],
  "vniCount":10,
  "flags":[
    "advertiseEVI"
  ],
  "originator_ip":"27.0.0.21",
  "remoteVniCount":10,
  "vrfCount":3,
  "macipPathCount":33,
  "macipGlobalPathCount":132,
  "inconsistentVniVtepCount":0,
  "localEsDfPreference":50000, ====> Added Local ES DF preference
  "fragments":[
    {
      "rd":"27.0.0.21:4",
      "eviCount":10
    }
  ]
}
cumulus@torm-11:mgmt:~$
```

Ticket:#3411906

Issue:3411906

Testing: UT done

Signed-off-by: Sindhu Parvathi Gopinathan's <sgopinathan@nvidia.com>

zebra: Install directly connected route after interface flap

Issue:
After vlan flap, zebra was not marking the selected/best route as installed.

As a result, when a static route was configured with nexthop as directly
connected interface's(vlan) IP, the static route was not being installed
in the kernel since its nexthop was unresolved. The nexthop was marked
unresolved because zebra failed to mark the best route as installed after
interface flap.

This was happening because, in dplane_route_update_internal() if the old and
new context type, and nexthop group id are the same, then zebra doesn't send
down a route replace request to kernel. But, the installed (ROUTE_ENTRY_INSTALLED)
flag is set when zebra receives a response from kernel. Since the
request to kernel was being skipped for the route entry, installed flag
was not being set

Fix:
In dplane_route_update_internal() if the old and new context type, and
nexthop group id are the same, then before returning, installed flag will
be set on the route-entry if it's not set already.

Signed-off-by: Pooja Jagadeesh Doijode <pdoijode@nvidia.com>

bgpd: Do not allow a `no router bgp XXX` when autoimport is happening

When we have these sequence of events causing a crash in
evpn_type5_test_topo1:

(A) no router bgp vrf RED 100
   this schedules for deletion the vrf RED instance
(B) a l3vni change event from zebra
   this creates a bgp instance for VRF RED in some cases
   additionally it auto imports evpn routes into VRF RED
   Please note this is desired behavior to allow for the
   auto importation of evpn vrf routes
(C) no router bgp 100
   The code was allowing the deletion of the default
   instance and causing tests to crash.

Effectively the test in bgp_vty to allow/dissallow
the removal of the default instance was not correct
for the case when (B) happens.

Let's just not allow the command to succeed in this case as that
the test was wrong.

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

bgpd: Do not allow l3vni changes when shutting down

When a `no router bgp XXX` is issued and the bgp instance
is in the process of shutting down, do not allow a l3vni
change coming up from zebra to do anything.  We can just
safely ignore it at this point in time.

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

bgpd: Use the actual pointer type instead of a void

Let's cut to the chase, we know the pointer type and
it allows us to not to some gyrations.

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

Merge pull request #13239 from opensourcerouting/fix/enable_scripting_for_docker_alpine_build

docker: Fix --enable-scripting for Docker images

ripd: Simplify code for distance

Simplify the code for `rip_distance_apply()`.

Signed-off-by: anlan_cs <vic.lan@pica8.com>

ripd: Correct one debug log

Correct one debug log, which wrongly mixed address and port.

Before:
```
ripd[469497]: [NDAGH-Z85V7] rip_send_packet 3.3.3.4 > 224.0.0.9 (enp1s0)
ripd[469497]: [VEJY5-67P5X] SEND to 224.0.0.9520
```
After:
```
ripd[471330]: [NDAGH-Z85V7] rip_send_packet 3.3.3.4 > 224.0.0.9 (enp1s0)
ripd[471330]: [T8DFR-P09JH] SEND to 224.0.0.9 port 520
```

Signed-off-by: anlan_cs <vic.lan@pica8.com>

bgpd:evpn-mh esi not active suppress ead-es route

update_type1_routes_for_evi() is called from
L3VNI/L2VNI up event, if ESI is not UP then
do not advertise EAD-ES Type-1 route.
Just like from multiple places EAD-ES route
origination checks for its oper status.

Ticket:#3413454
Issue:3413454

Signed-off-by: Trey Aspelund <taspelund@nvidia.com>
Signed-off-by: Chirag Shah <chirag@nvidia.com>

tests: Copy script1.lua for make dist

This is needed for `make dist` when building Docker images with Lua support.

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

docker: Enable Lua (scripting) for Alpine images

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

build: Take LUA_LIBS from pkg-config

Alpine Linux has LUA_LIB with a broken path, let's use pkg-config to grab
the LUA_LIBS properly.

checking for library containing lua_load... no
configure: error: Lua 5.3 libraries are required to build with Lua support. No other version is supported.

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

Merge pull request #13237 from LabNConsulting/chopps/fixriptest

tests: fix cause of intermittent failure

tests: fix cause of intermittent failure

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

ospf,ospf6: fix json key typo supoort

Fix json key 'supoort' -> 'support'; add 1-year deprecation
notice; fix topotests to use corrected json key.

Signed-off-by: Mark Stapp <mjs@labn.net>

Merge pull request #13109 from LabNConsulting/topotest-cleanup

minor topotest cleanup

Merge pull request #13228 from LabNConsulting/chopps/json-via-yang

Update ripd YANG operational state (ECMP routes)

tests: Check if RIP `allow-ecmp` command works correctly

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
Signed-off-by: Christian Hopps <chopps@labn.net>

ripd: implement new YANG operational state

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

ripd: yang: extend nexthops functionality in YANG model - skeleton

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

lib: Fix link state memory leak

Free link message data when a delete event is recorded.

Signed-off-by: Keelan Cannoo <keelan.cannoo@icloud.com>

Merge pull request #13162 from donaldsharp/pass_by_value_no_no_no

*: Use a `struct prefix *p` instead of a `struct prefix` in functions

Merge pull request #13222 from donaldsharp/limit_scope

Limit scope

tests: don't warn with expected results

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

tests: initialize parent test namespace too

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

tests: cleanup infra

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

tests: create 8 link, switch pairs, not 8 links on one switch

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

tests: don't flush ipv6 addresses

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

yang: fix YANG lint

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

*: Use a `struct prefix *p` instead of a `struct prefix` in functions

When passing a prefix into a function let's pass by address instead
of pass by value.  Let's save our stack space.

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

bgpd: Treat withdraw variable as a bool

Used as a bool, treated as a bool.  Make it a bool

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

ospfd: support configuration of socket buffer sizes

Add configurable socket send and receive buffer sizes,
configured at the instance level.

Signed-off-by: Mark Stapp <mjs@labn.net>

ripd: Fix memory leak for ripd's route-map

When cleaning `ripd`, it should free `ctx->name` of `struct if_rmap_ctx`,
not `ctx` itself.  Otherwise, it will lead to memory leak.

Signed-off-by: anlan_cs <vic.lan@pica8.com>

bgpd: Limit flowspec to no attribute means a implicit withdrawal

All other parsing functions done from bgp_nlri_parse() assume
no attributes == an implicit withdrawal.  Let's move
bgp_nlri_parse_flowspec() into the same alignment.

Reported-by: Matteo Memelli <mmemelli@amazon.it>
Signed-off-by: Donald Sharp <sharpd@nvidia.com>

bgpd: Conform bgp_packet.h with coding standards

FRR's standards state that function declarations should
have actual variable names for parameters passed in.
Let's make this so for bgp_packet.h

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

Merge pull request #13214 from chiragshah6/fdev2

zebra:return empty dict in json when evpn is disabled

Merge pull request #13220 from mjstapp/fix_zebra_gr_client

zebra: null-check client pointer during GR processing

zebra: null-check client pointer during GR processing

Add a null check.

Signed-off-by: Mark Stapp <mjs@labn.net>

mgmtd: Fixing style warnings

Description:
Fixing the style warnings in the mgmtd code.

Signed-off-by: Rajesh Girada <rgirada@vmware.com>

mgmtd: Fixing code coverity issues in mgmtd

Description:
the following list of coverity issues seen in mgmtd code.
1. CID 1536832:  Memory - corruptions  (OVERLAPPING_COPY)
   /mgmtd/mgmt_history.c: 85 in mgmt_history_create_cmt_rec()
2. CID 1536831:  Error handling issues  (NEGATIVE_RETURNS)
   /mgmtd/mgmt_be_server.c: 123 in mgmt_be_server_start()
3. CID 1536830:  Resource leaks  (RESOURCE_LEAK)
   /mgmtd/mgmt_history.c: 146 in mgmt_history_read_cmt_record_index()
4. CID 1536829:  Error handling issues  (NEGATIVE_RETURNS)
   /mgmtd/mgmt_fe_server.c: 123 in mgmt_fe_server_start()
5. CID 1536828:  Possible Control flow issues  (DEADCODE)
   /mgmtd/mgmt_txn.c: 1859 in mgmt_txn_get_config()
6. CID 1536827:  Null pointer dereferences  (NULL_RETURNS)
           /mgmtd/mgmt_ds.c: 526 in mgmt_ds_delete_data_nodes()

Signed-off-by: Rajesh Girada <rgirada@vmware.com>

Merge pull request #13194 from Keelan10/sharpd-memory-leak

[WIP] sharpd: fix leak

zebra:return empty dict when evpn is disabled

"show evpn json" returns nothing when evpn is disabled.

Code has been fixed to return {} when evpn is disabled or no entry
available.

Before Fix:-
```
cumulus@r2:mgmt:~$ sudo vtysh -c "show evpn json"
cumulus@r2:mgmt:~$
```

After Fix:-
```
cumulus@r1:mgmt:~$ sudo vtysh -c "show evpn json"
{
}
cumulus@r1:mgmt:~$
```

Ticket:#3417955

Issue:3417955

Testing: UT done

Signed-off-by: Chirag Shah <chirag@nvidia.com>
Signed-off-by: Sindhu Parvathi Gopinathan <sgopinathan@nvidia.com>

Merge pull request #13145 from donaldsharp/do_delete

Improve and fix zebra GR

zebra: fix race during shutdown

During shutdown, the main pthread stops the dplane pthread
before exiting. Don't try to clean up any events scheduled
to the dplane pthread at that point - just let the thread
exit and clean up.

Signed-off-by: Mark Stapp <mjs@labn.net>

Merge pull request #13206 from opensourcerouting/fix/docker_build_alpine_protobuf

docker: Install missing dependencies for Alpine build

Merge pull request #13200 from kuldeepkash/socat_cleanup

tests: [topojson] multicast pimv6 socat cleanup

Merge pull request #13209 from LabNConsulting/chopps/log-redux

tests: lower logging level on some infra logs

doc: Add CLI documentation for new log-pdu-drops config

Update isisd docs with log-pdu-drops config.

Signed-off-by: Isabella de Leon <ideleon@microsoft.com>

isisd: Add log-pdu-drops log functionality

If log-pdu-drops is configured, create an INFO log that displays the PDU type and drop counts when a PDU drop is detected.

Example logs:

2023/03/30 23:54:59.749 ISIS: [VAS9N-1JNNR] PDU drop detected of type: P2P IIH. 1 Total Drops; 0 L1 IIH drops;  0 L2 IIH drops; 1 P2P IIH drops; 0 L1 LSP drops; 0 L2 LSP drops; 0 FS LSP drops; 0 L1 CSNP drops; 0 L2 CSNP drops; 0 L1 PSNP drops; 0 L2 PSNP drops.
2023/03/30 23:54:59.848 ISIS: [VAS9N-1JNNR] PDU drop detected of type: P2P IIH. 2 Total Drops; 0 L1 IIH drops;  0 L2 IIH drops; 2 P2P IIH drops; 0 L1 LSP drops; 0 L2 LSP drops; 0 FS LSP drops; 0 L1 CSNP drops; 0 L2 CSNP drops; 0 L1 PSNP drops; 0 L2 PSNP drops.

Code changes:
Add a new PDU counter function that increments the drop counter and runs the logging functionality if log-pdu-drops is configured.

Signed-off-by: Isabella de Leon <ideleon@microsoft.com>

isisd: Add log-pdu-drops CLI/YANG support

New config functionality:

r1# conf
r1(config)# router isis 1
r1(config-router)# log-
  log-adjacency-changes  Log changes in adjacency state
  log-pdu-drops          Log any dropped PDUs
r1(config-router)# log-pdu-drops
r1(config-router)# end

Signed-off-by: Isabella de Leon <ideleon@microsoft.com>

Merge pull request #13160 from chiragshah6/fdev2

ospfd:display correct DR-BDR router-id in nbr cmd

Merge pull request #13192 from anlancs/fix/ripd-wrong-routemap

ripd: Fix malformed route-map

Merge pull request #13199 from pguibert6WIND/isis_ipv4_linklocal

isisd: consider link local ipv4 addresses as valid

Merge pull request #13131 from LabNConsulting/chopps/no-startup-file

mgmtd: remove startup config feature for now

Merge pull request #12969 from opensourcerouting/ospfd-nssa

ospfd: implement NSSA default routes & ranges

Merge pull request #12837 from donaldsharp/unlikely_routemap

Unlikely routemap

tests: lower logging level on some infra logs

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

docker: Install missing dependencies for Alpine build

protobuf-c-compiler
protobuf-c-dev

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