Merge pull request #8282 from volta-networks/fix_isisd_bfd_ipv4

isisd: fix BFD session when IPv6 not configured

Merge pull request #8291 from volta-networks/isis_timer_unstable

isisd: avoid lsp_sched loop when unstable

Merge pull request #8129 from mjstapp/backup_recursives

zebra: modify backup nexthop handling for recursive routes

isisd: avoid lsp_sched loop when unstable

no point in scheduling an LSP refresh immediately if we know it is
going to be postponed again due to the network still being in its
instability grace period

Signed-off-by: Emanuele Di Pascale <emanuele@voltanet.io>

Merge pull request #8286 from donaldsharp/nhrp_xlref

nhrpd: Remove newline from log message

Merge pull request #8263 from opensourcerouting/docker-update

tests: docker fixes & improvements

Merge pull request #8283 from chiragshah6/mdev

bgpd: add seqno in bgp as-path access-list policy

nhrpd: Remove newline from log message

We should not be putting new lines in log messages.

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

isisd: add debug logs to troubleshoot BFD issues

when we receive an event from BFDD and we end up throwing it away,
make sure that we log (with debug guards) the reason for this, so
we can troubleshoot issues like the one addressed by the previous
commit.

Signed-off-by: Emanuele Di Pascale <emanuele@voltanet.io>

isisd: fix BFD session when IPv6 not configured

A wrong check was silently skipping the initialization of the bfd_session
struct in the adjacency if the router was not configured for IPv6. This
would cause BFD events to be ignored regardless of the configuration.

Also add a function to return the "name" of an adjacency and use it in a
couple of places, including the new log, instead of repeating the same
code in a bunch of places.

Signed-off-by: Emanuele Di Pascale <emanuele@voltanet.io>

Merge pull request #8284 from mjstapp/fix_bgp_zero_timers

bgpd: use add_event instead of add_timer with zero timeout

doc: add seqno to bgp as path list section

Add sequence no option to bgp as-path list cli syntax.
Add sequence no to example config.
Add auto generated sequence no in running-config if its not
provided in config.

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

bgpd: add seqno in bgp as-path access-list policy

Add optional parameter seqno to bgp as-patch policy
to inline with other bgp policies commands at same
level.

Testing Done:

config t
bgp as-path access-list foo1 permit _40_
bgp as-path access-list foo1 seq 5 permit _41_
bgp as-path access-list foo1 seq 8 permit _42_
bgp as-path access-list foo2 permit _43_
no bgp as-path access-list foo1 permit _41_
bgp as-path access-list foo1 permit 41$

show run:
bgp as-path access-list foo1 seq 8 permit _42_
bgp as-path access-list foo1 seq 10 permit 41$
bgp as-path access-list foo2 seq 5 permit _43_

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

bgpd: use add_event instead of add_timer with zero timeout

Just use events in a few places where timers with zero timeout
were being used.

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

Merge pull request #8052 from ton31337/feature/show_bgp_peer-groups_json

Merge pull request #8232 from reubendowle/fixes/nhrp-ipsec-issues

nhrp: Fix some ipsec related issues

Merge pull request #8255 from pjdruddy/fix-ifindex-test

Fix ifindex test

Merge pull request #8248 from volta-networks/fix_ospf6_chunk_msgs

ospfd6: Clear locks when ospf messages need to be chunked

tests: simplify docker rsync & support worktrees

The previous, more complex mechanism failed to take into account that
git worktrees only have a stub .git file & copying the worktree itself
is not enough.  Just extract a file list beforehand & don't bother with
git inside the container.

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

tests: add libelf-dev to Dockerfile

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

Merge pull request #8276 from opensourcerouting/pathd-fmt-fix

pathd: fix some format strings

Merge pull request #8008 from chiragshah6/yang_nb5

[lib, yang] : multiple fixes in prefix-list northbound conversion

Merge pull request #8225 from idryzhov/bfd-echo-mode

bfdd: fix echo receive timer and disable echo mode

Merge pull request #8229 from idryzhov/bfdd-echo-rx-tx

bfdd: separate echo rx/tx timers

Merge pull request #8273 from volta-networks/fix_ospf6_coverity

ospf6d: Fix coverity errors

Merge pull request #8275 from donaldsharp/peer_afi_safi_mistake

pathd: remove mid-string line breaks

cf. workflow.rst ("lines over 80 characters are allowed for text strings
to make it possible to search the code for them"), matching Linux kernel
coding style.

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

pathd: use %pIA to print struct ipaddr *

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

pathd: fix %pI4 <> %pI6 snafu

... as noted by the frr-format GCC plugin.

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

nhrpd: When IPsec profile is added or deleted, reset the vc and re-check peer

Signed-off-by: Reuben Dowle <reuben.dowle@4rf.com>

nhrpd: Make comments clearer, fix style issues

Signed-off-by: Reuben Dowle <reuben.dowle@4rf.com>

nhrpd: Close IPSec connection when tunnel protection removed

Signed-off-by: Reuben Dowle <reuben.dowle@4rf.com>

nhrpd: Retry IPSec if NHRP is repeatedly failing

This prevents a failed IPSec connection from preventing DMVPN from working.
A failure situation can be reproduced using a Cisco peer, and and disabling then
re-enabling the tunnel IPSec protection (after the IPSec connection has
already been established).

Signed-off-by: Reuben Dowle <reuben.dowle@4rf.com>

bgpd: If we have a SAFI conflict do not allow labeled unicast to reset

If we have a SAFI conflict, ie we are trying to activate safi's
UNICAST and LABELED_UNICAST at the same time, we should not
cause bestpath to be rerun and we should not try to put
labels on everything.

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

bgpd: Attempting to activate unicast and labeled-unicast

Should return an actual useful error message.
Commit: 055679e915fc826fe13abdd8d60a22a0ca35c8c6 messed this error message
up.

Fixes: #8246
Signed-off-by: Donald Sharp <sharpd@nvidia.com>

Merge pull request #8258 from volta-networks/feat_ldp_snmp_with_stats

ldpd: Add support for the read-only snmp mib objects that are statistics

Merge pull request #8268 from mjstapp/doc_dev_topo_notes

doc: add info to topotest dev doc

Merge pull request #8270 from volta-networks/fix_isis_snmp_coverity

isisd: Fix coverity warnings

ospf6d: Fix coverity errors

Signed-off-by: Lynne Morrison <lynne@voltanet.io>

bfdd: separate echo rx/tx timers

Currently there is a single interval for both RX and TX echo functions.
This commit introduces separate RX and TX timers for echo packets.

The main advantage is to be able to set the receive interval to zero
when we don't want to receive echo packets from the remote system.

Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>

Merge pull request #8266 from idryzhov/eigrp-cli-fixes

eigrp cli fixes

Merge pull request #7963 from volta-networks/feat_pceplib_into_frr_github

pceplib: Integrate pcelib into frr

Merge pull request #8155 from idryzhov/zebra-support-bundle

zebra support bundle fixes and improvements

isisd: Fix coverity warnings

Signed-off-by: Lynne Morrison <lynne@voltanet.io>

zebra: capture backup nexthop info with recursive resolution

When resolving a recursive route, capture backup nexthop info
along with the resolving nexthops.

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

zebra: add ui control for use of backup nexthops in resolution

Add a control and api for the use of backup nexthops in
recursive resolution. With 'no', we won't try to use installed
backup nexthops when resolving a recursive route.

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

Merge pull request #7993 from mjstapp/reorg_resolve

zebra: reorg nexthop resolution code

ospfd6: Clear locks when ospf messages need to be chunked

When there are too many LSA updates to be sent in a packet the code needs to
correctly clear the locks that are taken while walking the lists and then wait
for the appropriate timer to expire to continue walking the list.   The routine
that was being called would not properly remove all the locks that needed to be
cleared, and would also try to incorrectly delete the lsa/route.   The code now
clears the locks and leaves the lsa on the list.  When the timers fire again
the code continues walking the list to send the rest of the lsas to the
neighbor.

Signed-off-by: Lynne Morrison <lynne@voltanet.io>

Merge pull request #8226 from adharkar/frr-evi_rx

bgpd: Reevaluate es_evi_vtep active state on disable-ead-evi-rx config flap

tests: re-enable RTE tests after removing fixed ifindices

Change ifindex test to grab the correct indices using the API
provided.

Signed-off-by: Pat Ruddy <pat@voltanet.io>

tests: fix formatting error to make black clean

Signed-off-by: Pat Ruddy <pat@voltanet.io>

doc: add more notes about new topotests

Add some extra notes to the dev doc section about writing new
topotests: check for OS/kernel support if necessary; avoid
volatile or unstable data like ifindices or link-locals.

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

doc: fix warnings in dev docs

Fix a couple of warnings in dev doc rst files.

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

eigrpd: fix dependency on operational state in cli

Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>

eigrpd: fix xpaths

Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>

bfdd: warn users about echo-mode restrictions

Echo-mode implementation is currently broken. Instead of sending packets
to it's own address, bfdd is sending echo packets to the peer's address.
It may seem to work when testing between two FRR instances, because FRR
loops back such packets, but no other implementation is supposed to do
that.

Let's warn users that the current implementation works only between two
FRR instances.

Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>

bfdd: fix starting echo receive timer

Currently this timer is only started when we receive the first echo
packet. If we never receive the packet, the timer is never started and
the user falsely assumes that echo function is working.

Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>

tests: update LDP SNMP topotest to walk mib objects that are statistics

Signed-off-by: Karen Schoener <karen@voltanet.io>

ldpd: Add support for the read-only snmp mib objects that are statistics

Add support for the read-only snmp mib objects as described in RFC 3815
that are statistics.

Signed-off-by: Lynne Morrison <lynne@voltanet.io>
Signed-off-by: Karen Schoener <karen@voltanet.io>

Merge pull request #8260 from volta-networks/fix_isis_snmp_test

test: fix isis-snmp test

test: fix isis-snmp test

Remove ifindex values being tested because it can change
between runs of the test.

Signed-off-by: Lynne Morrison <lynne@voltanet.io>

Merge pull request #8214 from chiragshah6/mdev

tools: frr-reload refine error handling

Merge pull request #8254 from donaldsharp/various_fixes

pimd: Fix clang issue with -Waddress-of-packed-member

Recent change in commit: 6b73800ba27e7f69453a191dfb873c2e07194516

Caused this error to pop up in pim_igmp_mtrace.c:

error: taking address of packed member 'rsp_addr' of class or structure 'igmp_mtrace' may result in an unaligned pointer value [-Werror,-Waddress-of-packed-member]

Follow the pattern used in the code to solve this problem for clang

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

bgpd: use appropriate printf formatter for some uint32_t

newm and existm are uint32_t so let's use %u instead of %d
to print them out.

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

Merge pull request #8234 from reubendowle/fixes/nhrp-trafic-indication

nhrpd: Set hop count to 1 in traffic indication

Merge pull request #8230 from donaldsharp/flex_more

more switchover to using our builtin printf functionality

Merge pull request #8220 from mjstapp/bgp_notify_race

bgpd: handle socket read errors in the main pthread

Merge pull request #8242 from opensourcerouting/format-warning-fix

bgpd: fix wrong debug format

Merge pull request #8231 from idryzhov/bfd-admin-down

bfdd: make sessions administratively up by default

Merge pull request #8147 from volta-networks/feat_ldp_snmp_without_stats

ldpd: Add support for read-only snmp mib objects (excluding statistics)

Merge pull request #7945 from volta-networks/feat_isis_snmp

isisd: add support for read-only snmp mibs objects

Merge pull request #8160 from idryzhov/fix-show-yang-oper

lib: fix crash when iterating over nb operational data

*: remove remaining severity prefixes

Having a "warning:" prefix on a debug message is particularly dumb...

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

zebra: move up prefix2str call in rib dump

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

zebra: use printfrr exts in EVPN/VXLAN code

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

fpm: use printfrr exts

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

*: fix printf extension types

Some mistakes have crept in again.

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

lib: Convert to builtin printf functionality

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

ospf6d: Convert to use builtin printf functionality

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

eigrpd: Convert to using our builtin printf stuff

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

pimd: Use builtin printf functionality

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

babeld: We do not need to put Warning into our warning messages

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

sharpd: Convert over to using builtin printf functionality

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

bfdd: make sessions administratively up by default

Current behavior is inconsistent. When the session is created by another
daemon, it is up by default. When we later configure peer in bfdd, the
session is still up, but the NB layer thinks that it is down.

More than that, even when the session is created in bfdd using peer
command, it is created in DOWN state, not ADM_DOWN. And it actually
starts sending and receiving packets. The sessions is marked with
SHUTDOWN flag only when we try to reconfigure some parameter. This
behavior is also very unexpected.

Fixes #7780.

Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>

nhrpd: Set hop count to 1 in traffic indication

If hop count is 0, this causes Cisco routers to reject the traffic indication
as invalid. This appears to be a Cisco bug, and has been observed in processing
of registration packets in the past. That problem was covered in issue #951

Signed-off-by: Reuben Dowle <reuben.dowle@4rf.com>

doc: Add `show [ip] bgp peer-group [json]` command

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

bgpd: Add JSON output for `show ip bgp peer-group` command

```
exit1-debian-9# show bgp peer-group

BGP peer-group test, remote AS 123
  Peer-group type is external
  Configured address-families: IPv4 Unicast; IPv6 Unicast;
  1 IPv4 listen range(s)
    192.168.100.0/24
  2 IPv6 listen range(s)
    2001:db8:1::/64
    2001:db8:2::/64
  Peer-group members:
    192.168.200.1  Active
    2001:db8::1  Active

BGP peer-group test3
  Peer-group type is external
  Configured address-families: IPv4 Unicast;
exit1-debian-9#
exit1-debian-9# show bgp peer-group json
{
  "test":{
    "remoteAs":123,
    "type":"external",
    "addressFamiliesConfigured":[
      "IPv4 Unicast",
      "IPv6 Unicast"
    ],
    "dynamicRanges":{
      "IPv4":{
        "count":1,
        "ranges":[
          "192.168.100.0\/24"
        ]
      },
      "IPv6":{
        "count":2,
        "ranges":[
          "2001:db8:1::\/64",
          "2001:db8:2::\/64"
        ]
      }
    },
    "members":{
      "192.168.200.1":{
        "status":"Active"
      },
      "2001:db8::1":{
        "status":"Active"
      }
    }
  },
  "test3":{
    "type":"external",
    "addressFamiliesConfigured":[
      "IPv4 Unicast"
    ]
  }
}
```

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

doc: update the FRR format table

Let other developers know that we have `%pIA` to print `struct ipaddr *`.

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

bgpd: fix wrong debug format

Use `%pIA` instead of `%pI4` for `struct ipaddr *`.

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

Merge pull request #8244 from opensourcerouting/ci-fix-tests

topotests: fix some recent test failures

topotests: temporary disable test assert

The test has been failing often recently and it is causing some false
positives for unrelated PRs.

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

topotests: remove some interface index check

Don't expect the interface index to be always the same.

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

topotests: remove some FRR version checks

We version the tests with the source code so we should no longer attempt
to support old versions in development branch.

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

Merge pull request #8085 from volta-networks/isis_label_blocks

isisd: configure SRGB and SRLB together

Merge pull request #7827 from GalaxyGorilla/ospf_sr_topotests

tests: enhance OSPF SR topotests

Merge pull request #8241 from Jafaral/first_p

Revert "zebra: Remove `first_p` which is never used"

This reverts commit 8617eb7c5ff9fe9ae1efb69af1493585d69a37f5.

Merge pull request #8239 from donaldsharp/first_p

zebra: Remove `first_p` which is never used

zebra: Remove `first_p` which is never used

Remove dead code.

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