build: try to use -D option for ar and ranlib

The -D option zeroes out timestamps in .a files and has become the
default on recent distributions to enable reproducible builds.

This also shuts up the "u ignored because D is default" warning that is
showing up on some distributions.

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

build: fix some escapes in configure.ac

Some test statements were missing "" around variables causing syntax
errors during configure runs.

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

Merge pull request #3191 from donaldsharp/more_vty

bgpd: Add a better breadcrumb for interface based peers

Merge pull request #3189 from opensourcerouting/libunwind

lib: add libunwind support for backtraces

Merge pull request #3195 from donaldsharp/nht_crash

Nht crash

Merge pull request #3190 from qlyoung/allow-static-onlink

staticd: add ability to create onlink static route

Merge pull request #3196 from dslicenc/l2vpn-evpn-upd-grps

bgpd: add command to display update groups for l2vpn evpn

lib: add libunwind support for backtraces

libunwind provides an alternate to backtrace() for printing out the call
stack of a particular location.  It doesn't use the frame pointer, it
goes by the DWARF debug info.  In most cases the traces have exactly the
same information, but there are some situations where libunwind traces
are better.

(On some platforms, the libc backtrace() also uses the DWARF debug info
[e.g.: ARM backtraces are impossible without it] but this is not the
case everywhere, especially not on BSD libexecinfo.)

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

bgpd: add command to display update groups for l2vpn evpn

For troubleshooting purposes, it's useful to display the update-groups
for l2vpn evpn

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

Merge pull request #3192 from lkrishnamoor/evpn_route_server

bgpd: Route Server support for evpn address-family

Merge pull request #3194 from opensourcerouting/bugfix/isis-misc

Bugfix: IS-IS fragmentation issues

bgpd: Cleanup bnch_nexthop_cache_reset to fit cleaner

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

bgpd: Fix crash when using v4 route w/ v6 nexthop

Recent changes to the nht code in bgp caused us to actually
keep a true count of v6 nexthop paths when using v4 over v6.
This change introduced a race condition on shutdown on who
got to the bnc cache first( the v4 table or not ).  Effectively
we were allowing the continued existence of the path->nexthop
pointing to the freed bnc.  This was especially true when
we had route leaking.   So when we free the bnc make sure
we clean up the path->nexthop variables pointing at it too.

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

bgpd: Route Server support for evpn address-family
Signed-off-by: Lakshman Krishnamoorthy lkrishnamoor@vmware.com

bgpd: Add a better breadcrumb for interface based peers

If you are using bgp unnumbered( or interface based peers )
when we detect an error give the user a bit more of a clue
what they may have done wrong.

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

staticd: add ability to create onlink static route

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

isisd: adjust lsp size to area->lsp_size before packing

IS-IS would ignore any area lsp-mtu setting configured after initial
creation of the LSP since move to the new tlv serialized/deserializer.

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

isisd: Unlink LSP fragments from LSP0 on lsp_destroy

isisd would crash when lsp fragments aged out, since they got freed
correctly, but were not removed from LSP0's linked list of fragments.

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

Merge pull request #3188 from opensourcerouting/bgp-snmp-fix-rename

bgpd: fix missed info->path rename in snmp

bgpd: fix missed info->path rename in snmp

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

Merge pull request #3157 from pguibert6WIND/doc_expanded

doc: inform user that expanded communities can not be used in set mode

Merge pull request #3179 from donaldsharp/vni_cmp

bgpd: The l2vni list compare function does not sort

Merge pull request #3181 from qlyoung/frr-sss-indent

tools: reindent frr

bgpd: Ensure that evpn_vtep_ip_cmp actually returns useful data

The evpn_vtep_ip_cmp function must return positive and negative
numbers for when we are doing sorted linked list inserts.

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

tools: reindent frr

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

bgpd: The l2vni list compare function does not sort

The purpose of adding a l2vni as an sorted list is
shot in the foot when the l2vni compare function only
returns 0 or 1.  This will cause subtle crashes when
we add sorted and we end up with multiple list node pointing
to the same thing.

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

Merge pull request #3178 from vincentbernat/fix/json-route-ipv6

bgpd: fix IPv6 next-hop field name for routes in JSON

bgpd: fix IPv6 next-hop field name for routes in JSON

This may break existing clients but the original name contains two
errors.

Signed-off-by: Vincent Bernat <vincent@bernat.ch>

doc: inform user that expanded communities can not be used in set mode

Expanded communities can not be used in route-map set actions.

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

Merge pull request #3172 from opensourcerouting/bfd-mhop-lookup

bfdd: fix multi hop hash lookup

bfdd: fix multi hop hash lookup

Use the proper multi hop hash for matching multi hop peers.

Spotted by Dmitrii Turlupov.

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

Merge pull request #3024 from ton31337/fix/validate_route-map

bgpd: Check if route-map really exists before applying to the peer

Merge pull request #3142 from donaldsharp/bgp_peerhash

Bgp peerhash

Merge pull request #3134 from donaldsharp/bum_flooding

Bum flooding control for BGP and Zebra

Merge pull request #3168 from adharkar/frr-bgp_json

bgpd: BGP JSON new fields

Merge pull request #3167 from chiragshah6/ospfv3_dev

ospf6d: fix ospf6 crash in inp lsa processing

Merge pull request #3137 from opensourcerouting/feature/isis-bfd

Feature: BFD for IS-IS

ospf6d: fix ospf6 crash in inp lsa processing

Use ospf6_prefix_same for comparing two exact same prefix
to determine ECMP for a route.
ospf6_route_cmp expects two different prefix rather
being exactly same.

Ticket:CM-22630
Testing Done:
performed ECMP of intra network prefix route via
sending same route via different available
ospf6 paths.

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

isisd: Fix license header of bfd code

isisd: Address code-style warnings

bgpd: Add '[no] flood <disable|head-end-replication>'

Add the '[no] flood <disable|head-end-replication>' command
to the l2vpn evpn afi/safi sub commands for bgp.  This command
when entered as 'flood disable' will turn off type 3 route
generation for the transmittal of the type 3 route necessary
for BUM replication on the remote VTEP.  Additionally it will
turn off the BUM handling via the new zebra command,
ZEBRA_VXLAN_FLOOD_CONTROL.

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

lib, zebra: Allow the specification of BUM flooding

Allow the modification of whether or not we will allow
BUM flooding on the vxlan bridge.  To do this allow
the upper level protocol to specify via the ZEBRA_VXLAN_FLOOD_CONTROL
zapi message.

If flooding is disabled then BUM traffic will not be forwarded
to other VTEP's.

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

Merge pull request #3140 from vishaldhingra/b_2802

zebra : VRF aware route map in zebra

bgpd: BGP JSON new fields

1. "show bgp ipv4 json"
  - Added "network" field which displays a prefix in 'prefix/prefixlen' format.

2. "show bgp ipv6 json"
  - Added "network" field which displays a prefix in 'prefix/prefixlen' format.
  - JSON does not have "prefix", "prefixLen" fields which are present in IPv4
    command. Added these fields as they are useful.

3. "show bgp ipv4/ipv6 neighbor <neighbor_addr> advertised-routes json"
  - Added "network" field.

4. "show bgp ipv4/ipv6 summary json"
  - Added "pfxSnt" for peers. This count is obtained from corresponding
    update_subgroup.

5. "show bgp neighbor json"
  - Added "sentPrefixCounter"

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

zebra: vrf aware routmap is missing in Zebra #2802(Part 4 of 4)

The new cli for show output

Signed-off-by: vishaldhingra vdhingra@vmware.com

zebra: vrf aware routmap is missing in Zebra #2802(Part 3 of 4)

The cli changes to allow this to work.

Signed-off-by: vishaldhingra vdhingra@vmware.com

zebra: vrf aware routmap is missing in Zebra #2802(Part 2 of 4)

Function parameter replacement of using zvrf instead of vrf_id

Signed-off-by: vishaldhingra vdhingra@vmware.com

 zebra: vrf aware routmap is missing in Zebra #2802(Part 1 of 4)

 Work to handle the route-maps, namely the header changes in zebra_vrf.h
 and the mapping of using that everywhere

Signed-off-by: vishaldhingra vdhingra@vmware.com

Merge pull request #3032 from donaldsharp/bgp_info

`struct bgp_info` to `struct bgp_path_info`

bgpd: Check if route-map really exists before applying to the peer

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

Merge pull request #3153 from adharkar/frr-scount

bgpd: Incorrect sent prefix count for a split subgroup

bgpd: Incorrect sent prefix count for a split subgroup

When a subgroup splits to form a new subgroup because of policy changes
for a peer, new subgroup copies adj out(state about advertised routes)
from the parent subgroup. At the same time, it should also copy
scount(advertised prefix count) to the new subgroup for the count to be
in sync with the adj_out for the subgroup.

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

Merge pull request #3155 from opensourcerouting/building_instructions

doc: update building instructions

Merge pull request #3159 from pguibert6WIND/doc_draft_bgp

doc: fix warning of doc compilation with bgp draft

Merge pull request #3154 from donaldsharp/bgp_mem_lem

Bgp mem issue with martian address interface strings

doc: simplify build instructions

Remove all --enable-xxx configuration flags for features that are enabled
by default.

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

doc: update build instructions for Debian 9

The instruction to checkout to the stable/3.0 branch doesn't belong
here.

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

doc: remove LibXML from the list of dependencies

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

Merge pull request #3145 from pguibert6WIND/ipv6nexthop

bgpd: in case nexthop is ipv6, set nh attribute flag in mpls vpn case

bgpd: in case nexthop is ipv6, set nh attribute flag in mpls vpn case

In mpls vpn import/export, the nexthop attribute should also be
recognised for ipv6 too.

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

doc: fix warning of doc compilation with bgp draft

Some draft ietf are not handle by :rfc: keyword. A workaround is
proposed, like it has been done for flowspec.

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

bgpd: Cleanup memory leak with the martian address intf string

When cleaning up a interface string, from the linked list we were
dropping the name pointer which held the allocated martian address
intf string.

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

bgpd: Use a more appropriate memory type for a string

Track the memory type associated with the bartian address
interface a bit better, instead of using MTYPE_TMP.

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

bgpd: Cleanup warnings issued by CI system from these changes

Modify code to cleanup warnings issued by the CI system from
this series of changes.

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

bgpd: Rename various variable names to something more appropriate

ri -> pi
bi -> bpi
info -> path
info -> rmap_path ( for routemap applications )

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

bgpd: Convert binfo to path

Convert the binfo variable to path.

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

bgpd: Convert all bgp_info_XXX functions to bgp_path_XXX functions

Rename all bgp_info_XXX functions to bgp_path_XXX functions

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

bgpd: Convert `struct bgp_info` to `struct bgp_path_info`

Do a straight conversion of `struct bgp_info` to `struct bgp_path_info`.
This commit will setup the rename of variables as well.

This is being done because `struct bgp_info` is not descriptive
of what this data actually is.  It is path information for routes
that we keep to build the actual routes nexthops plus some extra
information.

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

bgpd: Convert BGP_INFO_XXX to BGP_PATH_XXX

Search and replace all BGP_INFO_XXX to BGP_PATH_XXX

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

bgpd: Rename some BGP_PATH_XXX to BGP_PATH_SHOW_XXX

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

Merge pull request #3077 from rgirada/ospf_2980_fix

ospfd: Resolved the inconsistency behavior when “default-info origina…

isisd: don't add deprecated bfd command

Merge pull request #3037 from patrasar/2851

bgpd: remove ip prefix from as-path, <large,ext>community-list

Merge pull request #3143 from rgirada/fix_3124

ospfd: default route got flushed after lsa refresh timer.

ospfd: issue with the "default-information originate always" command #2980

Made changes such that message wont be sent to zebra to validate default
route existence  if user configured with “always”.

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

Merge pull request #3132 from toreanderson/fix_routemap_prefixlist_match_doc

doc: correct route map match for prefix lists

ospfd: default route got flushed after lsa refresh timer.

Default route type is not considered while  processing  lsa
refresh timer  expiry   which intern makes it flushed from lsdb.

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

Merge pull request #3125 from dslicenc/linklocal-nht

bgpd: allow nht registration on ipv6 link-local addresses

Merge pull request #3133 from qlyoung/bgp-fix-broken-show-community-cmds

fix broken bgp community show cmds

bgpd: Do a bit better job of tracking the bgp->peerhash

When we add/remove peers we need to do a bit better job
of tracking them in the bgp->peerhash.

1) When we have the doppelganger take over, make sure the
winner is the one represented in the peerhash.

2) When creating the doppelganger, leave the current one
in place instead of blindly replacing it.

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

bgpd: Cleanup peer->su handling

Cleanup calls where we were passing in the su for
peer creation a tiny bit.

Creating a peer from the cli will always have a conf_if *or*
a su but not both. While a doppelganger will have both.

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

bgpd: Add ability to dump the bgp peerhash

The bgp->peerhash is a secretive bit of data that we use
to quickly lookup data about peers.  Unfortunately
since we had not way to look at it, we had no way
of knowing if it had gotten in or out of sync.

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

Merge pull request #3135 from donaldsharp/zapi_handler_cabbage

zebra: Cleanup call parameters to some functions

Merge pull request #3119 from pacovn/Coverity_1465497_dead_code

zebra: dead code (Coverity 1465497)

zebra: Cleanup unused file

The zebra_vxlan_null.c file is never used.  Let's remove
from our build system.

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

isisd: Track BFD state and take down adjacency on failure

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

isisd: Add debugging for BFD

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

isisd: Implement BFD replay function

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

isisd: Register/deregister BFD peers for adjacencies

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

isisd: Write BFD config

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

isisd: Add a hook when writing interface config

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

isisd: Add BFD configuration commands

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

isisd: Replace isis_event_adjacency_state_change with a hook

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

isisd: Register as BFD client

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

isisd: Add forward declaration of struct isis_route_info

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

zebra_ptm: Add IS-IS and OpenFabric as protocols supporting BFD

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

zebra_ptm: Introduce IS_BFD_ENABLED_PROTOCOL macro

Unify the check if a client supports BFD into one location.

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

doc: correct route map match for prefix lists

This corrects the route map documentation to add the missing "prefix-list"
keyword, which is necessary when matching against a prefix list (as opposed to
an access list).

Additionally, change hyphens for underscores in the variables the user is
supposed to substitute in those commands, to prevent any confusion with the
"prefix-list" keyword itself, and also to make it more consistent with the
other documented commands (which are already using underscores).

Signed-off-by: Tore Anderson <tore@redpill-linpro.com>

bgpd: remove ip prefix from as-path, <large,ext>community-list

The existing commands "ip as-path", "ip community list", "ip extcommunity
list" & "ip largecommunity list" is used to configure both for ipv4 and
ipv6. So the prefix "ip" is removed from these commands.
All the configuration, show related configuration, show running config
& boot up with write memory is also verified with the provided fix.

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

Merge pull request #3098 from opensourcerouting/watchfrr-delay

tools/frr + watchfrr spring cleaning (southern hemisphere)