Merge pull request #6227 from pguibert6WIND/show_nexthop_group_permit_netns

zebra: permit usage of show nexthop-group with namespace based vrfs

Merge pull request #6220 from kuldeepkash/bgp_graceful_restart

tests: Add bgp-graceful-restart test suites

zebra: permit usage of show nexthop-group with namespace based vrfs

namespace based vrfs can be used along with show nexthop-group command.

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

Merge pull request #6387 from donaldsharp/reload

tools: revert frr-reload.py remove stderr redirects"

Merge pull request #6384 from kooky/set-src-doc

doc: Improve Zebra set src documentation.  (IPv6)

Merge pull request #6383 from opensourcerouting/init-config-perms

build: use configfile mode in init script

Merge pull request #6250 from chiragshah6/yang_nb6

*: RIB operational northbound callbacks implementation

yang: align keys and leafs

Align leafs according to keys order.
Fix some of the nits seen in yanglint -f yang format.

Signed-off-by: Chirag Shah <chirag@cumulusnetworks.com>

yang: add unified protocol types in rib model

Add v4/v6 unified protocol list to rib operational
model.

Signed-off-by: Chirag Shah <chirag@cumulusnetworks.com>

zebra: rib northbound callback implementation

- add RIB's northbound list create/destory callback changes.
- add RIB's northbound callbacks implementation.
- add RIB's nexthop callbacks implementation.
- add RIB's nexthop weight field callback changes.

Signed-off-by: Chirag Shah <chirag@cumulusnetworks.com>

doc: Improve Zebra set src documentation.  (IPv6)

Include an IPv6 example for set src

And a note that the IP address has to exist.   This is to try and make
people aware to avoid things like issue #4249
https://github.com/FRRouting/frr/issues/4249

Signed-off-by: Tim Bray <tim@kooky.org>

zebra: afi-safi identity to value apis

Helper APIs to convert afi-safi identity to values.

Signed-off-by: Renato Westphal <renato@opensourcerouting.org>
Signed-off-by: Chirag Shah <chirag@cumulusnetworks.com>

zebra: route node first next node

Add macros for route entry first and next node walk.

Signed-off-by: Chirag Shah <chirag@cumulusnetworks.com>

zebra: add zebra container to rib model

Add zebra container in rib tree.
When displaying RIB under vrf, it displays
zebra module name.

Signed-off-by: Chirag Shah <chirag@cumulusnetworks.com>

zebra: northbound changes for the rib model

This commit implements:
RIB operational list create/destroy.
Walk over RIB tables using keys.
The first RIB table will be IPV4/unicast (table-id 254)
will be fetched.
Create a new api to fetch RIB table based on
afi-safi and table id as the keys.

remove mandatory true statement from the leaf which
is part of the list key.

Signed-off-by: Chirag Shah <chirag@cumulusnetworks.com>

tools: revert frr-reload.py remove stderr redirects"

This reverts commit 3fa139a65be70e5a81b5f078530094f09a247416.

This is being reverted because this commit completely
breaks the invocation of frr-reload.

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

Merge pull request #6334 from ton31337/fix/labeled_unicast_table_scount

bgpd: Set a proper SAFI for labaled-unicast when looking for scount

Merge pull request #6340 from opensourcerouting/yang-license

Adopt the BSD-2-Clause license for our YANG modules

Merge pull request #6366 from volta-networks/fix_pw_show

ldpd and Zebra:  Expand existing debug commands.

Merge pull request #6370 from barajus/patch-1

overview.rst: add RFC to VRRP

ldpd and Zebra:  Expand existing debug commands.

L2VPN PW are very hard to determine why they do not come up.  The following
fixes expand the existing show commands in ldp and zebra to display a
reason why the PW is in the DOWN state and also display the labeled nexthop
route selected to reach the PW peer.  By adding this information it will
provide the user some guidance on how to debug the PW issue.  Also fixed an
assert if labels were changed for a PW that is between directly connected
peers.

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

build: use configfile mode in init script

This only applies for split-config;  the init script would create an
empty config file with default permissions.

Reported-by: Robert Scheck <robert@fedoraproject.org>
Signed-off-by: David Lamparter <equinox@opensourcerouting.org>

doc: add RFC to VRRP

Add the VRRP bloc + RFC associated

Signed-off-by: barajus <julien.dhaille@gmail.com>

Merge pull request #6381 from qlyoung/clean-grpc

lib: remove unnecessary rpc request in grpc

lib: remove unnecessary rpc request in grpc

Bit of leftover copypaste code there.

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

Merge pull request #6379 from donaldsharp/ip_as_path_removal_really

Ip as path removal really

doc: Fix doc to reflect reality on the ground

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

bgpd: really remove the `no ip as-path...` command

We had already removed the `ip as-path..` command
to have `bgp as-path` but for some reason a `no ip as-path..`
command ALIAS was still around.  Kill with extreme prejudice.

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

*: Update version string

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

Merge pull request #6374 from opensourcerouting/grpc-undue-warning

lib: fix undue warning during gRPC module initialization

lib: fix undue warning during gRPC module initialization

This warning only shows up when a daemon is started with --log=stdout.
Fix it.

Signed-off-by: Renato Westphal <renato@opensourcerouting.org>

Merge pull request #6368 from qlyoung/async-grpc

Async grpc

lib: convert GRPC plugin to async

Synchronous GRPC services are called from arbitrary threads. This makes
access to anything outside the GRPC module unsafe. We need to convert
the plugin to use the async model that allows us to control our own
threads.

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

lib: use frr_pthread to spawn grpc pthread

start grpc thread with frr_pthread library
callbacks to integrate with rcu infrastructure.

If a thread is created using native pthread callbacks
and if zlog is used then it leads to crash.

Signed-off-by: Chirag Shah <chirag@cumulusnetworks.com>

Merge pull request #6348 from dslicenc/frr-reload-bfd

tools: fix frr traceback on bgp neighbor bfd deletion

Merge pull request #6362 from donaldsharp/typedef_i_hatez_it

Typedef replacements

zebra: remove typedef rib_update_event_t from system

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

zebra: Remove typedef rib_table_info_t from system

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

zebra: Remove typedef rib_tables_iter_state from system

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

zebra: Remove typedef rnh_type_t from system

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

zebra: Remove typedef netlink_route_info_t

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

zebra: Remove the typedef netlink_nh_info_t

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

zebra: Remove typedef of mpls_srgb_t

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

zebra: Remove enum around ipv6_nd_suppress_ra_status

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

zebra: Remvoe typedef for zfpm_glob_t

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

zebra: Remove typedef zfpm_msg_format_e

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

zebra: Remove the typedef for zfrpm_state_t

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

zebra: Remove typedef of zfpm_stats_t

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

zebra: Remove typedef around zfpm_rnodes_iter

Typedef's are not used in our system remove.

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

Merge pull request #6363 from donaldsharp/no_no_no

bgpd: `bgp bestpath bandwidth` should not be a legal command

Merge pull request #6365 from donaldsharp/bsd_v6_Null_bs

zebra: Add code to install v6 blackhole routes on *bsd

zebra: Add code to install v6 blackhole routes on *bsd

This code was just missing.  Take the few minutes and
get it done.

!
ip route 4.5.6.7/32 Null0
ipv6 route 4::5/128 Null0
!
$ netstat -rn
Routing tables

Internet:
Destination        Gateway            Flags     Netif Expire
default            192.168.122.1      UGS      vtnet0
4.5.6.7/32         127.0.0.1          UG1B        lo0
127.0.0.1          link#2             UH          lo0
192.168.122.0/24   link#1             U        vtnet0
192.168.122.40     link#1             UHS         lo0

Internet6:
Destination                       Gateway                       Flags     Netif Expire
::/96                             ::1                           UGRS        lo0
::1                               link#2                        UH          lo0
::ffff:0.0.0.0/96                 ::1                           UGRS        lo0
4::5/128                          ::1                           UG1B        lo0
fe80::/10                         ::1                           UGRS        lo0
fe80::%vtnet0/64                  link#1                        U        vtnet0
fe80::5054:ff:fe5a:e705%vtnet0    link#1                        UHS         lo0
fe80::%lo0/64                     link#2                        U           lo0
fe80::1%lo0                       link#2                        UHS         lo0
ff02::/16                         ::1                           UGRS        lo0
$

$ uname -a
FreeBSD donna 12.1-RELEASE FreeBSD 12.1-RELEASE r354233 GENERIC  amd64
$

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

bgpd: `bgp bestpath bandwidth` should not be a legal command

The `bgp bestpath bandwidth` command should not be a legal
command.  Pull out the `no` form to allow this.  Allow
`no bgp bestpath bandwidth` to work as we would expect.

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

Merge pull request #6356 from ton31337/fix/aggregator_0_ignore

bgpd: Ignore aggregator attribute if it's malformed (0.0.0.0)

Merge pull request #6358 from donaldsharp/zebra_onlink

zebra: Loosen ONLINK restrictions a tiny bit

Merge pull request #6359 from sworleys/NHG-No-IPv4-IPv6

zebra: abstract 5549 check into a function

Merge pull request #6357 from mjstapp/fix_ldp_topo1_how

tests: remove unused ref files from ldp topotest

Merge pull request #6352 from volta-networks/fix_ldp_dscp

ldpd: LDP does not always send traffic with correct DSCP value.

zebra: Loosen ONLINK restrictions a tiny bit

Loosen the ONLINK restrictions such that when an upper
level protocol sends us a nexthop with an ONLINK attribute
just ensure that interface is up and usable.  ONLINK effectively
means we know what we are doing to the kernel.

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

ldpd: LDP does not always send traffic with correct DSCP value.

Adding code so that the LDP neighbor that does not initiate the
TCP connection also sets the DSCP (via setsocketopt).

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

tests: remove unused ref files from ldp topotest

Remove a couple of a couple of mis-named reference output files
from the ldp-topo1 suite.

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

bgpd: Ignore aggregator attribute if it's malformed (0.0.0.0)

This is not the attribute involved in path selection and by rfc7606 it should
be just ignored.

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

bgpd: Use slightly modified version of error message if AS path contains 0

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

Merge pull request #6347 from opensourcerouting/llvm-callgraph

tools: LLVM-based call graphs

Merge pull request #6350 from pureport/fix/bgp_double_community_free

bgpd: Check to ensure community attributes exist before freeing them

bgpd: Check to ensure community attributes exist before freeing them

Community attributes might have been removed by an inbound route map, so we
should check to ensure they still exist before trying to free them.
This fixes a segfault described in issue #6345.

Signed-off-by: Josh Cox <josh.cox@pureport.com>

Merge pull request #6349 from qlyoung/hook-cpp-compat

bgpd: Set a proper SAFI for labaled-unicast when looking for scount

The problem is that peer_af_array returns NULL when SAFI is changed to
unicast. We use unicast table, but peer is created and activated under
labeled-unicast, hence we should lookup with a proper SAFI id.

Without this patch peer_af_find() returns NULL and we can't show
PfxSnt in `show bgp summary`.

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

zebra: abstract 5549 check into a function

Abstract the 5549 ipv4 over ipv6 into a function for easier
code reuse and reading.

Signed-off-by: Stephen Worley <sworley@cumulusnetworks.com>

lib: make hook.h cpp-compatible

Some CPP compilers don't support these designated initializers, since
we're just zero initializing don't need em

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

tools: fix frr traceback on bgp neighbor bfd deletion

Problem reported that with certain configs, when the user
deleted a "neighbor x.x.x.x bfd 4 100 100" statement from
frr.conf and then reloaded, a traceback was seen and the
deletion did not succeed.  Found that in some scenarios
it was possible to have something in lines_to_add that
was in a different context and when the re.search was
attempted, it found an empy line and was unhappy.  This
fix avoids trying to search in the wrong context.

Ticket: CM-29145
Signed-off-by: Don Slice <dslice@cumulusnetworks.com>

Merge pull request #6103 from mjstapp/fix_all_unnumbered

zebra: don't treat every interface as unnumbered

Merge pull request #6335 from ton31337/feature/show_policy_in_bgp_summary

build: don't link ldpd.c twice

... this breaks LLVM bitcode linking.

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

python: add graphviz callgraphs

Uses the JSON data extracted from LLVM bitcode by tools/frr-llvm-cg.

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

build: add LLVM bitcode targets

Just an easy way to produce LLVM .bc (bitcode) files.  Not used during
normal builds.

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

tools: frr-llvm-cg

This dumps call graph data from LLVM bitcode files into a JSON file.
Specifically for FRR, it understands thread_add_*(), hook_*() and
install_element() so it can provide extra information in these cases.
As a general feature, it tries to track down function pointers as far as
easily feasible.

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

build: rework Makefile var extraction... again

*sigh* I can't seem to catch a break on this.  Add a regex variant.

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

yang: adopt the BSD-2-Clause license for our YANG modules

Signed-off-by: Renato Westphal <renato@opensourcerouting.org>

Merge commit '97cfd5d207e46a027809125cfb356fd301af0cf5'

(from PR #6137)

tests: Adding test suite bgp_gr_functionality_topo2

1. Test suite has 17 test cases to verify BGP-graceful-restart functionality
2. Execution time is ~20 Mins

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

tests: Adding test suite bgp_gr_functionality_topo1

1. Test suite has 25 test cases to verify BGP-graceful-restart functionality
2. Execution time is ~21 Mins

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

tests: Add library support for BGP-Graceful-Restart automation

1. Adding APIs to common_config.py to support BGP-Graceful-Restart automation
2. Adding APIs to create BGP-GR config to bgp.py
3. Adding verification API for BGP-GR functionality

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

tests: Enhance topotests scripts to kill/start specific Daemons

1. Added APIs to topogen.py and topotest.py to kill/start specific Daemons,
   ex - bgpd, staticd

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

doc: Add an example about what "(Policy)" means under `show bgp summary`

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

bgpd: Show "(Policy)" under PfxRcd/PfxSnt when eBGP policy is in use

It could be hard to notice when eBGP policy is missing in `show bgp summary`.

This adds more sugar to the output:

Normal:
```
 MsgSent   TblVer  InQ OutQ  Up/Down State/PfxRcd   PfxSnt
      10        0    0    0 00:02:07            6        8
```
When rfc8212 is enabled (default since 7.4) - OUTBOUND:
```
 MsgSent   TblVer  InQ OutQ  Up/Down State/PfxRcd   PfxSnt
      13        0    0    0 00:04:56            6 (Policy)
```
INBOUND:
```
 MsgSent   TblVer  InQ OutQ  Up/Down State/PfxRcd   PfxSnt
      19        0    0    0 00:06:27     (Policy)        2
```

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

Merge pull request #6330 from sworleys/No-NHG-Install-With-VRFns

zebra: force off kernel NHG install with netns VRFs

Merge pull request #6329 from donaldsharp/zebra_more_info

zebra: Display ifindex of interface being installed on

zebra: force off kernel NHG install with netns VRFs

Force off kernel NHG install with netns-based VRFs for
now. There is not really a good solution for allowing
kernel nexthop groups in namespaced based vrfs.

Signed-off-by: Stephen Worley <sworley@cumulusnetworks.com>

zebra: Display ifindex of interface being installed on

When installing a nexthop group, dump out the ifindex of the
nexthop being installed as a bit more data for the developer.

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

Merge pull request #6326 from mjstapp/ignore_emacs_desktop

git: ignore emacs desktop

git: ignore emacs desktop

Ignore the emacs desktop files - should never ever be committed

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

Merge pull request #6323 from donaldsharp/floom

bgpd: Ensure that we have a ifp pointer

Merge pull request #6322 from ton31337/feature/bgp_bogon_filter_example

doc: Add an example how to filter BGP bogon ASNs

bgpd: Ensure that we have a ifp pointer

It is possible that the if_lookup_by_index() call will return
a NULL value and calling zclient_send_interface_radv_req.  Just
test that we have a valid interface pointer.

Found by Coverity

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

doc: Add an example how to filter BGP bogon ASNs

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

Merge pull request #6318 from opensourcerouting/fix-exit-vrf

lib: update the CLI xpath index when exiting from the VRF node

Merge pull request #6320 from opensourcerouting/fix-vrf-misconfig-detection

zebra: fix detection of VRF misconfiguration

zebra: fix detection of VRF misconfiguration

The netlink_vrf_change() function is called both when a VRF device
is created in the Linux kernel and when it is activated. This
commit changes this function to perform the VRF misconfiguration
detection only when the VRF device is created, as doing the check
twice would cause a false positive followed by a hard failure (not
to mention the double check is unnecessary since the VRF table ID
can't change once the device is created).

Fixes #6319.

Signed-off-by: Renato Westphal <renato@opensourcerouting.org>

Merge pull request #6241 from volta-networks/fix_ldp_acl

ldpd: fix ACL rule modification