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

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`

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

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)

bgpd: complete description code for communities

When this description code was added, it was all dead code since none of
the bools that checked if the communities were present were ever changed
from 0.

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

bgpd: fix broken show community commands

SHA c5f1e1b broke about half of the community-related show commands; fix
them

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

Merge pull request #3126 from pacovn/Coverity_1424529_null_check

isisd: null check (Coverity 1424529)

Merge pull request #3127 from pacovn/Coverity_1472236_Improper_use_of_negative_value

zebra: netlink socket check (Coverity 1472236)

Merge pull request #3128 from donaldsharp/cpp_notice_build_failure

lib: Include compiler.h as early as is possible in the build

Merge pull request #3087 from opensourcerouting/bfd-memleak

bfdd: fix memory leak and echo-mode start

Merge pull request #3043 from qlyoung/doc-bgpd-multiple-ass

Doc bgpd multiple ASes

lib: refactor thread_execute

Don't allocate threads in the stack, but use the standardized
`thread_get` and `thread_add_unused` to avoid creating corner cases in
the thread API.

This fixes a thread mutex memory leak in FreeBSD.

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

lib: fix a memory leak in FreeBSD

Two important changes:
* Centralize the thread teardown procedure;
* Save and restore thread mutex context to avoid losing the memory
  pointer;

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

zebra: netlink socket check (Coverity 1472236)

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

lib: Include compiler.h as early as is possible in the build

The compiler.h header provides us with some useful macro's
that we are using in the system.  We do not know exactly
where the CPP_NOTICE and CPP_WARN macros are used but
they can move around.  Place this header early in the
build then.

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

isisd: null check (Coverity 1424529)

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

bgpd: allow nht registration on ipv6 link-local addresses

Signed-off-by: Don Slice <dslice@cumulusnetworks.com>
Testing Done: bgp-smoke completed with no new failures

While testing 5549 support using global addresses, discovered that
ipv6 nexthop tracking thru a route-reflector didn't work.  Since
the next-hop used for remote nexthops resolves to the link-local
of the route-reflector, we need to track it in order to react to
interface down events.  Also tripped over a crash in certain cases
which is also resolved in this fix.

doc: embellish bgp view docs

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

doc: add extra information on bgp views and vrfs

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

doc: clarify documentation on BGP multiple AS

Documentation on how to use multiple autonomous systems was inaccurate
and a bit scattered. Clarify usage of VRFs with multiple autonomous
systems, how to configure them, and their distinction from views. Also
moves a block on L3VPN VRFs out of the 'Basic Concepts' section.

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

Merge pull request #3004 from donaldsharp/other_peer

bgpd: Try to notice when configuration changes during startup

Merge pull request #3108 from dmbaturin/master

Add a script for building a Debian package in one step.

Merge pull request #3113 from donaldsharp/uninited_value

bgpd: prevent uninited variable from being used.

Merge pull request #3120 from opensourcerouting/remove-list-delete

lib: remove deprecated list_delete()/list_free()

zebra: dead code (Coverity 1465497)

The condition in the do/while is always false because 'return_nsid' cannot
reach the end of the loop with 'return_nsid' having a different value than
NS_UNKNOWN. Because of that, the condition can be replaced with 0 (false).
Also, the loop can be removed because the two assignments made at the end
of the loop before the condition check are not used (detected via Clang,
afterwards).

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

bgpd: prevent uninited variable from being used.

Valgrind reports that temp could be used uninitialized
sometimes.

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

Merge pull request #3116 from opensourcerouting/libfrr-exec

lib: print version information in libfrr.so

tools/frr: remove "watchfrr.started" hack

This is no longer neccessary since start-stop-daemon will block until
watchfrr's launch parent has exited.

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

watchfrr, lib: cleanup & delay detaching

This cleans up watchfrr to be more "normal" like the other daemons in
terms of what it does in main(), i.e. using the full frr_*() call set.

Also, this changes the startup behaviour on watchfrr to stay attached on
the daemon's parent process until startup is really complete.  This
should allow removing the "watchfrr.started" hack at some point.

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

Merge pull request #3117 from pacovn/Coverity_1460436_dead_code

eigrpd: dead code (Coverity 1460436)

Merge pull request #3118 from pacovn/Coverity_1399231_dead_code

ospf6d: dead code (Coverity 1399231)

*: list_delete_and_null() -> list_delete()

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

lib: remove deprecated list_delete()/list_free()

Deprecation time has passed.

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

Merge pull request #3102 from ton31337/feature/match_blackhole_nexthops

bgpd: Match routes by type under route-maps

ospf6d: dead code (Coverity 1399231)

The dead code has been disabled with '#if 0', in a similar way to other
disabled logs in the same function.

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

eigrpd: dead code (Coverity 1460436)

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

lib: print version information in libfrr.so

This makes libfrr.so executable to print its version info.  This is
useful if you need to check your libfrr.so matches your daemons.

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

bgpd: Match routes by type under route-maps

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

tools: add a script for building a Debian package in one step.

The script simplifies the relatively lengthy procedure.
It should be invoked from the top level source directory, for example:
./tools/build-debian-package.sh

Signed-off-by: Daniil Baturin <daniil@baturin.org>

Merge pull request #3097 from opensourcerouting/static-linking

build: add --enable-static-bin option

bfdd: allow timer deactivation on shutdown

Peers being shutdown were keeping their timers, so move the cancel code
before the shutdown check.

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

bfdd: disable echo on neighbor request

Don't send echo packets when the neighbor configures echo-interval of 0
(e.g. disabled).

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

Merge pull request #3105 from opensourcerouting/fix/sphinx-stamp

doc: Use `mv -f` in Makefile

ospfd: fix symbol collision

debug_init() was recently added in libfrr, which is now causing a build
error with static linking.

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