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>

Merge pull request #8223 from mjstapp/fix_pim_nb_sa

pimd: fix coverity SA warning in pim_nb_config.c

Merge pull request #8124 from pguibert6WIND/ipsec_iptable_dplane

zebra: move netfilter contexts to zebra dplane

Merge pull request #8177 from Jafaral/nhrp-ev-handler

Merge pull request #8227 from donaldsharp/flex_this

Start work to switch over to using our now built-in printf functionality

tools: add explicit licence phrase

Signed-off-by: Jafar Al-Gharaibeh <jafar@atcorp.com>

tools: example bash script client for the nhrpd event sock

Signed-off-by: Joe Maimon <jmaimon@jmaimon.com>

nhrp: enables the event system to interpret client results correctly

Signed-off-by: Joe Maimon <jmaimon@jmaimon.com>

zebra: move ipset and ipset_entry to zebra dplane contexts

like it has been done for iptable contexts, a zebra dplane context is
created for each ipset/ipset entry event. The zebra_dplane_ctx job is
then enqueued and processed by separate thread. Like it has been done
for zebra_pbr_iptable context, the ipset and ipset entry contexts are
encapsulated into an union of structures in zebra_dplane_ctx.

There is a specificity in that when storing ipset_entry structure, there
was a backpointer pointer to the ipset structure that is necessary
to get some complementary information before calling the hook. The
proposal is to use an ipset_entry_info structure next to the ipset_entry,
in the zebra_dplane context. That information is used for ipset_entry
processing. The ipset name and the ipset type are the only fields
 necessary.

Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

nhrpd: Use our built-in printf functionality

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

tests: Replace ospf-sr-topo1 by ospf-sr-topo2

The new ospf-sr-topo2 tests are much broader and detailed,
hence it makes no sense to keep the old ospf-sr-topo1
tests.

Signed-off-by: GalaxyGorilla <sascha@netdef.org>

Merge pull request #8224 from idryzhov/fix-bfd-detect-to

bfdd: fix detect timeout

tests: OSPF SR topotests (ospf-sr-topo2)

Signed-off-by: GalaxyGorilla <sascha@netdef.org>

isisd, yang, doc: combine config cmd for SRGB+SRLB

when changing both ranges at the same time the order of the commands
matters, as we need to make sure that the intermediate state is valid.
This represents a problem when pushing configuration via frr-reload.
To fix this, the global-block command was extended to optionally
allow setting the local-block range as well. The local-block command
is deprecated with a 1-year notice.

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

Merge pull request #8221 from mjstapp/fix_thread_xref_event

lib: use correct type in add_event xref

ripd: Convert to using internal printf formatters

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

ripngd: Convet to using internal printf modifiers

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

watchfrr: Convert to not use warning in warning messages

We do not need to display: `Warning: ...` in a zlog_warn
message

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

ospfd: Convert to using proper string formatting

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