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

zebra: remove unused var in zserv.c

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

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

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>

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

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>

*: ignore deprecated code statments in release versions
   by setting CONFDATE to 0

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

Merge pull request #2553 from opensourcerouting/release_proc

doc: Add document for Release Build by maintainer

doc: cleanup maintainer release document

* Sphinx numbered lists use 1. not 1)
* Indents are three spaces
* Blocks under lists should be indented along with the rest of the list
  content
* Sphinx converts :: to : when used as a code block marker
* Apply some preformatted markup to things that should be preformatted
* Remove trailing whitespace

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

libs: add ALL_NEXTHOPS_PTR iterator macro

Because sometimes we have a pointer to a nexthop_group.

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

lib: remove if_lookup_by_index portion of code with unknown vrf

This function should be called with a known vrf_id. All other cases, the
other API should be called.

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

zebra: mpls entry interface lookup adaptation for vrf backend

The search algorithm for interface based on ifindex only is adapted to
vrf netns based too. Only the default netns will be used to search the
interface index.

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

zebra: pbr rule lookup index adaptation for vrf netns

the interface lookup based on ifindex in the case the target vrf is
unknown is using the generic vrf api. Like that, in the case of vrf
based netns, the search across different netns other than the default
one are not searched.

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

zebra: upon RTM_NEWROUTE/DELROUTE, find appropriate interface

The interface lookup algorithm is different according to if we are on
netns vrf or not. If we are on the former case, then we only have to
parse the interfaces of the netns, while if we are on the other case, we
have to parse all the interfaces of all the vrfs ( since index is not
overlapping in the latter case).

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

Merge pull request #2643 from qlyoung/doc-workflow-reviews

doc: add dev docs on code review practices

bgpd: ability to set vpnv4 nexthop with route-map

Ability to set nexthop IP pointer with route-map.

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

doc: add Ldp user documentation

This commit gathers some basic information on how to configure and setup
Ldp, as well as depict some main principles for that protocol.

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

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

Add some information to inform the user that one can either display ipv4
or ipv6 routing table, with the above commands.

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

doc: Add document for Release Build by maintainer

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

zebra: install EVPN gateway MAC as static/sticky

SVI interface ip/hw address is advertised by the GW VTEP (say TORC11) with
the default-GW community. And the rxing VTEP (say TORC21) installs the GW
MAC as a dynamic FDB entry. The problem with this is a rogue packet from a
server with the GW MAC as source can cause a station move resulting in
TORC21 hijacking the GW MAC address and blackholing all inter rack traffic.

Fix is to make the GW MAC "sticky" pinning it to the GW VTEP (TORC11). This
commit does it by installing the FDB entry as static if the MACIP route is
received with the default-GW community (mimics handling of
mac-mobility-with-sticky community)

Sample output with from TORC12 with TORC11 setup as gateway -
root@TORC21:~# net show evpn mac vni 1004 mac 00:00:5e:00:01:01
MAC: 00:00:5e:00:01:01
 Remote VTEP: 36.0.0.11 Remote-gateway Mac
 Neighbors:
    45.0.4.1
    fe80::200:5eff:fe00:101
    2001:fee1:0:4::1

root@TORC21:~# bridge fdb show |grep 00:00:5e:00:01:01|grep 1004
00:00:5e:00:01:01 dev vx-1004 vlan 1004 master bridge static
00:00:5e:00:01:01 dev vx-1004 dst 36.0.0.11 self static
root@TORC21:~#

Signed-off-by: Anuradha Karuppiah <anuradhak@cumulusnetworks.com>
Ticket: CM-21508

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

When we receive a IGMP report on an interface, do not create upstream
state for that request, unless we are the DR for the incoming interface.

This will prevent a interface on a LAN segment from causing traffic
to flow to itself.

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

Merge pull request #2636 from ppmathis/cleanup/bgpd

bgpd: Cleanup of bgp daemon code

Merge pull request #2637 from donaldsharp/pim_malloc_me

pimd: Remove unnecessary alloc failures

doc: add dev docs on code review practices

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

Merge pull request #2642 from donaldsharp/deprecated

ospfd: Remove deprecated code

Merge pull request #2632 from ppmathis/bugfix/bgp-autoshut

bgpd: Fix bgp auto-shutdown behavior

ospfd: Remove deprecated code

This code has been deprecated for over a year, let's remove it now.

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

pimd: Remove unnecessary alloc failures

Remove from pim unnecessary alloc failure testing
as that alloc failure will cause an assert.

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

bgpd: Cleanup of bgp daemon code

This commit removes various parts of the bgpd implementation code which
are unused/useless, e.g. unused functions, unused variable
initializations, unused structs, ...

Signed-off-by: Pascal Mathis <mail@pascalmathis.com>

pimd: Fix up > 80 columns

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

pimd: Make code consistent in regards to lookup

In places where we do a pim_ecmp_nexthop_search, also
use pim_ecmp_nexthop_lookup instead of the single path
case of pim_nexthop_lookup.

This is in preparation of more serious surgery to fix
the weird api of pim_find_or_track_nexthop.

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