nhrpd: Remove unecessary new line from debug messages

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

nhrpd: Use %pSU instead of sockunion2str

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

nhrpd: Fix some style issues

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

nhrpd: Set prefix length in NAT extension in resolution-reply

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

nhrpd: Ensure invalid NAT extension info cannot be used

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

nhrpd: Use SU_ADDRSTRLEN where appropriate

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

nhrpd: populate correct prefix length in NAT extension in Reg-Reply

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

nhrpd: Add Claimed NBMA field in sh ip nhrp cache output

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

nhrpd: add NHS details in NAT Extension header

Hub uses this to check if is itself behind a NAT

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

nhrpd: copy NAT extension if present else attempt to populate

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

nhrpd: parse multiple CIEs in NAT extension header

Cisco devices send can send multiple CIEs so we must search for the correct CIE

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

nhrpd: Add empty NAT extension header for Non Natted Spoke in Resolution-Reply

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

nhrpd: Set correct MTU in NHRP extensions

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

nhrpd: Send prefix length in NAT extension the same as is done by Cisco

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

nhrpd: Add CIE to NAT extension in resolution requests

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

nhrpd: Add NAT extension in forwarded packets

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

nhrpd: Process NAT extension properly, and also fallback to IPSec NAT info

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

nhrpd: Add zbuf routine to peek at contents

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

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

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>

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>