pimd: mtrace router code improvments and fixes

Mtrace router code has been fixed for the case where no PIM
interface exists towards the source and next-hop is returned.

fwd_ttl is set to one, for correct reporting of mcast ttl.

Also, NO_MULTICAST error is returned when there is no PIM.

And prev_hop is set to source, for case when connected to source,
to help termination of hop-by-hop queries.

Dependency of logging on PIM, when sending mtrace packet, has been
removed.

Signed-off-by: Mladen Sablic <mladen.sablic@gmail.com>

Merge pull request #1920 from donaldsharp/rpik_more

bgpd: Fix warning on 32 bit systems

Merge pull request #1889 from donaldsharp/static_stuff1

zebra: Auto create the nexthop-vrf if needed.

Merge pull request #1914 from donaldsharp/fix_it

bgpd: Move back nh_vrf setting to where it was

Merge pull request #1918 from opensourcerouting/rpki-pkg-master

Rpki pkg master

bgpd: Fix warning on 32 bit systems

We have a signed/unsigned comparison warning that this should
fix.

This should be ok because the range of input is a very limited
value and should never be of concern

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

debianpkg: Fix - need to always run dh_gencontrol

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

debianpkg: Add option to build with RPKI

No RPKI support for Ubuntu 12.04

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

redhat: Add RPKI comment on build instructions

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

redhat: Add option to build with RPKI

Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>
Conflicts:
redhat/frr.spec.in

Merge pull request #1898 from opensourcerouting/reload-helper2

tools: Add helper frr-reload shell script for clean error

Merge pull request #1890 from LabNConsulting/working/master/checkpatch1

checkpatch minor tweaks

zebra: Extend unknown vrf to vrf part of ip route

When specifying a ip route:

ip route 4.3.2.0/24 192.168.201.1 vrf DONNA

Accept DONNA even if it has not been created yet.

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

zebra: Allow duplicate entry of a route

If a user enters a route inside a non kernel existant vrf:

vrf BLOOP
  ip route 4.3.2.0/24 192.168.201.1
!

They should be able to enter it over and over and over and
over and over no matter how futile it is.

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

zebra: Auto create the nexthop-vrf if needed.

Currently if I try to use a nexthop-vrf that has
not been specified yet we get a failure from the cli.

Add code to zebra so that if we fail to find the nexthop-vrf
we auto create it, instead of failing the install.

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

bgpd: Move back nh_vrf setting to where it was

Place the nexthop vrf id setting outside the afi if split
so that each side does not need to keep track of this.

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

Merge pull request #1902 from donaldsharp/vtysh_doc

doc: Clean up vtysh man page

Merge pull request #1885 from msablic/pim_mtrace_client

pimd: improvments to mtrace

doc: Clean up vtysh man page

The vtysh man page was referencing eigrp.  Looked like a
cut-n-paste error?

Added some missing cli for the vtysh program, that users
might want to see.

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

Merge pull request #1739 from LabNConsulting/working/master/bgp-vpn-vrf-leaking

bgpd: vpn - vrf route leaking

Merge pull request #1897 from donaldsharp/vrf_knowledge

zebra: Add some additional vrf info to debugs

zserv.c: delete unused zsend_write_nexthop

zserv.c: zebra_server_send_message() second parameter

tools: Add helper frr-reload shell script for clean error

Adding the shell script allows a clean error if frr-pythontools
is not installed.

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

zebra: Add some additional vrf info to debugs

There were several places where when I am attempting
to debug zebra functionality that I would really
like to have the ability to know what vrf I think
I am operating on.

Add the vrf_id to a bunch of zlog_debug messages
to help figure out issues when they happen.

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

Merge branch 'master' into working/master/bgp-vpn-vrf-leaking

Merge pull request #1821 from vivek-cumulus/pmsi-parse-display

bgpd: Parse PMSI Tunnel Attribute and display

Merge pull request #1893 from opensourcerouting/doc-isis-threeway

doc: document IS-IS three-way handshake config

doc: document IS-IS three-way handshake config

Merge pull request #1863 from donaldsharp/more_nh_groups

More nh groups

Merge pull request #1886 from donaldsharp/strlcpy

lib: Fixup strlcat and strlcpy to be a bit more descriptive

Merge pull request #1869 from opensourcerouting/frr-reload-vnc-fix

tools/frr-reload.py: process vnc settings as sub-context

tools: checkpatch.sh - Add warning that // style comments don't match
       recommended (kernel) comment style

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

Merge pull request #1819 from donaldsharp/static_stuff

Static stuff

tools: checkpatch.sh - use run specific /tmp directories, remove them when done

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

Merge pull request #1884 from donaldsharp/spammy_messages

ospf6d: Stop debug messages happening in rare case

lib: Fixup strlcat and strlcpy to be a bit more descriptive

When I use these functions and am programming on linux I
always have to pull up a man page for these two functions
since they exist in *BSD land only.

Modify the name of the size variable to destsize on
pass in to give me the small hint I need to know
what to do.

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

lib: Add the ability for other people to call a nexthop write line

Add code to allow nexthops to be written by people who are
interested in writing their own nexthop line.

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

lib: Expose nhgc_find command

Expose to the world the nhgc_find command so that
interested parties can find a stored nexthop group.

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

lib: Add nexthop-group cli

Add a nexthop-group cli:

nexthop-group NAME
  nexthop A
  nexthop B
  nexthop C
!

This will allow interested parties to hook into the cli for
nexthops.  Users can add callback functions for add/delete
of a nexthop group as well as add/delete of each individual
nexthop.

Future work( PBR and static routes ) will take advantage
of this.

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

Merge pull request #1830 from opensourcerouting/isis-three-way-adj

IS-IS: implement three-way handshake as per RFC5303

Merge pull request #1880 from pguibert6WIND/enforce_vrf_netns_enable

lib: enforce vrf netns if setns() returns ok

pimd: improvments to mtrace

mtracebis has been updated, so it behaves more like the original
mtrace command.

Signed-off-by: Mladen Sablic <mladen.sablic@gmail.com>

Merge pull request #1867 from LabNConsulting/working/master/indent2

more indent.py related changes

ospf6d: Stop debug messages happening in rare case

When we have a interface disabled in ospfv3 and
we are receiving messages on it.  We will spam
the log file repeatedly.  Debug guard the message.

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

Merge pull request #1704 from qlyoung/docuser

Documentation update to ReST

Merge pull request #1831 from qlyoung/frr-pthread-fixups

lib: some frr_pthread fixes

doc: remove misc trailing whitespace

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

doc: Update CentOS6 doc to use puias repo for updated sphinx

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

bgpd: Fix warning

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

bgpd: Fix enum, use API for log string

Suggested-by: Philippe Guibert <philippe.guibert@6wind.com>
Signed-off-by: Vivek Venkatraman <vivek@cumulusnetworks.com>

Merge branch 'master' of https://github.com/frrouting/frr into pmsi-parse-display

doc: don't check for sphinx-build on `make clean`

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

Merge branch 'master' into docuser

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

lib: privileges are granted to vty netns command

Vty commands that link netns context to a vrf is requiring some
privileges. The change consists in retrieving the privileges at the
vrf_cmd_init() called by the relevant daemon. Then use it.

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

zebra: upon associating netns with vrf, prileges are raised

In order to create the netns context, the zebra parser at startup needs
to have its privileges raised.

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

vtysh: restore some DEFUNs that should not have been reformated by indent.py

tools: fix indent.py to recognize all DEFUN types:
    DEFUN
    DEFUN_ATTR
    DEFUN_CMD_ELEMENT
    DEFUN_CMD_FUNC_DECL
    DEFUN_CMD_FUNC_TEXT
    DEFUN_DEPRECATED
    DEFUN_HIDDEN
    DEFUN_NOSH
    DEFUNSH
    DEFUNSH_ATTR
    DEFUNSH_DEPRECATED
    DEFUNSH_HIDDEN

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

Merge pull request #1806 from vivek-cumulus/evpn-ipv6-tenant-routing

*: EVPN symmetric routing for IPv6 tenant routes

Merge pull request #1860 from qlyoung/debug-mt-safe

lib: add convenience debugging macros

Merge pull request #1828 from qlyoung/zapi-cleanup

zebra: giant zapi cleanup

Merge pull request #1873 from qlyoung/fix-zlog-hexdump

Fix zlog_hexdump

Merge pull request #1781 from mkanjari/type5-default-originate

Type5 default originate

Merge branch 'master' into type5-default-originate

tests: add unit test for zlog

Just tests zlog_hexdump right now

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

lib: fixup zlog_hexdump

* Allocate correct amount of memory
* Use snprintf() instead of sprintf()

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

lib, zebra: slight cleanup after rebase

Rebased zapi-cleanup, needs a bit of poking.

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

lib, zebra: clean up zapi nits

* Get correct data size when parsing VRF ids
* Move some vars into smaller scope

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

lib: zclient.h style fixes

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

lib, zebra: use existing zapi header struct

Nobody uses it, but it's got the same definition. Move the parser
function into zclient.c and use it.

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

zebra: reorganize zserv, batch i/o

Group send and receive functions together, change handlers to take a
message instead of looking at ->ibuf and ->obuf, allow zebra to read
multiple packets off the wire at a time.

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

zebra: standardize ZAPI message handler args

A lot of the handler functions that are called directly from the ZAPI
input processing code take different argument sets where they don't need
to. These functions are called from only one place and all have the same
fundamental information available to them to do their work. There is no
need to specialize what information is passed to them; it is cleaner and
easier to understand when they all accept the same base set of
information and extract what they need inline.

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

zebra: add struct zmsghdr

Formalize the ZAPI header by documenting it in code and providing it to
message handlers free of charge to reduce complexity.

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

zebra: dont return a status code in zapi handlers

All of the ZAPI message handlers return an integer that means different
things to each of them, but nobody ever reads these integers, so this is
technical debt that we can just eliminate outright.

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

tests: add unit test for zlog

Just tests zlog_hexdump right now

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

lib: fixup zlog_hexdump

* Allocate correct amount of memory
* Use snprintf() instead of sprintf()

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

Merge pull request #1868 from ajones-rvbd/ajones-for-upstream

docs/install.texi: document --enable-numeric-version

docs/install.texi: document --enable-numeric-version

Document the configure option to strip non-numeric characters from
the version string.

Testing done:

Build alpine packages in docker, run texinfo on the doc apk package.

Issue: https://github.com/FRRouting/frr/issues/1859
Signed-off-by: Arthur Jones <arthur.jones@riverbed.com>

tools/frr-reload.py: process vnc settings as sub-context

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

Merge pull request #1858 from qlyoung/gitignore-clippy-sources

*: globally ignore clippy-generated source

lib: add convenience debugging macros

* Add DEBUG*() macros

This set of macros allows you to write printf-like debugging lines that
automatically check whether a debug is on before printing. This should
eliminate the need for explicit checks in simple cases. For example:

if (SUCH_AND_SUCH_DEBUG_IS_ON) {
zlog_warn(...);
}

Becomes:

DEBUG(warn, such_and_such, ...);

Or, equivalently,

DEBUGE(such_and_such, ...);

The levels passed to DEBUG are expanded into the names of zlog_*
functions, so the same zlog levels are available. There's also a set of
macros that have the level built into them; DEBUGE for errors, DEBUGW
for warnings, etc. Good for brevity.

* Add singular setting macros

Change the 'SET' macros to accept a boolean indicating whether the
provided bits should be set or unset, and map on/off macros to them.
Helps condense code where you already have a boolean condition that
tells you what you want to do as you can avoid writing the branch.

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

Merge pull request #1866 from ajones-rvbd/ajones-for-upstream

alpine: minimal alpine packaging

alpine: minimal alpine packaging

For building dev packages for alpine, we provide a minimal APKBUILD
file and add a configure option for only numeric versions in the
VERSION variable as alpine does not allow non-numeric characters
in the version string.

These changes allow alpine to be built, but don't yet provide a
mechanism to build.  Changes to do the build in docker are coming
soon...

Testing done:

Built alpine packages in local docker environment, packages
showed no "dev" in the package name.  Also built CentOS packages
with numeric version disabled and the "dev" is still in the package
name.

Issue: https://github.com/FRRouting/frr/issues/1859
Signed-off-by: Arthur Jones <arthur.jones@riverbed.com>

isisd: add nerd-knob to turn three-way-adj off

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

isisd: P2P adjacencies should only be included in LSPDB/SPF when they are up

isisd: implement threeway adjacencies

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

isisd: add support for TLV 240 P2P Three-Way Adjacency

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

isisd: remove unused debug code

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

isisd: allocate circuit_id only for broadcast circuits

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

isisd: cleanup usage of circuit_id

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

isisd: revert some counterproductive indentation

This reverts some of commit 996c93142d3abfab0f6d6c800474e22a8cfbdbc5.

Merge pull request #1847 from msablic/pim_auto_mtrace

pimd: mtrace only IGMP sockets

Merge pull request #1861 from ajones-rvbd/ajones-for-upstream

zebra/if_netlink: compile under musl-libc

Merge branch 'master' of https://github.com/frrouting/frr into pmsi-parse-display

Conflicts:
bgpd/bgp_attr.c

Merge branch 'master' of https://github.com/frrouting/frr into evpn-ipv6-tenant-routing

Conflicts:
bgpd/bgp_evpn.c

zebra/if_netlink: compile under musl-libc

musl-libc is a lightweight libc used by alpine linux:

https://www.musl-libc.org/

AFAICT, this is the only change to the source needed to get
basic frr support compiling on musl.

Two changes in one patch, get ethhdr from netinet/if_ether.h
and replace the only __caddr_t I could find in the source base
with caddr_t.

Testing done:

Compiled apk packages using a docker environment (patches
coming soon) also compiled redhat and debian using a similar
docker environment (RFC patches for those changes are queued
up too)...

Issue: https://github.com/FRRouting/frr/issues/1859
Signed-off-by: Arthur Jones <arthur.jones@riverbed.com>

*: globally ignore clippy-generated source

Tired of dealing with this

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

vtysh: add bgpd vpn-policy nodes

Signed-off-by: G. Paul Ziemba <paulz@labn.net>

bgpd: vpn-vrf route leaking
    - add "debug bgp vpn label" CLI
    - improved debug messages for "debug bgp bestpath"
    - send vrf label to zebra after zebra informs bgpd of vrf_id
    - withdraw vrf_label from zebra if zebra informs bgpd that vrf_id is disabled
Signed-off-by: G. Paul Ziemba <paulz@labn.net>

isisd: debug msg: display distance when adding redist route

Signed-off-by: G. Paul Ziemba <paulz@labn.net>