Merge pull request #1909 from donaldsharp/zebra_rule

Zebra rule

Merge pull request #1936 from qlyoung/fix-static-route-config-write-missing-space

zebra: fix static route config write

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

docker build: build Alpine Linux dev packages in docker

zebra: fix static route config write

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

docker build: build Alpine Linux dev packages in docker

Building alpine packages in a "standard" distro can be
complicated due to the limited scope of the distro (embedded
and small docker images).  Building in a VM is one possibility,
but docker support for alpine is very good (default docker images
come in alpine due to the very small size).

Here, we want to package up the current git repo into apk packages
that can be easily installed in alpine linux using the apk tool.
This support is not intended to package released versions of
apk packages, that, if it comes to be, should be done here:

git://git.alpinelinux.org/aports

We're content here to build packages that can be used by developers
to try out frr in docker and other alpine environments.

This is a very minimal environment, we don't support importing
keys (so, installing the packages with apk requires the
--allow-untrusted option).  In addition, we can't use the
git commit id in hex as version tag, as alpine doesn't support hex
digits in the version string.  So, we need to convert the git hash
to decimal before tagging the package with the extra version.
This is yucky, but I can't think of another way to get a
unique version per package.  The alpine way (using a numeric date),
only works for released packages, not for dev packages.

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

Merge pull request #1908 from donaldsharp/peer_established

bgpd: Clean up peer status checking for a received nlri

Merge pull request #1924 from donaldsharp/cs

Cs

Merge pull request #1925 from opensourcerouting/bugfix-isis_bpf_read

isisd: Fixed the way isis reads from bpf

bgpd: Don't leak the ecommunity_ecom2str string in debug

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

zebra: Free memory leak

Free the memory leaked stream in failure cases.

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

pimd: Fix leaked fd and prevent null pointer deref

When the pim_nexthop_lookup fails, close the opened fd
as part of the failure condition.

Additionally pim_nexthop_lookup assumes that we've
actually already looked up a nexthop in the past.

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

zebra: Compare to the number of elements not size of array

When figuring out whom to call and if we actually can legally
call into the handler array actually use the number of elements
in the array instead of the size of the array.

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

zebra: Fix leaked fd.

When we detect an error condition, close down the opened
fd.

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

isisd: Fixed the way isis reads from bpf

With this fix, we parse the bpf to process every packet read

Signed-off-by: Ali Rezaee nlndipi@hotmail.com

Merge pull request #1911 from donaldsharp/mpls_love

Mpls love

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

zebra: Cleanup function prototypes for rib.h

Add to the function prototypes the names of variables
to hopefully make it easier for people to program against
this header.

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

sharpd: Add ability to register for nht

Add code to sharpd to allow it to register for nexthop tracking
and then to display when it receives information about the nexthop
it is watching.

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

lib, zebra: Signal the existence of labels on a nexthop for nht

When we are signaling to a client from zebra that a nexthop
has changed, include the labels on the nexthop as well.
Upper level protocols need to know if the labels exist
in order to make intelligent decisions about what to do.

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

zebra: Apply label to all nexthops when needed

The application of a label to a route entry needs to
look at all non-recursive nexthops to be attached to
instead of just the first one.

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

zebra: Cleanup dead function rib_weed_table

the rib_wib_table function was uncalled by anyone remove
and additionally remove it's static function it called.

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

zebra: Read in on startup arbitrary tables

When we receive an arbitrary table over the netlink bus
save it for later perusal and sweep any routes that
we may have created from an earlier run.

The current redistribute code is limited to
ZEBRA_KERNEL_TABLE_MAX.  I left this alone for the
moment because I believe it needs to be converted
to a RB tree instead of a flat array.  Which is more
work for the future.  Additionally this proposed
change might necessitate some cli changes or rethinks.

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

zebra: Upon client disconnect remove routes from all tables

It is possible for clients to install routes into tables
that they desire.  Modify the code to delete these routes
from these tables as well.

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

zebra: Allow for deletion of rules when the originator goes away

When zebra detects that the originator has dissapeared
delete all rules associated with that client.

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

bgpd: Clean up peer status checking for a received nlri

In bgp_update_receive the first thing we do is establish
that the peer->status is Established.  We then do a bunch
of work and call bgp_nlri_parse where we break out for
each address family.  Each AFI is then checking for
being peer->status is Established again.  There is no
point in checking this again.

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

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