configure.ac: Correct library name for sysrepo

Northbound_sysrepo: Correct sysrepo library name in configure.ac

Signed-off-by: Bo Zhang <logbob0401@gmail.com>

Merge pull request #7878 from opensourcerouting/ldpd-metric-issue

ldpd: fix sporadic failures in the ldp-topo1 topotest

Merge pull request #7875 from deastoe/fpm-nlmsg-pid

zebra: set nlmsg_pid in netlink msgs sent by 'fpm'

ldpd: fix sporadic failures in the ldp-topo1 topotest

Commit 220e848cc5 introduced an optimization that would prevent ldpd
from sending redundant label mappings when it receives notifications
from zebra about routes that didn't effectively change (such
notifications can happen under certain circumstances).

The problem is that that commit didn't take into account the metric
of the received routes, so it would dismiss a notification of a
route with a better metric taking the place of another route in the
RIB, preventing the newly selected route from receiving the label
mappings it needs.

Revert 220e848cc5 temporarily to fix sporadic failures in the CI
system until we have a better solution.

Debugged-by: Lynne Morrison lynne@voltanet.io
Signed-off-by: Renato Westphal <renato@opensourcerouting.org>

zebra: set nlmsg_pid in netlink msgs sent by 'fpm'

Use nl_pid from the netlink socket used for programming the kernel
(netlink_dplane) in netlink route messages sent by the 'fpm' module.

This makes 'fpm' consistent with 'dplane_fpm_nl' which already
behaves this way, and allows FPM server implementations to determine
route origin via nlmsg_pid.

Signed-off-by: Duncan Eastoe <duncan.eastoe@att.com>

Merge pull request #7844 from vishaldhingra/static

staticd: correct table-id handling for static routes.

Merge pull request #7870 from donaldsharp/pytest_marking

tests: Start the ability to mark tests

Merge pull request #7872 from ton31337/feature/topotests_for_peer-groups

tests: Check if peer-group is working in topotests

Merge pull request #7868 from mjstapp/fix_fpm_conn_up

zebra: don't set connection-up event pointer directly

tests: Check if peer-group is working in topotests

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

Merge pull request #7867 from dewi-morgan/bgp-max-prefix-overflow-clear

bgpd: clear max prefix overflow on de-config

Merge pull request #7869 from donaldsharp/peer_group

bgpd: Temp fix to allow numbered peers to be part of a peer group

tests: Start the ability to mark tests

Add the ability for our topotests to take advantage of pytest `mark`ing.
This effectively allows you to tell pytest to run against certain sets
of tests.  For a demonstration purpose I've added in marks for:
babel
eigrp
ldp
ospf
pim
rip

And setup tests to run against those tests that only test those protocols.

You can run against eigrp tests by running `pytest -k eigrp`
Other combinations are also available based upon simple boolean logic.
Just read the pytest.mark documentation.

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

bgpd: Temp fix to allow numbered peers to be part of a peer group

Talking w/ Chirag and he indicated that we can just backout the command
to the original and things would `work` and they do( at least a quick test does )

Put this in place until a proper fix can be done.

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

Merge pull request #7866 from kishorekunal01/fpm_dump_issue

zebra: Scale setup RMAC is send multiple time to fpm

zebra: don't set connection-up event pointer directly

Use thread_cancel to reset the connection-up processing timer.

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

Merge pull request #7821 from donaldsharp/evpn_flag_dump

Evpn flag dump

zebra: Scale setup RMAC is send multiple time to fpm

Thread zfpm_conn_up_thread_cb can Yield and send RMAC multiple
times to FPM.

Signed-off-by: Kishore Kunal <kishorekunal01@broadcom.com>

Merge pull request #7712 from dslicenc/frr-reload-prefix-list-acl

tools: ignore missing seq nums in prefix and access lists in frr-relo…

bgpd: clear max prefix overflow on de-config

A bgp neighbor remains in Idle state in the event that the number
of received prefixes exceeds the configured maximum prefix for the
neighbor. The neighbor remains in idle state even after de-configuring
the maximum prefix limit for the neighbor.

The fix is to clear the neighbor overflow state if set, after
de-configuring the neighbor maximum-prefix commnd.

This allows the neighbor to establish without having to perform a
clear operation. It also avoids the misleading neigbor summary
indicating that the neighbor is in prefix overflow state (PfxCt)
when no limit is configured for the neighbor.

Signed-off-by: Dewi Morgan <dewi.morgan@intl.att.com>

Merge pull request #7746 from donaldsharp/eigrp_cleanup

Eigrp cleanup

Merge pull request #7837 from ckishimo/tests_ospf

tests: fix redistribute key in ospf tests

zebra: in zebra_evpn_mac.c use size_t for buffer length

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

zebra: Create a dump function for mac->flags and use it

Create a function that can dump the mac->flags in human readable
output and convert all debugs to use it.

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

zebra: Create function to dump MACIP flags

Create a function to dump MACIP flags and to use it.

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

Merge pull request #7819 from donaldsharp/more_data_for_debug_dumps

zebra: Add ability to display human readable format re->flags and status

Merge pull request #7818 from donaldsharp/ip_proto_denied

zebra: notify installing protocol when nexthops cannot be resolved

zebra: Add ability to display in human readable format re->flags and status

The re->flags and re->status in debugs were being dumped as hex values.
I can never quickly decode this.  Here is an idea.  Let's let FRR do
it for me.

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

Merge pull request #6853 from mjstapp/fix_rib_dups

zebra: reduce impact of route-update overload

tools: ignore missing seq nums in prefix and access lists in frr-reload.py

If frr.conf contains a prefix-list or access-list without a seq number,
frr-reload needs to be aware that it should not delete/add if the running
config contains a seq number.

Ticket: CM-32623
Signed-off-by: Don Slice <dslice@nvidia.com>

tools: apply black formatting changes to frr-reload.py

Since new workflow instructions state to run black against
python change and it found formatting changes required that
were not part of my change set, committing those changes
separately.

Signed-off-by: Don Slice <dslice@nvidia.com>

Merge pull request #7849 from gpnaveen/static_rte_test

tests: Static route cases with admin dist and tag.

Merge pull request #7861 from mjstapp/fix_dev_doc_index_no

doc: fix several issues, remind about 'no' in cli index statements

Merge pull request #7748 from adrianomarto/bgp-listen-on-multiple-addresses

BGP daemon listen for connections on multiple addresses

doc: fix sphinx warnings

Fix a couple of sphinx source processing warnings.

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

doc: add missing files to index.rst and subdir.am

Some missing doc files, add them.

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

doc: remind not to include 'no' in cli doc index statements

Remind folks not to include the 'no' in cli doc index lines.

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

Merge pull request #7707 from opensourcerouting/isisd-rlfa

isisd, ldpd: add Remote LFA support

Merge pull request #7439 from opensourcerouting/bgp-damp-profile2

bgpd: BGP route-flap dampening profiles for peers and peer groups

doc: remove 'no' from doc 'index' lines

Index based on the cli commands, not the possible 'no' form.

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

Merge pull request #7247 from ton31337/feature/enhanced_route_refresh_capability

bgpd: Enhanced Route Refresh capability

Merge pull request #7860 from mjstapp/fix_exec_perms

libs,nhrpd: remove exec perm

Merge pull request #7733 from rgirada/ospf_json

ospfd: Adding authentication details to ospf interafce command o/p

staticd: Backend cofiguration code to fix table-id problem

problem: table-id gets overwritten for a given route.

RCA: table-id was getting overwritten from the NB layer,
     So route was getting installed with the latest table-id.

Fix: make the table-id as the key in the NB layer.
     This will program the route in zebra correctly.

- Removed the table-id modify callbacks.
- Moved the validate and apply table-id changes to path-list creation

issue #7347

Signed-off-by: vishaldhingra <vdhingra@vmware.com>

Merge pull request #7841 from volta-networks/fix_isis_mstyle_routes

isisd: ignore routes with incompatible metric style

Merge pull request #7850 from mjstapp/build_dplane_plugin

zebra: build the sample dataplane plugin

libs,nhrpd: remove exec perm

Some source files got the exec bit set in a recent commit - undo
that.

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

Merge pull request #7687 from gpnaveen/ospf_p2mp

tests: Adding ospf p2mp basic test.

eigrpd: Cleanup eigrp_packet.h to our proper formatting

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

eigrpd: Introduce eigrp_metric.[ch] and eigrp_types.h

Add these file and introduce some new fundamental types

Signed-off-by: Donald Sharp <sharpd@nvidia.com>
Signed-off-by: Donnie Savage <diivious@hotmail.com>

eigrpd: Rename nexthop_entry and prefix_entry to better names

Rename struct eigrp_nexthop_entry to struct eigrp_route_descriptor
Rename struct eigrp_prefix_entry to struct eigrp_prefix_descriptor

Fix all the associated function names as well.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>
Signed-off-by: Donnie SAvage <diivious@hotmail.com>

eigrpd: Create eigrp_yang.h and move stuff around

Move the prototypes for eigrp_yang that are in eigrpd.h
into eigrp_yang.h.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>
Signed-off-by: Donnie Savage <diivious@hotmail.com>

eigrpd: Create eigrp_cli.h and move forward declarations

Move the forward declarations for eigrp_cli.c from eigrpd.h
into eigrp_cli.h.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>
Signed-off-by: Donnie Savage <diivious@hotmail.com>

eigrpd: Move `struct eigrp_master` to eigrpd.h

The `struct eigrp_master` data structure belongs in the
eigrpd.h file.  Move it over, in addition start forward
thinking about the different TLV versions so we can
track them better in the future.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>
Signed-off-by: Donnie Savage <diivious@hotmail.com>

Merge pull request #7670 from ton31337/fix/bgpd_nexhop_handling_for_zebra

bgpd: Handle IPv6 prefixes with IPv4 nexthops for zebra

tests: Static route cases with admin dist and tag.

1. Adding api to verify ip nht command.
2. 5 cases of static routes with admin dist and tag
3. Run time = 89Secs

Signed-off-by: nguggarigoud <nguggarigoud@vmware.com>

tests: move redistribute function into lib

Signed-off-by: ckishimo <carles.kishimoto@gmail.com>

tests: fix ospf cosmetic output

Signed-off-by: ckishimo <carles.kishimoto@gmail.com>

tests: remove old def for redistribution in ospf

Signed-off-by: ckishimo <carles.kishimoto@gmail.com>

tests: use new def for redistribution in ospf

Signed-off-by: ckishimo <carles.kishimoto@gmail.com>

tests: add new def for redistribution in ospf

Signed-off-by: ckishimo <carles.kishimoto@gmail.com>

tests: Adding ospf p2mp basic test.

This test verifies basic p2mp test

Signed-off-by: nguggarigoud <nguggarigoud@vmware.com>

zebra: add the sample dataplane plugin to the build

Build the sample dataplane plugin with debug/dev builds.

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

zebra: remove unused local in dplane sample plugin

Remove an unused local in the sample dataplane plugin.

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

zebra: notify installing protocol when nexthops cannot be resolved

In the case where a routes nexthops cannot be resolved as part
of route processing, immmediately notify the upper level protocol
that their routes failed to install if they are interested in
being informed about this issue.

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

Merge pull request #7828 from mobash-rasool/pim-fixes-3

pimd: SPT-bit is not set to false as per RFC section in one flow

Merge pull request #7823 from donaldsharp/zebra_delay_timer

Zebra delay timer

Merge pull request #7847 from opensourcerouting/fix/skip-sr-te-topotest

pathd: Skip SR-TE topotests if pathd has been disabled

Merge pull request #7830 from volta-networks/misc_fixes_2021

Misc fixes (bgpd, pathd, tools)

Merge pull request #7845 from cgoncalves/fix-broken-links

Fix BGP and resource certification reference links

pathd: Skip SR-TE topotests if pathd has been disabled

 * If pathd binary is not found, skip the SR-TE topotests.
 * Fix some compilation warnings when pathd is not built.

Signed-off-by: Sebastien Merle <sebastien@netdef.org>

doc: fix reference links

The current securing BGP and resource certification reference links lead
to a page not found. This patch fixes that by pointing to their
corresponding PDF format resources.

Signed-off-by: Carlos Goncalves <cgoncalves@redhat.com>

bgpd: fix removal of rd vpn export

a missing '!' operator meant that correct rd strings were being
rejected in the destroy callback, making it impossible to remove
once configured.

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

bgpd: fix unsetting of multipath as-set

in the nortbound callback for the boolean leaf multi-path-as-set, the
actual value of the leaf was being ignored. As a result, going from
as-set to no-as-set was having no effect.

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

tests: tests for bgpd listing on multiple addresses

* Added a new topotest to test bgpd listening on multiple addresses.
* Updated the existing bgpd tests according to the parameter added to
bgp_master_init.

Signed-off-by: "Adriano Marto Reis" <adrianomarto@gmail.com>

doc: Explaining that bgpd can listen on multiple addresses

Updated the documentation clarifying that multiple addresses can be
specifyed via -l option.

Signed-off-by: "Adriano Marto Reis" <adrianomarto@gmail.com>

bgpd: bgpd listening on multiple addresses

Changed bgpd so multiple IP addresses can be specified via -l option.

Signed-off-by: "Adriano Marto Reis" <adrianomarto@gmail.com>

Merge pull request #7810 from imzyxwvu/no-deforig

bgpd: Fix default-originate clearing from peer-groups.

pimd: SPT-bit is not set to false as per RFC section in one flow

1. As per RFC 4601 Sec 4.5.7:
* JoinDesired(S,G) -> False, set SPTbit to false.

2. Change the debug type.

Signed-off-by: Mobashshera Rasool <mrasool@vmware.com>

Merge pull request #7652 from adharkar/frr-vni_switch

zebra: L3VNI to L2VNI conversion is not handled

Merge pull request #7747 from reubendowle/fixes/nhrpd

nhrpd fixes

staticd: autogenerated code modifications due to yang changes

updated callback methods based on autogenerated code.

Signed-off-by: vishaldhingra <vdhingra@vmware.com>

staticd: make table-id as the key for path-list

modified the yang model for path-list.
table-id should be a key, as one route can have
multiple table-ids.

Signed-off-by: vishaldhingra <vdhingra@vmware.com>

Merge pull request #7751 from mobash-rasool/igmp-fixes

pimd: IGMP conformance 5.10 test case is failing

Merge pull request #7753 from mobash-rasool/pim-fixes

pimd: when node changes from non-DR to DR S,G entry not created

Merge pull request #7756 from pjdruddy/bgplu-fixes

Bgplu fixes

Merge pull request #7787 from deastoe/fpm-work-ready-fixes

dplane_fpm_nl: routes stuck with 'q' flag (revisited)

Merge pull request #7785 from volta-networks/fix_show_pce_confidence_for_pr

Add level of confidence to show pcep-session

Merge pull request #7775 from vishaldhingra/pim

pimd : Added the command to clear the pim bsr data.

isisd: ignore routes w/ incompatible metric style

Currently the transition metric style is redundant because isis will
always read both reachability TLVs regardless of the configured
metric style. Correct this by only considering TLVs matching our
configuration.

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

tools: fix pce-config removal

make sure that the order in which the pcep-related commands are
removed by frr-reload.py is the correct one, i.e., pce followed
by pce-config followed by pcc.

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

pathd, tools: fix peer preference config

on one hand, the default value for a peer preference was always being
displayed, and on the other there was some code in frr-reload.py which
was attempting to add a default value to match this behavior, and which
was incorrectly overriding a specified preference. Fix this by removing
this code and making pathd behave like other daemons in this respect,
i.e. not displaying the default value.

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

tools: add bfdd to frr-reload.py daemons list

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

Merge pull request #7839 from ton31337/feature/print_large-community_dump

bgpd: Print large-community inside bgp_dump_attr()

Merge pull request #7835 from donaldsharp/cleanup_support_bundle

Cleanup support bundle

bgpd: Print large-community inside bgp_dump_attr()

Missing large communities.

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

Merge pull request #7842 from donaldsharp/wait_for_convergence

tests: Wait for bgp convergence *after* we re-add the interfaces

Merge pull request #7836 from volta-networks/fix_isis_net

isisd: When last area address is removed, resign if we were DR

zebra: zebra route-map delay-timer is global not per vrf

The zebra route-map delay timer value is a global value
not a per vrf change.  As such we should only print it
out one time.

We are seeing this:

zebra route-map delay-timer 33
 exit-vrf
zebra route-map delay-timer 33

When we have 2 vrf's configured.

Fix the code to only write it out for the default vrf

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

zebra: Remove uncalled function

Remove the dead function zebra_route_map_write_delay_timer

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