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>

zebra: dup addr detect mac consolidation

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

zebra: dup addr detect neigh consolidation

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

bgp, zebra: address review comments

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

zebra: dup addr detect add fields to show evpn cmd

TOR# show evpn
L2 VNIs: 5
L3 VNIs: 0
Advertise gateway mac-ip: No
Duplicate address detection: Enable
  Detection max-moves 5, time 180

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

zebra: dup addr detect fix warnings

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

zebra: dup addr detect clear command

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

zebra: dup addr detect operation commands

Display duplicate detected list of MACs and VNIs.
- Per VNI
- Across all VNIs

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

lib, zebra: dup addr detect display detection fields

Display following Per MAC and Neigh's output:
If duplicate address detection is under process,
display detection start time and detection count.
If duplicate address detection detected an address
as duplicate, display detection time and duplicate
status.

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

zebra: dup addr detect freeze action

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

zebra: dup addr detect warn-only

Duplicate address detection warning only action
upon an address detected as duplicate.

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

zebra: dup addr detect zapi changes

EVPN Duplicate Address Detection (DAD)
zapi information

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

bgpd: dup addr detect config cli

Duplicate address detection configuration clis
under bgp l2vpn evpn config mode.
- Enabled/Disable (global knob) for feature.
- Configure cli for duplicate detection action
freeze and freze until time (auto-recovery).

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

bgpd: dup addr detect data struct for cfg

Enable/disable duplicate address detection
there are 3 actions
warning-only: Default action which generates
only frr warning (syslog) to user for any
duplicate detecton
freeze: Permanently freezes address, manual
intervene required.
freeze with time: An address will recover once
the time has expired (auto-recovery).

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

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

The if_is_loopback() function is the right abstraction for identifying
loopback interfaces. There should be no reason for not using it in the
router-id code.

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

Merge pull request #3341 from donaldsharp/unneeded

zebra: Remove uncompiled file

Merge pull request #3352 from pacovn/Coverity_1475489_resource_leak

bgpd: fix resource leak (Coverity 1475489)

Merge pull request #3353 from qlyoung/fix-bgpd-community-list-deletion-nit

bgpd: fix small error in community-list patch

Merge pull request #3338 from ton31337/fix/optional_args_for_exclude

bgpd: Optionally remove AS number when using `no set as-path exclude`

Merge pull request #3344 from ton31337/fix/optional_args_for_community-lists

bgpd: Remove community-list by name without typing full rule

ospf6d: abr summary advertise best route

ABR summary should contain best intra prefix LSA.
There might be a case where intra-prefix route with different
cost from different advertising router present, summary
advertisement should only consider best intra-prefix route.

A route which is marked for remove, while purging the route's
ABR summary LSA also remove summary route from the summary
table.

Ticket:CM-22665
Testing Done:
Validate on ABR topology, intra-prefix route with
different cost, a higher cost intra-prefix route skip from sending
ABR summary LSA.

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

bgpd: fix resource leak (Coverity 1475489)

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

Merge pull request #3348 from donaldsharp/bsd_route_install

zebra: Fix bsd privs elevation

zebra: Fix privs elevation

This Commit: f183e380fae61b7c1f89fed6e32ed5a9d1ede8a8 broke
priviledge escalation on *bsd.  This puts it back.

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

pimd: When we fail to initiate ifchannel backout work done

When we receive a igmp report and attempt to initiate
a pim ifchannel for it and that fails to work then
let's back out the work done setting stuff up to this
point.

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

pimd: Add some debugs for when local membership fails

When we fail to add a local membership add some additional debugs
so that we can have a bit more information on when something goes
bad.

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

bgpd: Remove community-list by name without typing full rule

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

bgpd: Optionally remove AS number when using `no set as-path exclude`

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

zebra: Remove uncompiled file

The zebra_l2_null.c is not compiled and not needed, remove.

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

isisd: Fix behavior on reception of self-originated LSP

We should only update and reflood our own LSPs when the received LSP
is newer than the local copy.

In all other cases, we should simply acknowledge it or resend our own
LSP.

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

fabricd: Fix flooding bug

Due to `lsp` getting shadowed, we would send each T0 its own LSP
whenever we actually wanted to flood a different LSP.

Fix this and set -Wshadow=local in my build environment. m(

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

isisd: Always consider purges as newer

When receiving an LSP with same sequence number but different
checksum as in the local database, we would always treat it as
newer than the local LSP.

That behavior is incorrect if the local LSP is indeed a purged
LSP waiting for age-out and the received one is not.

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

Merge pull request #3326 from qlyoung/fix-lla-reinstallation

zebra: force neighbor entry reinstallation

Merge pull request #3316 from qlyoung/remove-pr-admonishment

.github: remove style reminder from PR template

Merge pull request #3323 from qlyoung/doc-zebra-protocol

doc: update zebra protocol documentation

Merge pull request #3329 from rubenk/autoconf-cleanup

configure.ac: consistently quote all m4 macros

Merge pull request #3331 from mjstapp/fix_lib_id_warning

libs: rename two id_alloc macros to resolve bsd conflict

Merge pull request #3332 from qlyoung/fix-code-block-err

doc: use correct specifier for code block

Merge pull request #3164 from pguibert6WIND/bgpstandalone_importexport

bgpd: allow vrf validity and bgp vrf import/export, when zebra is off

libs: rename two id_alloc macros to resolve bsd conflict

Two of the macros in lib/id_alloc had conflicts on some platforms;
rename them to be unique.

Signed-off-by: Mark Stapp <mjs@voltanet.io>

configure.ac: consistently quote all m4 macros

Signed-off-by: Ruben Kerkhof <ruben@rubenkerkhof.com>