Merge pull request #13612 from chiragshah6/fdev1

ospfd: fix interface param type update

Merge pull request #13775 from mobash-rasool/fixes2

Merge pull request #13767 from opensourcerouting/fix/convert_rip_topo1_to_polling

tests: Convert rip/ripng tests using run_and_expect

Merge pull request #13763 from LabNConsulting/mgmtd/simplify

fix 'exit' bug in config file processing, et al.

pimd,pim6d: Correct the socket to send reg-stop msg

We were using the pim interface socket to send the register
stop msg, it works fine in cases where the interface on which
register msg is received and the interface on which the register-stop
msg is supposed to be sent is the same.
But when the interfaces are different, msg send fails because
the outgoing interface is not right.

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

Merge pull request #13704 from donaldsharp/flowspec_support_bundle

bgpd: Add some color to why nexthop_set failed

We are seeing some frequent test failures with
setting the nexthop correctly.  At this point
in time, I have no idea what is going wrong,
but I don't have a bunch of information either,
so let's add the local and remote values.

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

tests: test_bgp_flowspec_topo.py changes as per linter

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

tests: The bgp_flowspec frequently fails try to gather more data

Add a bit of code to allow the bgp_flowspec topotest to gather
data when something goes wrong.

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

tests: Apply black formatting for rip_topo1/ripng_topo1

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

tests: Use run_and_expect() for ripng_topo1

Avoid checking once and failing immediately.

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

tests: Use run_and_expect() for rip_topo1

Avoid checking once and failing immediately.

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

Merge pull request #13762 from anlancs/doc/pthread-typo

doc: Adjust pthread description

tests: add mgmtd config test

Testing early exits/ends from config files loaded with `vtysh -f cfgfile`
as well as `vtysh < cfgfile`, verify the same as non-mgmtd behavior.

Signed-off-by: Christian Hopps <chopps@labn.net>

lib: mgmtd: fix/stdize debug message macros

Signed-off-by: Christian Hopps <chopps@labn.net>

mgmtd: remove unused code

Signed-off-by: Christian Hopps <chopps@labn.net>

lib: mgmtd: session create and destroy both short-circuit

For creation this is the first thing done so short-circuit just means inline
sync response. However, for destroy there could be commands in-flight, these
will be discarded when they match no session, and the state cleaned up
immediately when the message short-circuits.

Signed-off-by: Christian Hopps <chopps@labn.net>

lib: mgmtd: avoid recursion with vty_close and add error log

Avoid recursion into vty_close() when being notified of a session closure that
happened inside vty_close().

If a vty is closed with outstanding config yet to be commited
issue a warning that it is being lost.

Signed-off-by: Christian Hopps <chopps@labn.net>

vtysh: stop reading config file if user `exit`s from root level.

This is required to make sure that we properly send the
XFRR_end_configuration tag to the daemons. Previously if the user had an
`exit` at the root level the parser would just drop out of the config
node and so XFRR_end_configuration, even if sent, would be ignored

Signed-off-by: Christian Hopps <chopps@labn.net>

staticd: staticd no longer loads config files

We need to ignore SIGHUP rather than reload config now.

Signed-off-by: Christian Hopps <chopps@labn.net>

doc: Adjust pthread description

Refer to the latest code, modified the description of pthread.

Signed-off-by: anlan_cs <vic.lan@pica8.com>

Merge pull request #13752 from donaldsharp/fix_suppress_fib_test

tests: Fix missing assert for test case in bgp_suppress_fib

Merge pull request #13675 from zmw12306/bfd_version_check

bfd:fix version bits check

Merge pull request #13743 from LabNConsulting/chopps/docker-ci-update

docker: ubuntu-ci 18.04 -> 22.04, topotest parallel run examples

Merge pull request #13727 from anlancs/pimd/cleanup-1

pimd: adjust the display for debug

tests: Fix missing assert for test case in bgp_suppress_fib

Test was attempting to test the 60.0.0.0 route but was querying
10.0.0.3 and ignoring the result.  Let's fix it.

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

docker: ubuntu-ci 18.04 -> 22.04, topotest parallel run examples

Update the READMEs, and give example of a full parallel topotest run with
result extraction for analyze.py

Signed-off-by: Christian Hopps <chopps@labn.net>

Merge pull request #13745 from FRRouting/revert-13509-bgp_nht_other_if_cases

Revert "bgpd: upon if up event, evaluate bnc with matching nexthop"

Merge pull request #13734 from LabNConsulting/chopps/removeunused

remove mentions of socat

Merge pull request #13739 from donaldsharp/zebra_dplane_crash

zebra: Prevent crash because nl is NULL on shutdown

Revert "bgpd: upon if up event, evaluate bnc with matching nexthop"

Merge pull request #13730 from anlancs/doc/thread-event-typo

doc: Adjust event system

Merge pull request #13596 from Pdoijode/pdoijode/ospf-intf-json-changes

ospfd: json hierarchy change for "show ip ospf interface json"

Merge pull request #13509 from pguibert6WIND/bgp_nht_other_if_cases

bgpd: upon if up event, evaluate bnc with matching nexthop

Merge pull request #13664 from routingrocks/pim_nb_fix

pimd: Change in PIM northbound error, when a path to RP is not found …

Merge pull request #13364 from zmw12306/bfd_auth

bfd: fix missing Authentication in control pkt

Merge pull request #13650 from opensourcerouting/feature/bgpd_default_originate_route_map_timer

bgpd: Add an ability to control default-originate route-map timer

zebra: Prevent crash because nl is NULL on shutdown

When shutting down the main pthread was first closing
the sockets associated with the dplane pthread and
then telling it to shutdown the pthread at a later point
in time.  This caused the dplane to crash because the nl
data has been freed already.  Change the shutdown order
to stop the dplane pthread *and* then close the sockets.

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

Merge pull request #13645 from idryzhov/bfd-nb-destroy

bfdd: remove redundant nb destroy callbacks

doc: update doc removing socat req + remove unused tests code

Signed-off-by: Christian Hopps <chopps@labn.net>

Merge pull request #13726 from LabNConsulting/chopps/fixpim6topotest

tests: fixing pim6 topotest bugs

tests: convert old pim test to more cleanly use pytest fixture

This is a good way to run a per-test background helper process. Here the
helper object is created before the test function requesting it (through param
name match), and then cleaned up after the test function exits (pass or failed).

A context manager is used to further guarantee the cleanup is done.

Signed-off-by: Christian Hopps <chopps@labn.net>

tests: fixing pim6 topotest bugs

- Remove use of bespoke socat
- Use ipv6 support in mcast-tester.py
- do not run processes in the background behind munet/micronet's
  back with `&` (ever) -- use popen or the helper class

Signed-off-by: Christian Hopps <chopps@labn.net>

Merge pull request #13421 from mobash-rasool/igmp-ups2

pimd, pim6d: re-arrange some code and pimv6 deletion flow fix

doc: Adjust event system

Refer to the latest code, modify the description of event system.

Signed-off-by: anlan_cs <vic.lan@pica8.com>

Merge pull request #13390 from ryndia/master

ospfd: ospf_route.c memory leak fix

tests: mcast-tester.py handles IPv6

Signed-off-by: Christian Hopps <chopps@labn.net>

pimd: adjust the display for debug

The vrf name should be separated when it is displayed. And remove
unnecessary space after number.

Before:
```
pim_upstream_sg_running: (100.100.1.100,232.100.100.100)x is not installed in mroute
pim_upstream_del(pim_ifchannel_delete): Delete (100.100.1.100,232.100.100.100)[x] ref count: 1 , flags: 1048585 c_oil ref count 2 (Pre decrement)
```

After:
```
pim_upstream_sg_running: (100.100.1.100,232.100.100.100)[x] is not installed in mroute
pim_upstream_del(pim_ifchannel_delete): Delete (100.100.1.100,232.100.100.100)[x] ref count: 1, flags: 1048585 c_oil ref count 2 (Pre decrement)
```

Signed-off-by: anlan_cs <vic.lan@pica8.com>

Merge pull request #13624 from Keelan10/bgp_flowspec-leak-fix

bgpd: Free temp memory

Merge pull request #11560 from patrasar/fix_pim6_state_on_prune

pimd, pim6d: Move mld/igmp deletion code to a common api

Move the mld/igmp deletion common code to api pim_gm_interface_delete
code for IPv6 deletion(gm_ifp_teardown) for MLD was missing in this flow
Making the code common fixes this too.

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

pimd, pim6d: Rename and move api pim_cmd_interface_delete

Rename pim_cmd_interface_delete to pim_pim_interface_delete
and move the api to pimd/pim_iface.c

Changed the return type of the api from int to void.

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

pimd, pim6d: Move api pim_if_membership_clear

Move pim_if_membership_clear api from pimd/pim_nb_config.c
to pimd/pim_iface.c
Also fixed curly braces warning
WARNING: braces {} are not necessary for single statement blocks
1773: FILE: /tmp/f1-127504/pim_iface.c:1773:

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

Merge pull request #13622 from kuldeepkash/bgp_always_compare_med

tests: Adding bgp_always_compare_med testuite

Merge pull request #13693 from chiragshah6/fdev2

tools: fix list value remove in frr-reload

Merge pull request #13706 from LabNConsulting/chopps/close-files-coverity

lib: close config files after reading (coverity)

Merge pull request #13707 from LabNConsulting/chopps/coverity-strftime

mgmtd: assert an assertion for coverity

Merge pull request #13709 from Jafaral/pkg-9.1

debian, redhat: updating changelog for new release

debian, redhat: updating changelog for new release

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

Merge pull request #13697 from sri-mohan1/srib-ldpd1

ldpd: changes for code maintainability

mgmtd: assert an assertion for coverity

I believe coverity can't tell the length of the return value from strftime based
on the format string (like we can), so it allows `n` to be larger than it could
be which then allows `sz - n` to be negative which is size_t positive and very
large so it thinks an overrun is possible.

CID#1563211

Signed-off-by: Christian Hopps <chopps@labn.net>

build: FRR 9.1 development version

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

Merge pull request #13701 from opensourcerouting/fix/update_show_ip_bgp_summary_doc

doc: Document RFC8212 under `Displaying BGP Information` section

Merge pull request #13687 from donaldsharp/eigrp_fakeout

tests: Babel tests do not need to log to eigrp

lib: close config files after reading (coverity)

fixes coverity CID# 1564375

Signed-off-by: Christian Hopps <chopps@labn.net>

tests: Babel tests do not need to log to eigrp

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

doc: Document RFC8212 under `Displaying BGP Information` section

Looks like people don't find what does it mean (Policy) at first shot, let's
try giving more hints here.

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

Merge pull request #13690 from LabNConsulting/chopps/mgmtd-cleanup

Chopps/mgmtd cleanup

Merge pull request #13618 from LabNConsulting/chopps/fixlogging

ldpd: changes for code maintainability

these changes are for improving the code maintainability and readability

Signed-off-by: sri-mohan1 <sri.mohan@samsung.com>

Merge pull request #13649 from donaldsharp/unlock_the_node_or_else

zebra: Unlock the route node when sending route notifications

Merge pull request #13689 from sri-mohan1/srib-ldpd1

ldpd: changes for code maintainability

tools: fix list value remove in frr-reload

There might be a time element(s) from
temporary list are removed more than once
which leads to valueError in certain python3
version.

commit-id 1543f58b5 did not handle valueError
properly. This caused regression where
prefix-list config leads to delete followed
by add.

The new fix should just pass the exception as
value removal from list_to_add or list_to_del
is best effort.
This allows prefix-list config has no change
then removes the lines from lines_to_del and
lines_to_add properly.

Ticket:#3490252
Testing:

Configure prefix-list in frr.conf and perform
multiple frr-reload. After first reload operatoin
subsequent ones should not result in delete followed
by add of the prefix-list but rather no-op operation.

(Pdb) lines_to_add
[(('ip prefix-list FOO permit 10.2.1.0/24',), None)]
(Pdb) lines_to_del
[(('ip prefix-list FOO seq 5 permit 10.2.1.0/24',), None),
 (('ip prefix-list FOO seq 10 permit 10.2.1.0/24',), None)]
(Pdb) lines_to_del_to_del
[(('ip prefix-list FOO seq 5 permit 10.2.1.0/24',), None),
 (('ip prefix-list FOO seq 10 permit 10.2.1.0/24',), None)]
(Pdb) lines_to_add_to_del
[(('ip prefix-list FOO permit 10.2.1.0/24',), None),
 (('ip prefix-list FOO permit 10.2.1.0/24',), None)]
(Pdb) c
> /usr/lib/frr/frr-reload.py(1562)ignore_delete_re_add_lines()
-> return (lines_to_add, lines_to_del)
(Pdb) lines_to_add
[]
(Pdb) lines_to_del
[]

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

Merge pull request #13670 from donaldsharp/coverity_99

Coverity 99

mgmtd: rm unused/unneeded code add couple comments

Signed-off-by: Christian Hopps <chopps@labn.net>

ldpd: changes for code maintainability

these changes are for improving the code maintainability and readability

Signed-off-by: sri-mohan1 <sri.mohan@samsung.com>

tests: mgmtd: add prologue to bigconf tests

Signed-off-by: Christian Hopps <chopps@labn.net>

Merge pull request #13658 from louis-6wind/fix-flex-algo2

isisd: fix wrongly disabled flex-algorithm

tests: fix some broken logging

- make sure we close and remove all handlers for named logs on each reuse.
- test module level exec.log no longer truncated to last test case output
- cleanup the log names, and make sure they are present in all exec logs
- keep separate exec logs for each pytest worker when running in distributed mode
- disabled code due to CI infra can't handle it: add per test case exec logs

Signed-off-by: Christian Hopps <chopps@labn.net>

Merge pull request #13685 from sri-mohan1/srib-ldpd1

ldpd: changes for code maintainability

mgmtd: Fix up some coverity issues

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

bgpd: entry->any is never true

The only places entry->any could ever be set to true was
when str was NULL.  Unfortunately with the way our CLI works
str is impossible to be NonNULL.  The entry->any value *used*
to work prior to commit e961923c7217b935027107cad30c35c3907c936f
but it was changed back in 2016 and no-one has noticed the changed
ability.

Let's just admit that there are no users of this and remove this
dead code.

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

tests: Rename a test to test_bgp_gr_functionality_topo3.py

Since the test was not named test_bgp_gr_functionality_topo3.py
pytest was not picking it up to run.  Let's run it.

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

Merge pull request #13684 from LabNConsulting/chopps/pause-functionality

tests: pause (with CLI) at step() functions when run with --pause

Merge pull request #13681 from LabNConsulting/chopps/bad-abstract2

lib: mgmtd: remove backend abstraction layer and other cleanup

Merge pull request #13677 from LabNConsulting/chopps/bad-abstract

lib: mgmtd: remove obfuscating abstraction layer and other cleanup

pimd: remove api source_channel_oil_detach()

This API is no more in use.

Signed-off-by: Sarita Patra <saritap@vmware.com>

pim6d: Clear channel_oil on prune

Receiver---LHR---RP

Problem:
In LHR, ipv6 pim state remains after MLD prune received.

Root Cause:
When LHR receives join, it creates (*,G) channel oil with
oil_ref_count = 2. The channel_oil is used by gm_sg sg->oil
and upstream->channel_oil.

When LHR receives prune, currently upstream->channel_oil is
deleted and gm_sg sg->oil still present. Due to this channel_oil
is still present with oil_ref_count = 1

Fix:
When LHR receives prune, upstream->channel_oil and pim_sg sg->oil
needs to be deleted.

Issue: #11249

Signed-off-by: Sarita Patra <saritap@vmware.com>

ldpd: changes for code maintainability

these changes are for improving the code maintainability and readability

Signed-off-by: sri-mohan1 <sri.mohan@samsung.com>

tests: pause (with CLI) at step() functions when run with --pause

Signed-off-by: Christian Hopps <chopps@labn.net>

lib: mgmtd: remove abstraction layer and other cleanup

Code is no longer using a global FE "client context", and instead
creates client objects, rename the structure and it's uses to reflect this.

Remove an obfuscating abstraction layer whose existence was entirely
based on using a uintptr_t rather than a pointer to an declared-only struct.

Change multi-duty "params" structure into a single duty callbacks one.

Remove unsupported API code.

Signed-off-by: Christian Hopps <chopps@labn.net>

lib: mgmtd: remove obfuscating abstraction layer and other cleanup

Remove an obfuscating abstraction layer whose existence was entirely
based on using a uintptr_t rather than a pointer to an declared-only struct.

As the code is no longer using a global FE "client context", and instead
create client objects, rename the structure and it's uses to reflect this.

Change init "params" structure into a single use callbacks one.

Signed-off-by: Christian Hopps <chopps@labn.net>

Merge pull request #13656 from sri-mohan1/srib-ldpd1

ldpd: changes for code maintainability

Merge pull request #13668 from donaldsharp/fsm_more_descriptive

bgpd: Give more data when state machine fails to change state

bfdd: fix version bits check.

The version of bfd pkt is represented by 3 bits in B[0].
Signed-off-by: zmw12306 <zmw12306@gmail.com>

bgpd: Give more data when state machine fails to change state

When a state machine transition fails, bgpd would output
data about what happened, but not necessarily give the
reason why.  Add that data to the output.

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

Merge pull request #13659 from donaldsharp/increase_mgmt_time

tests: new mgmt_startup tests are failing due to insufficient time

Merge pull request #13646 from donaldsharp/logically_illogical

mgmtd: xpath is already NULL in the failure path

Merge pull request #13641 from donaldsharp/com_list_str

Bunch of code cleanup from Coverity

Merge pull request #13660 from anlancs/bgpd-cleanup-12

bgpd: Fix typo in debug message