Merge pull request #2607 from pguibert6WIND/complement_fs_patch3

Complement fs patch3

Merge pull request #2681 from qlyoung/doc-cleanup-rpki

doc: cleanup rpki.rst

Merge pull request #2662 from donaldsharp/tests

Tests

tests: Let compiler know this was an intentionall fallthrough

Let the compiler know that this code intentionally fell
through to the next case statement in a switch.

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

tests: Prevent weird type promotion objection in bgp tests

The va_start function cannot take a object that can be type promoted
Looks like a new warning coming in from a new compiler.

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

tests: Fix const change

Some of the parameters for lib/srcdest.. have changed to
consts, make the test cases respect that.

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

Merge pull request #2711 from pacovn/isis_test_fixes

tests/isisd: fix warnings

Merge pull request #2712 from pacovn/Security_Uninitialized_memory_access

pimd: uninitialized memory access fix

Merge pull request #2704 from sworleys/Netlink-Nexthop-Hang

zebra: Add check for nexthop loop to prevent hanging

Merge pull request #2706 from chiragshah6/mdev

bgpd: Add evpn prefix to debug bgp updates prefix

Merge pull request #2714 from qlyoung/fix-unused-variable-zserv

zebra: remove unused var in zserv.c

pimd: uninitialized memory access fix

Signed-off-by: F. Aragon <paco@voltanet.io>

doc: move rpki reference to correct spot

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

zebra: remove unused var in zserv.c

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

Merge pull request #2699 from sworleys/Netlink-MTU-Check

zebra: Add check for mtu on netlink RTM_NEWLINK

tests/isisd: fix warnings

Commit ae9c9aba changed isis_vertex_id_init() and isis_find_vertex()
parameters, so compiler reported warnings in the test (the actual data
passing through is the same because of the union used in the latest changes).
This commit fixes the warnings in the test.

Signed-off-by: F. Aragon <paco@voltanet.io>

Merge pull request #2665 from chiragshah6/evpn_dev

bgpd: support evpn nd ext community

Merge pull request #2707 from opensourcerouting/fix-srcdest-netlink-read

zebra: do not ignore ipv6 srcdest routes

Merge pull request #2640 from pguibert6WIND/doc_ldp

doc: add Ldp user documentation

bgpd: Add evpn prefix to debug bgp updates prefix

Adding EVPN prefix of Type 2, 3 and 5 routes to bgp updates
prefix filters.

Ticket:CM-14476
Testing Done:

Configure multiple evpn options under 'debug bgp updates prefix'.
Below is the running-config output.

MAC-IP route with just MAC:
debug bgp updates prefix l2vpn evpn type macip mac
00:02:00:0a:0a:0a
MAC-IP route with MAC and IP:
debug bgp updates prefix l2vpn evpn type macip mac
00:02:00:00:00:0c ip 45.0.1.9
MAC-IP route with just MAC and IPv6:
debug bgp updates prefix l2vpn evpn type
macip mac 00:02:00:00:00:0a ip 2001:fee1:0:1::8
Type-3:
debug bgp updates prefix l2vpn evpn type multicast ip 27.0.1.19
Type-5:
debug bgp updates prefix l2vpn evpn type prefix
ip 2060:1:1:1::/64

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

Merge pull request #2703 from opensourcerouting/ldpd-missing-command

ldpd: add missing command under the ipv6 address-family node

zebra: do not ignore ipv6 srcdest routes

Commit a2ca67d1d2 consolidated IPv4 and IPv6 handling. It also applied
our ignorance for IPv4 srcdest routes onto IPv6.

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

zebra: add information about which port is monitored

Each ipset with port value monitors either src port or dst port.
The information is added to show pbr iptable commmand.

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

bgpd: flowspec 'no local-install any' wrong order in show runni

When configuring an interface, the no local-install any command appears,
and leads to confusions. because the effect of that command differs if
it is executed after local-install <interfaces> or before executing
local-install <interfaces>, the proposal fix here is to suppress that
command from the vty available commands.

PR=59595
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
Acked-by: Alain Ritoux <alain.ritoux@6wind.com>

bgpd: display more than one FS entre per IP

because the IP destination criterium may match several entries, the show
command may return more than one entry.

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

bgpd: add the pbr identifier in the log messages

for clarity sake, the identifier is inserted in the debug line.

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

bgpd: flush remaining entry if bgp_pbr_match is equal

When adding an entry, a check is done in order to flush previously
configured entries. The whole parameters are checked so as to not remove
some entries that have ipset entries equal, but not iptable settings.

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

bgpd: rework jhash algorithm for bgp_pbr_match entries

Instead of relying on jhash_1word for some parameters that are not 32
bit size, the jash(pointer, len) function is used.

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

bgpd: add debug trace when bgp fs pbr remove too

Removal action from flowspec to policy-routing is also traced back.

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

zebra: show pbr iptable per iptable

Add ability to pass a ip table parameter.

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

bgpd: flowspec pbr entries listed on the bgp information entry

Because one flowspec entry can create 1-N bgp pbr entries, the list is
now updated and visible. Also, because the bgp_extra structure is used,
this list is flushed when the bgp_extra structure is deleted.

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

zebra: Add check for nexthop loop to prevent hanging

Bad nexthop messages from netlink were causing zebra
to hang here. Added a check to verify the length
of the nexthop so it doesn't keep trying to read.

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

ldpd: add missing command under the ipv6 address-family node

Bug found by Patrick Velder.

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

Merge pull request #2698 from sworleys/Netlink-Filter-AFI

zebra: Add address family filters

Merge pull request #2697 from mjstapp/routemap_const

libs, daemons: use const in route-map apply

Merge pull request #2701 from FRRouting/revert-2670-feature/use_med_if_applied_for_default_route

Revert "Use metric for default-originate with route-map if applied"

Revert "Use metric for default-originate with route-map if applied"

Merge pull request #2670 from ton31337/feature/use_med_if_applied_for_default_route

Use metric for default-originate with route-map if applied

Merge pull request #2646 from AnuradhaKaruppiah/evpn-fixes

zebra: install EVPN gateway MAC as static/sticky

Merge pull request #2641 from donaldsharp/pim_igmp_dr

pimd: Do not create upstream state when not DR for igmp request

Remove not needed filters and fix typos

Some more address family filters we can safely ignore
as well as typos in logger. Added AF_MPLS as filterable.

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

Merge pull request #2668 from dslicenc/cm21722-v6-vrf-display

zebra: remove default vrf output for kernel vrf ipv6 blackhole default

Merge pull request #2689 from dslicenc/vrf-exit-reload

tools: handle exit-vrf as end of context block

zebra: Add check for mtu on netlink RTM_NEWLINK

Zebra needed a check for mtu from the message it
received from the kernel before adding the new link.

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

zebra: Add address family filters

The zebra netlink socket was attempting to read netlink
messages with invalid address families in a couple areas.
Added filters and warn messages.

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

zebra: clean up const use in bsd build

Missed a bsd/routing socket change when enforcing use of const
in some kernel-facing api calls.

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

libs, daemons: use const in route-map apply

Use 'const prefix *' in route-map apply apis; led to some
corresponding changes in several daemons.

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

Merge pull request #2652 from LabNConsulting/working/master/confdate_cleanup

Simplify deprecation check

Merge pull request #2672 from pcarana/pcarana-add-rpki-rh

RedHat spec: Add the rpki module when needed

Merge pull request #2692 from manuhalo/module_param_docs

doc: add cmd line param section to modules.rst

Merge pull request #2690 from opensourcerouting/fix-srcdest-route-display

zebra: fix do_show_route_helper to include srcdest routes

Merge pull request #2688 from pguibert6WIND/vrf_delete_on_netns_fail

zebra: cancel vrf creation if netns activation failed

Merge pull request #2691 from adharkar/frr-bgp_cli

bgpd: Changes to BGP show json commands

Merge pull request #2687 from opensourcerouting/fix-cpp-notice-on-old-compilers

lib/compiler.h: Always define CPP_NOTICE

bgpd: Changes to BGP show json commands
Added localRouterID to "show bgp neighbor json"
Added json O/P to "show bgp [AFI] community <community>"

Signed-off-by: Ameya Dharkar <adharkar@vmware.com>

zebra: fix do_show_route_helper to include srcdest routes

tools: handle exit-vrf as end of context block

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

doc: add cmd line param section to modules.rst

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

Merge pull request #2679 from qlyoung/fix-zapi-fuzzing

Fix zapi fuzzing

zebra: cancel vrf creation if netns activation failed

To keep configuration consistent, vrf that have not been able to be
associated with netns are removed.

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

Merge pull request #2635 from donaldsharp/more_pim_neighbor

More pim neighbor

Merge pull request #2686 from opensourcerouting/master-fix-isis-issue-2584

isisd: don't crash when isis_sock_init fails

isisd: don't crash when isis_sock_init fails

When isis_sock_init fails in isis_circuit_up, isis_circuit_down would
be called to cancel timers which were scheduled. However
isis_circuit_down would immediately return, since the state had not been
changed to 'UP' yet.

Fix this by having isis_circuit_down always cancel all the timers.

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

lib/compiler.h: Always define CPP_NOTICE

On old compilers CPP_NOTICE should be a macro evaluating to an empty
statement, instead of being undefined.

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

Merge pull request #2608 from pacovn/PVS-Studio_dead_code_1

eigrpd lib pimd zebra: dead code (PVS-Studio)

Use metric for default-originate with route-map if applied

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

Merge pull request #2678 from pguibert6WIND/sanity_netns

zebra: sometimes, it is not possible to assign a NSID to a vrf

doc: cleanup rpki.rst

Previous commit on this file introduced formatting issues.

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

zebra: fix some cli option descriptions

Couple typos / clarity fixes.

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

zebra: fix zapi fuzzing infra

Some of this stuff got broken with the introduction of pthreads.

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

zebra: sometimes, it is not possible to assign a NSID to a vrf

This test case happens in scenarios with mininet, where external netns
may be impossible for the local instance to be modified. The error is
ignored and the netns parsed is ignored too.

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

zebra: fix nexthop type check for kernel default route display

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

bgpd: support evpn nd ext community

EVPN ND ext community support NA flag R-bit, to have proxy ND.

Set R-bit in EVPN NA if a given router is default gateway or there is a
local
router attached, which can be determine based on local neighbor entry.

Implement BGP ext community attribute to generate and parse  R-bit and
pass along zebra to program neigh entry in kernel.

Upon receiving MAC/IP update with community type 0x06 and sub_type 0x08,
pass the R-bit to zebra to program neigh entry.

Set NTF_ROUTER in neigh entry and inform kernel to do proxy NA for EVPN.

Ref:
https://tools.ietf.org/html/draft-ietf-bess-evpn-na-flags-01

Ticket:CM-21712, CM-21711
Reviewed By:
Testing Done:
Configure Local vni enabled L3 Gateway, which would act as router,
checked
show evpn arp-cache vni x ip <ip of svi> on originated and remote VTEPs.
"Router" flag is set.

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

Merge pull request #2669 from netravnen/hotfix/documentation/activate-bgp-rpki

doc: Update rpki.rst for how-to enable rpki module

RedHat spec: Add the rpki module when needed

Additional help complementing #2669, this adds the rpki module to the bgpd daemon at RPM creation.

doc: Update rpki.rst for how-to enable rpki module

It has not been properly documented yet how to enable the RPKI module with BGP when having installed FRR with RPKI support. This PR seeks to add a section to the user documentation about this. So users will not have to resolve to searching the internet and looking through the FROG mailing list threads for answers.

Merge pull request #2625 from pguibert6WIND/ipv4_ipv6_bgp_show

doc: add more information about show ip bgp and show bgp

Merge pull request #2647 from pguibert6WIND/fix_routemap_vpnv4

bgpd: ability to set vpnv4 nexthop with route-map

zebra: remove default vrf output for kernel vrf ipv6 blackhole default

It was reported that "show ipv6 route vrf <vrfname>", "show ipv6 route
vrf <vrfname> ::/0 " or "show ipv6 route vrf <vrfname> json" all
displayed that the nexthop was in the default vrf.  This was because
the kernel netlink messages would supply the RTA_OIF of the loopback
interface for the kernel-created default route for the vrf, where ipv4
did not supply any RTA_OIF.  This fix suppresses the display if the
nexthop and route entry are in different vrfs and the nexthop is
NEXTHOP_TYPE_BLACKHOLE.

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

Merge pull request #2589 from donaldsharp/admiral_ACK_bar

Netlink sockets are synchronous

Merge pull request #2658 from mjstapp/srcdest_test_const

tests: align with srcdest lib changes, remove bgp binary

tests: add test_bgp_table to gitignore

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

zebra: Remove need for ACK for netlink messages

Kernel requests via netlink are synchronous.

Therefore we do not need to specify a need for a ACK and
we can make the netlink_cmd NONBLOCKING

1) If the netlink message is going to cause an error
we will still get one.  Since results from the kernel
are synchronous we will get the error message on the
netlink_cmd socket and handle it

2) If the netlink message is going to send more than
one packet we will still get them all.  Since the results
from the kernel are synchronous we will receive all data.

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

zebra: Listen to error codes from netlink_socket

When creating a netlink_socket, listen to error
codes and abandon ship if it crashes and burns.

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

Merge pull request #2595 from pguibert6WIND/fix_netlink_rx_route_vrfnetns

zebra: upon RTM_NEWROUTE/DELROUTE, find appropriate interface

Merge pull request #2660 from sworleys/master

zebra: Fix function call in fuzzing code

zebra: Fix function call in fuzzing code

The fuzzing code was calling zebra_client_create which was refactored to zserv_client_create.

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

tests: align with srcdest lib changes, remove bgp binary

Align the srcdest table test with recent api changes that use
'const' more strictly. Remove test_bgp_table binary - looks
as if it was committed by mistake.

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

Merge pull request #2469 from ajones-rvbd/ajones-issue-2442

unit tests: support code coverage instrumentation and reports

Merge pull request #2655 from mjstapp/lib_const_prefix

zebra, libs: use const prefix ptrs in apis

zebra, libs: use const prefix ptrs in apis

Add 'const' to prefix args to several zebra route update,
redistribution, and route owner notification apis.

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

Merge pull request #2651 from qlyoung/doc-cleanup-release-process

doc: cleanup maintainer release document

Merge pull request #2627 from rtrlib/revalidation

bgpd: rpki revalidation

Merge pull request #2650 from mjstapp/nh_iter

libs: add ALL_NEXTHOPS_PTR iterator macro

Merge pull request #2591 from LabNConsulting/working/master/bgp-delayed-default-instance

bgpd: fix NULL dereference in vrf-vpn leak config if before default instance

ospf6d: remove VERSION_TYPE_DEV from CONFDATE checks

Signed-off-by: Lou Berger <lberger@labn.net>

bgpd: remove VERSION_TYPE_DEV from CONFDATE checks

Signed-off-by: Lou Berger <lberger@labn.net>

lib: remove VERSION_TYPE_DEV from CONFDATE checks

Signed-off-by: Lou Berger <lberger@labn.net>

doc: update doc to no longer requires use of VERSION_TYPE_DEV

Signed-off-by: Lou Berger <lberger@labn.net>

*: VERSION_TYPE no longer needed -- remove it

Signed-off-by: Lou Berger <lberger@labn.net>