Merge pull request #9587 from opensourcerouting/all-proto-test-fix

tests: Fix BGP check in all_protocol_startup

Merge pull request #9604 from mjstapp/fix_emacs_dir_local

tests: remove python format block from dir-locals

Merge pull request #9543 from donaldsharp/actually_set_mpls_data

tests: Fix wrong setting of mpls being turned on

tests: remove python format block from dir-locals

Remove the python block from the local emacs settings file
.dir-locals.el.

Signed-off-by: Mark Stapp <mjs.ietf@gmail.com>

Merge pull request #9558 from LabNConsulting/ziemba/doc-cli-new-node

doc/developer: how to add a CLI node

Merge pull request #9600 from kuldeepkash/cut_execution_time

tests: Optimize test_multicast_pim_sm_topo1.py execution time

Merge pull request #9571 from LabNConsulting/chopps/impr-zeb-netlink

tests: increase wait and update test

Merge pull request #9599 from Enigamict/fixtypo

tests: fix typo in zebra.conf

Merge pull request #9592 from ton31337/fix/bgp_neighbor_strip_whitespace

bgpd: Do not strip peer's description by whitespace in `show bgp summary`

Merge pull request #9586 from idryzhov/bgp-default-originate-rmap-fixes

BGP default-originate with route-map fixes

Merge pull request #9479 from AnuradhaKaruppiah/stale_path

zebra: Send path del to bgp for local-inactive path

Merge pull request #9475 from iqras23/change1

bgpd: VRF-Lite fix nexthop type

tests: Optimize test_multicast_pim_sm_topo1.py

1. Optimized test: test_clear_pim_neighbors_and_mroute_p0 run time by clearing
mroute and verifying mroutes separately. Execution time is reduced from almots 10 mins
to ~220 sec.

Signed-off-by: Kuldeep Kashyap <kashyapk@vmware.com>

tests: fix typo in zebra.conf

Signed-off-by: enigamict <mochienper@gmail.com>

Merge pull request #9019 from pjdruddy/ospfv3-early-break-list-walk

ospf6d: break early on route prefix mismatch

Merge pull request #9597 from donaldsharp/ospf6_header_cleanup

ospf6d: Cleanup headers to our standard

Merge pull request #9594 from donaldsharp/strict_prototypes

Strict prototypes

Merge pull request #9573 from LabNConsulting/chopps/update-template

tests: update the test template and doc

ospf6d: Cleanup headers to our standard

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

Merge pull request #9497 from opensourcerouting/cli-better-no

isisd: Remove weird wrapper function that downgrades time_t to 32 bit

Just use time_t, instead of downgrading time_t to a 32 bit value.
We should be using time_t instead of 32 bit unsigned values.

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

pathd: Ensure node_src_id is inited before usage

Compiler is warning that node_src_id may be used uninited
we know this is not possible but the compiler doesn't.

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

bgpd: Do not strip peer's description by whitespace in `show bgp summary`

```
~/frr# vtysh -c 'show run' | grep description
 neighbor 192.168.0.2 description test 1 2 3
~/frr# vtysh -c 'show ip bgp summary' | grep -E "Desc|192.168.0.2"
Neighbor        V         AS   MsgRcvd   MsgSent   TblVer  InQ OutQ  Up/Down State/PfxRcd   PfxSnt Desc
192.168.0.2     4          0         0         0        0    0    0    never      Connect        0 test 1 2 3
```

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

tests: We follow strict prototyping rules

Compiling with -Wstrict-prototypes is causing some complaints
during compiling. Make things happy.

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

Merge pull request #9581 from idryzhov/rpki-source-move

bgpd: move rpki source after the server config

tests: Fix BGP check in all_protocol_startup

Fix issue of topotest failures with BGP status Connect or Idle
instead of the expected Active

Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>

Merge pull request #9578 from donaldsharp/SLASH_ESCAPE

Slash escape

Merge pull request #9553 from donaldsharp/router_id

Router id optimizations

bgpd: move rpki source after the server config

Currently the source IP parameter must be entered between destination IP
and destination port parameters. This is not obviously understandable
when you read such config so let's move the source parameter to the end
of the command line, after the whole list of destination parameters. We
can do this without any deprecation cycle as the parameter was introduced
just recently and isn't in any public release yet.

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

tests: add one more check for default-originate with route-map

Make sure that we don't set communities from a random RIB route when
originating the default route.

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

bgpd: fix aspath memory leak

We allocate an as-path using bgp_attr_default_set and should free it
before exiting the function.

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

bgpd: fix default-originate route-map processing

When processing a route-map for default-originate, we actually want to
match by attributes in routes from the RIB, but set attributes in the
newly originated route. Currently, it's not the case. Instead, we
construct a dummy path combining attributes from both routes, and we end
up with multiple problems:
- match by as-path doesn't work
- communities from the matched RIB route are copied to the newly
  originated route
- we corrupt the RIB routes

To fix the issue, we should use the new route-map API that allows using
separate match/set objects.

Fixes #9584.

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

lib: add ability to supply separate match/set objects to routemaps

Sometimes it's needed to match by fields of one object but set fields of
another object. The following commit is an example.

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

tests: increase wait and update test

Modernize the test a bit, generate expected results rather than load from
file, and add a general json_cmp with retry function and use it.

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

Merge pull request #9582 from LabNConsulting/chopps/fix-xterm-windows

tests: fix xterm windows for topotests, better errors

tests: update the test template and doc

- Update the template and documentation to use newer pytest fixutres for
setup and teardown, as well as skipping tests when the suite fails.

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

Merge pull request #9572 from LabNConsulting/chopps/fix-cleanup

tests: deal with parallel exit of process we are reaping

Merge pull request #9566 from LabNConsulting/chopps/easy-fancy-json-setup

tests: new improved simple JSON template w/ doc update

Merge pull request #9583 from donaldsharp/remove_addKernelRoute_usage

Remove add kernel route usage

tests: Set mask to a default value for addKernelRoute

When looking for a implied host route it is not necessary
to add the `/32` to an ip route add.  As such masks
will not be set in this case.  Set the value of masks
to a known good value so that when the route installation
fails the test for it actually being there will tell you
that the route is not there -vs- complaining about mask
being uninited.

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

tests: Remove unneeded calls to addKernelRoutes

There is no need to add calls to addKernelRoutes for
groups.  They do not need to be routed via the
normal kernel methodology.

Tests run successfully with this change.

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

Merge pull request #9491 from opensourcerouting/totally-nssa

ospf6d: add support for NSSA totally stub areas

tests: fix xterm windows for topotests, better errors

- Fix xterm support to work, previously it mostly didn't, not it should
in all cases (i.e., single or dist mode).

- Catch when the user tries to use various window requiring topotests
features (e.g., --cli-on-error) but isn't running under supported
system (e.g., byobu/tmux/xterm), and fail the run with an explanation.

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

zebra: Send path del to bgp for local-inactive path

Problem:
When IP1:M1 (local) moved to IP1:M2 (remote-VTEP) bgpd continues to
advertise IP1:M1.

Fix:
Local path del is sent to bgp if the neigh was {local-active||peer-active}.
So path del needs to be called before the sync flags (including peer-active)
are cleared.

Ticket: #2706744

Signed-off-by: Anuradha Karuppiah <anuradhak@nvidia.com>

zebra: If we hand set the router-id only update everyone if it changes

When we hand set the router-id, but we have choosen a router-id
that is already the `winner` there is no point in updating anyone
with this data.

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

zebra: Do not send a router-id of 0.0.0.0 when we don't know it yet

At startup there exists a time frame where we might not know
a particular vrf's router id.  When zebra gets a request for
it let's not just blindly send whatever we have.  Let's be
a bit smart and only respond with one if we have one.
The upper level protocol can wait for it to have one.

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

Merge pull request #9576 from MajesticFalcon/master

ripd: Fix spelling errors

doc/developer: adding a CLI node: mention frr-reload.py and VTY_PUSH_XPATH

Signed-off-by: G. Paul Ziemba <paulz@labn.net>

Merge pull request #9533 from mobash-rasool/ospfv3-bug-fixes

ospfd: ASBR Summary LSA is not originated when process is reset

Merge pull request #9539 from ton31337/fix/floating_point_cli

bgpd: Replace `Enable` to `Disable` for floating-point bw encoding CLI

Merge pull request #9551 from mobash-rasool/ospfv2-bug-fixes-04

ospfd: GR Conformance fix in Hello packet DR election

Merge pull request #9552 from mobash-rasool/ospfv2-bug-fixes-03

ospfd: ANVL Test case 25.22, 25.23 and 28.11 fixes

tests: extend the ospf6_topo2 to test totally stub NSSAs

Signed-off-by: Renato Westphal <renato@opensourcerouting.org>

ospf6d: add support for NSSA totally stub areas

Add a knob to turn a NSSA area into a totally stub area. In this
configuration a Type-3 default summary route is generated by default.

Syntax: `area A.B.C.D nssa no-summary`.

Signed-off-by: Renato Westphal <renato@opensourcerouting.org>

ospfd: Don't allocate json memory unless we have json output

Several functions in ospf_vty.c were allocating json memory
irrelevant if it was needed or not and then at the end of the loop
free'ing it if it was not used.  Clean up the access pattern.

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

bgpd: Fix usage of JSON_C_TO_STRING_NOSLASHESCAPE

The usage of json_object_to_json_string_ext is mean for
generation of output string and returns a `char *` pointer
to the `formatted` output.  Just calling it does nothing
and it's expensive to boot.

Modify the code in bgpd to just output with the NOSLASHESCAPE
when outputting.

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

ospfd: Fix usage of JSON_C_TO_STRING_NOSLASHESCAPE

The usage of json_object_to_json_string_ext is meant for
generation of output string and returns a `char *` pointer
to the `formatted` output.  Just calling it does nothing
and it's expensive to boot.

Modify the code in ospfd to just output with the NOSLASHESCAPE
when outputting.

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

ripd: Fix spelling errors

Signed-off-by: schylar <schylarutley@hotmail.com>

Merge pull request #9577 from slankdev/slankdev-topotest-nsenter

topotests: to create pid file of topotest router

bgpd: VRF-Lite fix nexthop type

Description:
Change is intended for fixing the following issues related to vrf route leaking:

Routes with special nexthops i.e. blackhole/sink routes when imported,
are not programmed into the FIB and corresponding nexthop is set as 'inactive',
nexthop interface as 'unknown'.

While importing/leaking routes between VRFs, in case of special nexthop(ipv4/ipv6)
once bgp announces route(s) to zebra, nexthop type is incorrectly set as
NEXTHOP_TYPE_IPV6_IFINDEX/NEXTHOP_TYPE_IFINDEX
i.e. directly connected even though we are not able to resolve through an interface.
This leads to nexthop_active_check marking nexthop !NEXTHOP_FLAG_ACTIVE.
Unable to find the active nexthop(s), route is not programmed into the FIB.

Whenever BGP leaks routes, set the correct nexthop type, so that route gets resolved
and correctly programmed into the FIB, in the imported vrf.

Co-authored-by: Kantesh Mundaragi <kmundaragi@vmware.com>
Signed-off-by: Iqra Siddiqui <imujeebsiddi@vmware.com>

Merge pull request #9557 from idryzhov/bgp-view-cleanup

bgpd: cleanup special checks for views

doc: don't use abbreviate

Signed-off-by: Hiroki Shirokura <slank.dev@gmail.com>

topotests: update file format and resolving id

Signed-off-by: Hiroki Shirokura <slank.dev@gmail.com>

topotests: to create pid file of topotest router

Create a pid file for the router created by topotest.
By executing nsenter directly against this pid, developers
can execute commands directly from outside the unet shell.
This allows the developer to use script, tab completion, etc.,
and improves efficiency.

Signed-off-by: Hiroki Shirokura <slank.dev@gmail.com>

Merge pull request #9569 from LabNConsulting/chopps/parallel-config-nonjson

tests: refactor parallel reset/load config for non-json

tests: deal with parallel exit of process we are reaping

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

Merge pull request #9563 from opensourcerouting/build-strict-prototypes

build: `-Wstrict-prototypes` (+ BSD `sed`)

tests: refactor parallel reset/load config for non-json

Refactor the bgp_auth test to create common_config code to allow
non-json based tests to reset routers and load configs in parallel.

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

Merge pull request #9550 from donaldsharp/better_fails

bgpd: Add `PEER_DOWN_SOCKET_ERROR` to the list of peer failure modes

Merge pull request #9561 from idryzhov/bgp-no-router-vrf-default

bgpd: fix "no router bgp X vrf default"

Merge pull request #9564 from qlyoung/debian-update-proto-list

debian: update list of protos in pkg desc

Merge pull request #9521 from opensourcerouting/ospf6d-gr-fixes

ospf6d: minor GR fixes

Merge pull request #9547 from ton31337/fix/addpath_non_json_broken

bgpd: Drop afi/safi duplicate string notation for AddPath capability

Merge pull request #9567 from LabNConsulting/chopps/bgp-auth-speedup

tests: speed up bgp_auth test (24m -> 12m)

tests: speed up bgp_auth test (24m -> 12m)

- Reduce OSPF timers to 1 and 4
- Reduce BGP connect timer to 5
- Apply configs in parallel as single file
- Remove the switches as all links are p2p, perhaps this will help with
  reliability?

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

tests: new improved simple JSON template w/ doc update

Utilizes new pytest fixtures to completely factor out setup and teardown
functionality. Supply the JSON config and write your tests.

"The best topotest template yet!"

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

debian: update list of protos in pkg desc

Signed-off-by: Quentin Young <qlyoung@nvidia.com>

Merge pull request #9199 from LabNConsulting/chopps/micronet-prime

tests: cleanup: rerun changed files through black

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

tests: cleanup - remove unused imports

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

tests: Disable test which fails under micronet

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

tests: use common exa-receive.py script

New generic script uses a new default node specific log dir to avoid
collisions when running in parallel.

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

tests: add generic exa-receive.py script

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

tests: Make bgp_multiview_topo1 predictable

Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>

tests: remove unneeded mcast group kernel routes and sysctl

- The PIM tests do not need kernel routes to help them bind joins and
sources to specific interfaces. They should do that themselves directly.
Also do not change system wide "rp_filter" sysctl away from the value
required by everyone else.

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

tests: add back a 10 second delay to see if this fixes the failures

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

tools: move frr-reload.py to python3 explicitly

We already, reasonably, require python3 elsewhere. Do so here, and reap some
benefit.

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

tests: keep revisions of configs

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

tests: use new helper object for mcast-tester and iperf

- Decrease igmp query interval to fix pim test run times

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

tests: add helper object for mcast-tester and iperf tool.

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

tests: remove legacy Topo class from micronet

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

tests: remove legacy Topo class from infra

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

tests: remove legacy Topo class (fixes many pylint errors)

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

tests: fix pylint test errors

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

tests: fix pylint infra errors

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

docker: update with micronet changes

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

doc: minor update based on mininet removal

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

tests: micronet: update defaults for results+logging

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

tests: micronet: adapt tests

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

tests: micronet: update infra

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