lib: Send Output error from AddressSanitizer to StdErr

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

rip-topo1: Add check to make sure daemons are still running between essential tests

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

(all tests): Add extra check to make sure daemons are still running after each essential stage.

If daemon crashed at a later stage (not startup), then the test scripts didn't properly detect it and report unpredictable errors. This will properly log the daemon crashes

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

lib: Add different failure comment on processes killed by AddressSanitizer

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

lib: Move updated topotest.py lib with AddressSanitizer to correct directory (fix mistake from 2 commits ago)

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

Doc: Update manual sample build instruction with symlink for vtysh and reference to user creation

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

Add support for collecting AddressSanitizer output. See README.md for details

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

ldp-topo1: Adjust to accept new format in "show ip route" as introduced with PR 495 (extra comma)

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

Fix memory leak detection and reporting which accidentally was dropped a month ago

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

lib: Fix another typo on kernel compare

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

lib: Fix version check in previous kernel compare

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

Actually fix version comparison

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

Spell Verifying correctly

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

Fix tests to handle new output

The show commands for bgp now have a bit more
data associated with labeled unicast routes.

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

lib: Fix kernel version parsing

When we have a kernel sub version > 10 the float conversion
of the kernel version causes 4.10 to be less than 4.5

Get the kernel version in groups on <major>.<minor> and do
comparison that way

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

ospf6-topo1: Update to ignore the proto type in linux kernel table

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

all-protocol-startup: Fix bad assert in case of failed vtysh output test

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

all-protocol-startup: Fix pref commit - bamboo prefix is lower case

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

all-protocol-startup: Add option to skip currently broken checks and relax BGP check

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

ripng-topo1: Fix duplicate IP (Thanks Renato!) and documentation typos

all-protocol-startup: Add test suite which runs all protocols at once and verifies startup and spurious errors of each

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

lib: Add function to check for daemon available (i.e. LDPd) and function to return typ (quagga or frr)

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

Update FreeRangeRouting -> FRRouting name

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

rip_topo1: Adding new test for RIPv2 Topology

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

ripng_topo1: Adding new test for RIPng Topology

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

topotest library: Fix creation of /etc/XXX/vtysh.conf

- Needs to be created in local filesystem of each router

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

bgp_multiview_topo1: Fix case of tests hanging if convergence is not successful.

- Part of a command was missing which caused a hang in cases when BGP didn't converge

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

LDP-Topo1: Update test to accept new format of LDP output

* Added test to check for version
* Adopted all tests to verify against the correct version of output

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

Fix README incorrect reference to Quagga

ldp-topo1: Fix ordering of reference output to check against

Ordering of output is now done by LDP and should be consistent

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

ldp-topo1: Delete accidentally include tar file

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

Update Readme to have correct ordering for frr user

We cannot usermod the frr user until after we
create the user with adduser.

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

Update Readme to reflect FreeRangeRouting name

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

bgp_multiview_topo1: Mask out BGP Table version in comparison

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

Added optional output for memleaks at the end of test. Enable them by setting the environment variable TOPOTESTS_CHECK_STDERR to some value

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

Move common function and class to topotest library

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

ldp-topo1: Add new LDP Topology Test

bgp_multiview_topo1: Fix bad check for running daemons

Only one ruter running in the setup

Add support for FRR in addition to Quagga

change in checkRunning: Check all daemons on all nodes before aborting with an assert

ospf6-topo1: Add Markdown doc with testdescription

bgp_multiview_topo1: Rename test_description.md to README.md for automatic rendering in online git

bgp_multiview_topo1: Add Markdown doc for test

README: Add example configure statement for building test quagga code

bgp_multiview_topo1: Change from unified to context diff for better readability

ospf6-topo1: Add check for Linux Kernel Routing Table as well and change to better human readable context diff

ospf6-topo1: Enable coredumps for Quagga Daemons and add logic to skip remaining test in case of fatal error

bgp_multiview_topo1: Added Network Diagram as comment

bgp_multiview_topo1: Fix limit to create coredumps on quagga crashes

bgp_multiview_topo1: Rename received bgp route log with .log for easy clean between runs

example-test: Add simple example pytest for documentation (and exclude it from running in pytest.ini)

Update Documentation

bgp_multiview_topo1: Skip remaining on fatal error and try to get more crash info of daemon is not running

bgp_multiview_topo1: Add "bgp multiview" simple topology to test. Initial commit

ospf6-topo1: Code cleanup

ospf6-topo1: Less chatty timeout messages during convergence

ospf6-topo1: Cleanup log messages

Add .gitignore

test_ospf6_topo1: Restructure to run under pytest

Renamed Test Directory: ospf6-test1 -> ospf6_topo1

ospf6-test1: Renamed testscript to have test_ prefix for pytest discovery

ospf6-test1: Look for test config in directories relative to main script location

Cleanup README and a reference to License

Initial commit with first OSPFv6 Topology Test

tests: Exclude topotests from consideration

When running make check, exclude the topotests directory from considertion.

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

Merge pull request #3351 from chiragshah6/ospfv3_dev

ospf6d: abr summary advertise best route

Merge pull request #3356 from opensourcerouting/router-id-loopbacks

zebra: improve identification of loopback interfaces in the router-id code

Merge pull request #3361 from opensourcerouting/yang-embed-models

yang: embed models into binaries

Merge pull request #3362 from pacovn/Coverity_1475469_null_check

bgpd: null check (Coverity 1475469)

Merge pull request #3364 from opensourcerouting/bugfix/isis-ipv6-only

IS-IS: Bugfix ipv6 only

Merge pull request #3374 from opensourcerouting/bugfix/vty-shadow

lib/vty: Fix warning about shadowed variable

Merge pull request #3371 from donaldsharp/vtysh_ospf_instance

vtysh: Don't attempt to reconnect the non-instanced ospf process

Merge pull request #3325 from adeg/bugfix/bgpd-vtysh-fixes

bgp_vty: Fix vpn_leak_postchange_all() every time "router bgp ASNUM" context is entered

Merge pull request #3176 from chiragshah6/evpn_dev

zebra: duplicate address detection and dampening

lib/vty: Fix warning about shadowed variable

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

vtysh: Don't attempt to reconnect the non-instanced ospf process

When running ospf instances we should not attempt to reconnect
the default ospf instance on running a command.

This commit should be targeted enough because in the case
of normal operation we connect to everything we should
and only set the VTYSH_WAS_ACTIVE flag for those we
truly have lost connection too.

Before:

donna.cumulusnetworks.com# config t
donna.cumulusnetworks.com(config)# router ospf 100
Warning: connecting to ospfd...failed!
donna.cumulusnetworks.com(config-router)#

After:
donna.cumulusnetworks.com# conf t
donna.cumulusnetworks.com(config)# router ospf 100
donna.cumulusnetworks.com(config-router)# end
donna.cumulusnetworks.com#

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

Merge pull request #3367 from karamalla0406/frr3333

bgpd: BGP daemon crashed when a L2VNI is unconfigured

Merge pull request #3363 from pacovn/static_analysis__ISO_C_return_compliance

bgpd isisd lib: fix return on void functions

Merge pull request #3368 from pacovn/static_analysis__ISO_C_empty_initializer

isisd lib ospfd pbrd python: fix empty init

isisd lib ospfd pbrd python: fix empty init

ISO C forbids empty initializer braces. Empty initializers have been
replaced with {0}

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

bgpd: BGP daemon crashed when a L2VNI is unconfigured

When a VNI is unconfigured it deletes all of its import and export
route-targets.  There is a export route-target link list and import
route-target linked list.  There are redudant loops in  the
route-target deletion code.  In the first iteration it deleted the
route-target and freed the RT node, but not list node.
In the 2nd iteration it tries to free the RT node again, resulting in
 the double free of RT node.

Signed-off-by: "Kishore Aramalla karamallavmware.com"

Merge pull request #3339 from opensourcerouting/bugfix/isis-flooding-issues

isis: Fix flooding issues

isisd: Fix bug in tlvs_protocols_supported_to_adj

We should set nlpids[0] if we only set on protocol,
not nlpids[1].

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

isisd: Consider non-mt links usable when either v4 or v6 works

When we run in non-mt mode, we should consider links which have either
working IPv4 or IPv6 active and look at the neighbors nlpids to judge
wether a link is usable.

Fixes: #3336

Merge pull request #3359 from qlyoung/true-atomics

Restrict atomics to 32-bits only

bgpd isisd lib: fix return on void functions

ISO C forbids ‘return’ with expression, in function returning void.

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

Merge pull request #2929 from kssoman/rtr_fix

bgpd, zebra: Creating Loopback Interface Flaps BGPd, it should update

bgpd: null check (Coverity 1475469)

Null check of 'rn' returned by bgp_node_lookup() because it could be
deferenced afterwards into bgp_nexthop_get_node_info()

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

yang: add .gitignore

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

yang: embed models into binaries

This bakes our YANG models straight into the library/daemons, so they
don't need to be loaded from /usr/share/yang.  This makes the
installation quite a bit more robust, as well as gets us halfway to
running uninstalled.  (The other half is baking in the extension type
module.)

The /usr/share/yang directory is still searched as a fallback, as well
as for the experimental YANG model translator.  This is likely to stay
as is for the time being.

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

tools: add non-32 bit atomic warning to checkpatch

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

*: only use 32-bit atomics

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

Merge pull request #3358 from opensourcerouting/libtool-cfg-warn

build: refuse non-working linking options

Merge pull request #3346 from donaldsharp/pim_possible_issues

Pim possible issues

Merge pull request #3294 from pguibert6WIND/distribute_list_ipv6

lib: distribute-list ipv6 can be (un)configured

Merge pull request #3311 from dslicenc/static-int-up

staticd: install static routes in a vrf when next-hop interface comes up

Merge pull request #2725 from pguibert6WIND/workflow_lts

doc: change workflow to support long term maintenance branches

Merge pull request #3343 from qlyoung/doc-add-vtysh-dev

doc: add dev doc for vtysh

build: refuse non-working linking options

We only support:
* --enable-shared --disable-static --disable-static-bin
* --enable-shared --enable-static --disable-static-bin
* --enable-shared --enable-static --enable-static-bin

(The second option is not particularly useful.)

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

bgpd: Creating Loopback Interface Flaps BGPd (#2865)

* The function bgp_router_id_zebra_bump() will check for active bgp
  peers before chenging the router ID.
  If there are established peers, router ID is not modified
  which prevents the flapping of established peer connection

* Added field in bgp structure to store the count of established peers

Signed-off-by: kssoman <somanks@vmware.com>