Merge pull request #8295 from opensourcerouting/ospf6-topo-stabilize

topotest: stabilize OSPFv3 topology

Merge pull request #8360 from kuldeepkash/bgp_graceful_restart

tests: Fix for BGP_GR test failures

Merge pull request #8391 from idryzhov/fix-bgp-nb-prefix-list

bgpd: prefix-list and route-map nb fixes

Merge pull request #8384 from volta-networks/fix_no_srb_block

tools: do in-place SRGB/SRLB changes

Merge pull request #8348 from chiragshah6/mdev

yang, bgpd: add northbound transactional callbacks for neighbor route-map for af l2vpn-evpn

tests: Fix for BGP_GR test failures

Few tests were failing in BGP-GR intermittently. Fixing the failures.

Signed-off-by: Kuldeep Kashyap <kashyapk@vmware.com>

Merge pull request #8240 from reubendowle/fixes/nhrp-nat

nhrp: NAT fixes

Merge pull request #8378 from pguibert6WIND/listen_group_limit

bgpd: increase the maximum number of neighbors in a listen group

Merge pull request #8364 from stipmonster/master

Added pathd to the init files

Merge pull request #8392 from idryzhov/ospf-ignore-dup

ospfd: ignore duplicated "ip ospf area" commands

Merge pull request #8376 from mjstapp/fix_remove_inet6_ntoa

lib, ripngd: Remove inet6_ntoa

Merge pull request #8382 from taspelund/add_rd_all_v2

Add support for 'rd all' in EVPN and L3VPN commands

bgpd: increase the maximum number of neighbors

increase the maximum number of neighbors in a bgp group.
Set the maximum value to 50000 instead of 5000.

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

ospfd: ignore duplicated "ip ospf area" commands

Fixes #8388.

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

Merge pull request #8375 from mjstapp/fix_ignore_pcep_test_files

build: add pceplib test output to gitignore

Merge pull request #8358 from idryzhov/fix-nb-vrf-crash

*: modify VRF_CONFIGURED flag only in VRF NB layer

bgpd: prefix-list and route-map nb fixes

Fix missing/incorrect callbacks.

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

Merge pull request #8385 from idryzhov/fix-no-bfd

Fix "no bfd" crash

Merge pull request #8383 from idryzhov/bgpd-fix-build

bgpd: fix build warnings

bfdd: fix profiles autocompletion

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

bbfd: clear nb config entries when removing bfd node

When bfd node is removed, we must clear all NB entries set by its
children - sessions and profiles. Let's store some fake data as an entry
for the bfd node to be able to unset it later.

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

bfdd: remove profiles when removing bfd node

Fixes #8379.

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

bgpd: fix build warnings

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

tools: do in-place SRGB/SRLB changes

avoid issuing a [no] command if we are then issuing the affirmative
one. This avoids spurious requests for the default label ranges,
which might fail if something else is using those labels.

Signed-off-by: Emanuele Di Pascale <emanuele@voltanet.io>

nhrpd: Fix memory leak in error path when forwarding packets

Signed-off-by: Reuben Dowle <reuben.dowle@4rf.com>

bgpd: fix old vpn command compilation failures

set_vpn_nexthop/no_set_vpn_nexthop were failing due to missing
declarations and unused variables.
This adds the missing declaration and removes unused variables.

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

doc: Add Displaying Routes by RD section

Add docs for commands to display BGP table per-RD.
Also update commands to mention flowspec, routes,
advertised-routes, and received-routes.

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

bgpd: Add 'rd all' keyword to EVPN/L3VPN show cmds

New and improved submission for this commit -- updated to accommodate
changes from 4027d19b0.

Adds support for 'rd all' matching for EVPN and L3VPN show commands.
Introduces evpn_show_route_rd_all_macip().
Cleans up some show commands to use SHOW_DISPLAY string constants.

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

nhrpd: Ensure invalid pointer is not used if two NAT extensions received

Also cleanup some minor style issues

Signed-off-by: Reuben Dowle <reuben.dowle@4rf.com>

Merge pull request #8345 from dslicenc/frr-reload-vrf-static

Frr reload vrf static

Merge pull request #8377 from idryzhov/fix-disable-bfdd

ospfd: fix building with --disable-bfdd

tools: frr-reload fixes for deleting vrf static routes

Problems reported that in certain cases, frr-reload.py would
delete vrf static routes inadvertantly due to two different
reasons. First, vrf statics with null0 or Null0 nexthops would
fail the match since rendered as blackholes.  This was already
fixed for non-vrf statics so added for vrf-based.  Second,
frr-reload would fail to match due to different formats for
adding the command. If entered in the old way
"ip route x.x.x.x/x y.y.y.y vrf NAME" and rendered
in the new sway "vrf NAME\nip route x.x.x.x/x y.y.y.y" it would
fail to match do an inadvertant delete.

Ticket: 2570270
Signed-off-by: Don Slice <dslice@nvidia.com>

tools: frr-reload.py changes to make black happy

Since black is finding issues before applying my change, committing
those changes separately

Signed-off-by: Don Slice <dslice@nvidia.com>

Merge pull request #8350 from opensourcerouting/printfrr-revamp

lib: `printfrr()` care package

ospfd: fix building with --disable-bfdd

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

lib: remove inet6_ntoa api

Remove inet6_ntoa() - used static buffer, not needed.

Signed-off-by: Mark Stapp <mjs@voltanet.io>

ripngd: use pI6 instead of inet6_ntoa

Stop using unsafe inet6_ntoa(), just use %pI6.

Signed-off-by: Mark Stapp <mjs@voltanet.io>

build: add pceplib test output to gitignore

Add some pcep unit-test output files to gitignore.

Signed-off-by: Mark Stapp <mjs@voltanet.io>

Merge pull request #7419 from patrasar/routemap_nb

Routemap nb

doc/developer: improve printfrr extension docs

The table is getting rather clunky, let's just break this out and make
it pretty there.

Signed-off-by: David Lamparter <equinox@diac24.net>

lib: save errno in vty_out()

... so `%m` works correctly, without us trampling over `errno` before we
get to formatting it.

Signed-off-by: David Lamparter <equinox@diac24.net>

lib: add `%dPF` & `%dSO` formats

Just quick helpers to print `AF_*` and `SOCK_*` constants.

Signed-off-by: David Lamparter <equinox@diac24.net>

lib: fix & improve `%pSU` format

This wasn't quite formatting IPv6+port in a useful way (no brackets),
and printing the scope ID (interface index) and unix addrs is useful
too.

Signed-off-by: David Lamparter <equinox@diac24.net>

lib: add `%pSQ` and `%pSE` string escape formats

These are for string quoting (`%pSQ`) and string escaping (`%pSE`); the
sets / escape methods are currently rather "basic" and might be extended
in the future.

Signed-off-by: David Lamparter <equinox@diac24.net>

lib: add `%*pHX` + `%*pHS` hexdump in printfrr

(I'll get to `zlog_hexdump()` in a separate pass.)

Signed-off-by: David Lamparter <equinox@diac24.net>

lib: add `%pFB` extension to print struct fbuf *

Useful to insert output from another bprintfrr() call.

Signed-off-by: David Lamparter <equinox@diac24.net>

lib: add `%pVA` recursive printfrr

Analogous to Linux kernel `%pV` (but our mechanism expects 2 specifier
chars and `%pVA` is clearer anyway.)

Signed-off-by: David Lamparter <equinox@diac24.net>

lib: add `FMT_NSTD()` for non-standard printf exts

... to suppress the warnings when using something that isn't quite ISO C
compatible and would otherwise cause compiler warnings from `-Wformat`.

Signed-off-by: David Lamparter <equinox@diac24.net>

lib: allow discerning unspec width in printfrr ext

With 0 currently the default value for the width specifier, it's not
possible to discern that from a %*p where 0 was passed as the length
parameter.  Use -1 to allow for that.

Signed-off-by: David Lamparter <equinox@diac24.net>

lib: put printfrr extension args into struct

... for easier extensibility.  Add width, # and - flags while at it.

Signed-off-by: David Lamparter <equinox@diac24.net>

bgpd: North-bound implementation for bgp rmaps

This commit introduces the implementation for the north-bound
callbacks for the bgp-specific route-map match and set clauses.

Signed-off-by: NaveenThanikachalam <nthanikachal@vmware.com>
Signed-off-by: Sarita Patra <saritap@vmware.com>

ospfd: North-bound implementation for ospfd rmaps

This commit introduces the implementation for the north-bound
callbacks for the ospfd-specific route-map match and set clauses.

Signed-off-by: NaveenThanikachalam <nthanikachal@vmware.com>
Signed-off-by: Sarita Patra <saritap@vmware.com>

ospf6d: North-bound implementation for ospf6d rmaps

This commit introduces the implementation for the north-bound
callbacks for the ospf6d-specific route-map match and set clauses.

Signed-off-by: NaveenThanikachalam <nthanikachal@vmware.com>
Signed-off-by: Sarita Patra <saritap@vmware.com>

zebra: North-bound implementation for zebra rmaps

This commit introduces the implementation for the north-bound
callbacks for the zebra-specific route-map match and set clauses.

Signed-off-by: NaveenThanikachalam <nthanikachal@vmware.com>
Signed-off-by: Sarita Patra <saritap@vmware.com>

lib: Modifications to route-map NB

This commit introduces the changes to the library route-map
north-bound callback implementation in order to align it to
the modified yang definitions.

Signed-off-by: NaveenThanikachalam <nthanikachal@vmware.com>
Signed-off-by: Sarita Patra <saritap@vmware.com>

lib,zebra,bgpd,ospfd,ospf6d: Route-map yang defns

This commit includes the following:
1) Modifications to the frr-route-map.yang to enable addition of
   bgpd, ospfd, ospf6d and zebra specific route map match/set clauses.
2) Yang definitions for bgpd match/set clauses.
3) Yang definitions for ospfd and ospf6d match/set clauses.
4) Yang definitions for zebra match/set clauses.

Signed-off-by: NaveenThanikachalam <nthanikachal@vmware.com>
Signed-off-by: Sarita Patra <saritap@vmware.com>

Merge pull request #8058 from rgirada/ospf-ecmp

ospfd: Max multipath config support

Merge pull request #8372 from FRRouting/revert-8216-add_rd_all

Revert "bgpd: Add 'rd all' keyword to EVPN/L3VPN show cmds"

Revert "bgpd: Add 'rd all' keyword to EVPN/L3VPN show cmds"

Merge pull request #8216 from taspelund/add_rd_all

bgpd: Add 'rd all' keyword to EVPN/L3VPN show cmds

Merge pull request #8267 from idryzhov/bgp-cli-fixes

bgp cli fixes

Merge pull request #8311 from idryzhov/nb-sorting

Sort route-maps/access-lists/prefix-lists by sequence number in running-config

Merge pull request #8057 from ranjanyash54/default-route

ospf6d: Default Route functionality

Merge pull request #8137 from Orange-OpenSource/ospf_ls

ospfd: Add Link State support

Merge pull request #8056 from jmmikkel/peertype

bgpd: Add "bgp bestpath peer-type multipath-relax"

Merge pull request #8368 from volta-networks/fix_isis_ospfv3_coverity

ospf6d: fix coverity warning

Merge pull request #8369 from idryzhov/filter-fixes

access-list and prefix-list fixes

Merge pull request #8362 from idryzhov/fix-ospf-cli-count

ospfd: fix counting of "ip ospf area" commands

ospfd: fix counting of "ip ospf area" commands

Instead of trying to maintain if_ospf_cli_count, let's directly count
the number of configured interfaces when it is needed. Current approach
sometimes leads to an incorrect counter.

Fixes #8321.

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

ospfd: Max multipath config support

Description:
OSPF does not have an option to control the maximum multiple
equal cost paths to reach a destination/route(ECMP).
Currently, it is using the system specific max multiple paths.
But Somtimes, It requires to control the multiple paths from ospf.
This cli helps to configure the max number multiple paths in ospf.

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

Merge pull request #8351 from volta-networks/fix_ospf6_abr_examin_summary_check_cost_change

ospf6d: fix ospf6_abr_examin_summary to check for a path cost change

lib: fix checking for duplicated prefix-list entries

Restore the behavior that was before the NB conversion.

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

lib: restore checks for duplicated prefix-list entries

The checks were incorrectly removed in commit 4d2f546f under the
assumption that it is needed only in CLI. Actually the checks are needed
for the case when the sequence number is explicitly set by a user.

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

lib: finish consolidation of prefix-list nb callbacks

There was an attempt to consolidate the code in commit fae60215, but the
work was not actually finished and some necessary checks were missed.

Let's finish it.

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

lib: fix checking for duplicated access-list entries

Restore the behavior that was before the NB conversion.

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

Merge pull request #8346 from mjstapp/fix_cov_sa

ospf6d: fix coverity warning

Signed-off-by: Lynne Morrison <lynne@voltanet.io>

tools: Added pathd to the relevant init files

Currently pathd is missing from the deamon list in frrcommon
with this missing frrinit can't start pathd if it is added to
the deamon file. This commit adds it to the frrcommon deamon list
and updates the example deamon file.

Signed-off-by: Erik Kooistra <me@erikkooistra.nl>

topotests: add tests for OSPFv3 default route

Three new tests:

 - OSPFv3 convergence using 'ipv6 ospf6 neighbor json'
 - Default route functionality:
   * Check that the LSA is present
   * Check that the route was installed

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

ospf6d: Minor changes for stub area

Signed-off-by: Yash Ranjan <ranjany@vmware.com>

doc: Add documentation for default-information originate.

Signed-off-by: Yash Ranjan <ranjany@vmware.com>

ospf6d: Add logic to set metric if route-map not present

Signed-off-by: Yash Ranjan <ranjany@vmware.com>

ospf6d: Add logic to change the status of router to ASBR

Signed-off-by: Yash Ranjan <ranjany@vmware.com>

ospf6d: Add CLI and logic for default-information originate command

Signed-off-by: Yash Ranjan <ranjany@vmware.com>

*: modify VRF_CONFIGURED flag only in VRF NB layer

This is to fix the crash reproduced by the following steps:

* ip link add red type vrf table 1

  Creates VRF.

* vtysh -c "conf" -c "vrf red"

  Creates VRF NB node and marks VRF as configured.

* ip route 1.1.1.0/24 2.2.2.2 vrf red
* no ip route 1.1.1.0/24 2.2.2.2 vrf red
  (or similar l3vni set/unset in zebra)

  Marks VRF as NOT configured.

* ip link del red

  VRF is deleted, because it is marked as not configured, but NB node
  stays.

Subsequent attempt to configure something in the VRF leads to a crash
because of the stale pointer in NB layer.

Fixes #8357.

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

nhrpd: Make some debug prints more informative

Signed-off-by: Reuben Dowle <reuben.dowle@4rf.com>

nhrpd: Display post-nat NBMA for local addresses

Signed-off-by: Reuben Dowle <reuben.dowle@4rf.com>

nhrpd: Remove unecessary new line from debug messages

Signed-off-by: Reuben Dowle <reuben.dowle@4rf.com>

Merge pull request #8285 from donaldsharp/rpki_find

Rpki find

lib: record output positions in printfrr

This replaces `%n` with a safe, out-of-band option that simply records
the start and end offset of the output produced for each `%...`
specifier.

The old `%n` code is removed.

Signed-off-by: David Lamparter <equinox@diac24.net>

lib: print `(null)` rather than `NULL`

... for consistency with `%s`, which also prints `(null)`.

Signed-off-by: David Lamparter <equinox@diac24.net>

lib: rework printfrr extensions to output directly

Allowing printfrr extensions to directly write to the output buffer has
a few advantages:
- there is no arbitrary length limit imposed (previously 64)
- the output doesn't need to be copied another time
- the extension can directly use bprintfrr() to put together pieces

The downside is that the theoretical length (regardless of available
buffer space) must be computed correctly.

Extended unit tests to test these paths a bit more thoroughly.

Signed-off-by: David Lamparter <equinox@diac24.net>

ospf6d: fix unguarded debug

Put a debug under a conditional.

Signed-off-by: Mark Stapp <mjs@voltanet.io>

isisd: fix coverity SA warning

Add assert to clear SA warning.

Signed-off-by: Mark Stapp <mjs@voltanet.io>

Merge pull request #8317 from mjstapp/fix_short_printfrr_buf

ospf6d: fix ospf6_abr_examin_summary to check for a path cost change

Fixes a regression in test case: anvl-ospfv3-16.14.

Signed-off-by: Karen Schoener <karen@voltanet.io>

tools/gcc-plugin: support [un]signed in pragma

Need `unsigned char *` for `%pHX`.

Signed-off-by: David Lamparter <equinox@diac24.net>

tools/gcc-plugin: fix format precision/width type

`%*.*pEXT` applied the extension type to the precision and width (*.*)
too.  Oops.

Signed-off-by: David Lamparter <equinox@diac24.net>

bgpd: Use rpki_curr_state instead of curr_state

During Review it was suggested that appending rpki_
to curr_state and target_state would be better
variable names.  Instead of going and fixing
3 or so commits up.  Just do this one.

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

doc: Add doc for new rpki commands

`show bgp [afi] [safi] A.B.C.D rpki <valid|invalid|notfound>`
`show bgp [afi] [safi] rpki <valid|invalid|notfound>`

commands need to be documented

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