Merge pull request #9300 from donaldsharp/ospf_header

ospfd: Cleanup ospf_interface.h to work with our standards

Merge pull request #9328 from donaldsharp/peer_deletion

bgpd: bgp_packet_process_error can access peer after deletion

Merge pull request #9343 from ton31337/fix/dont_convert_by_default_to_alias

bgpd: Fix bgp routes filtering by [l]community-list

Merge pull request #9321 from donaldsharp/no_leak_re

zebra: Prevent memory leak if route is rejected early

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

tools: Add ospfv2 and ospfv3 commands in support bundle

tests: Make sure BGP filters by community lists work with community alias

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

bgpd: BGP extended [l]community-list regexp match must work with aliases

We have to convert BGP alias to numerical format to compare in regexp.

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

Merge pull request #9299 from donaldsharp/zebra_should_continue

Zebra should continue

Merge pull request #9273 from dlqs/luadofilefix

lib: Fix dead code from lua_dofile

zebra: Properly note add/update for rib_add_multipath_nhe

When calling rib_add_multipath_nhe ensure that we have
well aligned return codes that mean something so that
interersted parties can properly handle the situation.

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

zebra: Prevent memory leak if route is rejected early

When receiving a route via zapi, if the route is rejected
there exists a code path where we would not free the corresponding
re created.

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

Merge pull request #9324 from donaldsharp/bgp_info_cmp

bgpd: Store distance received from a redistribute statement

Merge pull request #9305 from proelbtn/patch-1

bgpd: fix typo in ensure_vrf_tovpn_sid

Merge pull request #9325 from anlancs/fix-lib-warning

lib: Fix coverity warning of return value

bgpd: bgp_packet_process_error can access peer after deletion

in bgp_io.c upon packet read of some error we are storing
the peer pointer on a thread to call bgp_packet_process_error.
In this case an event is generated that is not guaranteed to be
run immediately.  It could come in *after* the peer data structure
is deleted and as such we now are writing into memory that we
no longer possibly own as a peer data structure.

Modify the code so that the peer can track the thread associated
with the read error and then it can wisely kill that thread
when deleting the peer data structure.

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

Merge pull request #8262 from reubendowle/fixes/nhrp-misc-fixes

nhrp misc fixes

Merge pull request #8870 from anlancs/master-fix-reload-service

tools: add "vni" to oneline list

lib: Fix coverity warning of return value

Suppress coverity warning on unchecked str2sockunion's return value.

Signed-off-by: anlan_cs <anlan_cs@tom.com>

bgpd: Store distance received from a redistribute statement

When bgp receives the admin distance from a redistribution statement
let's store that distance for later usage.

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

Merge pull request #9283 from ton31337/fix/documentation_for_aggregate-address

doc: Specify how aggregate-address works for BGP

Merge pull request #9253 from donaldsharp/igmp_no

pimd: Allow more permisive versions of some no commands

Merge pull request #9313 from idryzhov/bgp-zebra-opaque

BGP opaque data improvements

Merge pull request #9315 from idryzhov/pathd-cli-fixes

pathd: a couple of cli/doc fixes

Merge pull request #9312 from opensourcerouting/topo-small-fixes

topotests: small fixes for two tests

doc: fix pathd example config

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

ospfd: Cleanup ospf_interface.h to work with our standards

When declaring functions we must name the variables as part
of our standard.

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

Merge pull request #9256 from idryzhov/dampening-revert

BGP per-peer dampening revert

Merge pull request #9280 from idryzhov/bgp-nb-revert-master

BGP NB revert

Merge pull request #9294 from anlancs/fix-ospfd-init

ospfd: fix coverity warning

bgpd: add protection against too large opaque data structure

BGP opaque data shouldn't be larger than zebra's buffer.

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

bgpd: use longer aspath string in opaque data

32 bytes are not enough to carry relatively long AS paths so let's make
the buffer larger.

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

topotests: reduce convergence check frequency

Check for convergence less frequently to reduce resources usage.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

topotests: skip daemons without configuration

Lets not start daemons without configurations.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

topotests: remove code that checks for version

We version the tests with FRR so we no longer need to check for version.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

topotests: use new OSPFv3 interface area syntax

The old one is not VRF aware and is being deprecated.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

topotests: reduce MSDP topology

Meanwhile we don't get all MSDP features (MSDP route validation via BGP
AS Path as described in RFC 4611 Section 2), kill one of the links of
the topology to avoid intermittent test failures due to different
traffic route.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

bgpd: fix typo in ensure_vrf_tovpn_sid

In eusure_vrf_tovpn_sid, there is a check to ensure not to select both
SID index and SID auto mode. But, this current check is wrong and not
meaningful.

Signed-off-by: proelbtn <contact@proelbtn.com>

pathd: fix indentation and redundant space in index config

Before:
```
segment-routing
 traffic-eng
  segment-list SL1
 index  10 mpls label 16010
```

After:
```
segment-routing
 traffic-eng
  segment-list SL1
   index 10 mpls label 16010
```

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

pathd: fix indentation for mpls-te config

Before:
```
segment-routing
 traffic-eng
 mpls-te on
 mpls-te import ospfv2
```

After:
```
segment-routing
 traffic-eng
  mpls-te on
  mpls-te import ospfv2
```

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

Merge pull request #9302 from ton31337/fix/comments

bgpd: Fix misleading comments for some parts

Merge pull request #9023 from idryzhov/static-nb

staticd: simplify the northbound code

bgpd: Fix misleading comments for some parts

Mostly just retrun => return and misleading comments at all.

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

doc: Specify how aggregate-address works for BGP

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

ospfd: fix coverity warning of one field initialization

Add the initialization of prefixlen field in struct prefix.

Signed-off-by: anlan_cs <anlan_cs@tom.com>

Merge pull request #9050 from LabNConsulting/chopps/reset-parallel

Chopps/reset parallel

Merge pull request #9261 from vivek-cumulus/fix_evpn_quick_flap

bgpd: Handle quick flaps of an EVPN prefix properly

zebra: prevent crash when nhlfe is NULL

There exists a call path where the nhlfe_alloc can return NULL
for blackhole nexthops.  In this case we were still trying
to save the nhlfe pointer causing a crash when we attempted
to add it to a self-contained list.

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

lib: Blackhole nexthops cannot be used as part of label generation

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

zebra: don't use default case when switching over enum nexthop

Do not use the `default` case when switching over an enumerated
type.  This allows the code to fail to compile when we add a
new enumeration.  Thus allowing us developers to know all
the places in the code we'll need to touch.

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

Merge pull request #9291 from donaldsharp/fix_long_sleep

tests: Remove sleep(200) from bgp-evpn-overlay-index-gateway

lib: Convert assert to error and record it instead of aborting

When we get a bad value for the opaque data length, instead
of stopping the program, discard the data and move on.

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

Merge pull request #8182 from mjstapp/topotest_start_tgen

tests: make the topogen object available when starting daemons

tests: Remove sleep(200) from bgp-evpn-overlay-index-gateway

Remove a 200 second sleep from bgp-evpn-overlay-index-gateway.
There does not seem to be any evidence that this is needed
and I cannot make the test fail without it.

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

Merge pull request #9085 from mobash-rasool/pim-upst-4

pimd: memory leak fix and issue fix

Merge pull request #9092 from rgirada/rmap

ospfd: OSPF hello packets not sent with configured hello timer

Merge pull request #9108 from opensourcerouting/ospf6d-range-fixes

ospf6d: fix argument processing in the "area ... range" command

Revert "tests/bgpd: initialized candidate_config to vtysh"

This reverts commit 64fb3f86661c0f5ac2f9017a7b6b999dbcb8e2e4.

Revert "tests: add bgp yang module registery to attr test"

This reverts commit 15cbc382061ee8807dbe79827c96ae94a4aa90c6.

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

Revert "bgpd: register northbound callbacks to bootstrap"

This reverts commit e4a464160dee2b53f2851879b2c736139061e74c.

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

Revert "bgpd: bgp northbound callbacks prototypes"

This reverts commit a1950c80a570e721503de4cf86de7a247e900781.

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

Revert "bgpd: convert global config to transactional cli"

This reverts commit ff8a8a7ac10b3211d5586c29dfcc5a3b32841029.

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

Revert "bgpd: register nb routing hook"

This reverts commit 374069da83f5f00f53156e1886372391c3f00b3f.

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

Revert "bgpd: shorten northbound callback names"

This reverts commit 906ebeec9800d3aef0eddb70ce74b48e370cb5ce.

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

Revert "bgpd: correct bgp global context"

This reverts commit 87ce25646be36bb9a27e75d177615e3d19a07df0.

Revert "bgpd: move router bgp nb callback"

This reverts commit 5e42cb2fb088a28202e60ec57a4116f3d1f0afd5.

Revert "bgpd: no router bgp check candidate config"

This reverts commit 4b63e358bc213bfa9d1baeda6fbbdfa39df60d2e.

Revert "bgpd: fix SA warnings in bgp northbound"

This reverts commit 082de39e7155e958416564f65286d4b57434ce4a.

Revert "bgpd: Use default VRF name if using `router bgp` command"

This reverts commit 5ce106b72bac1c45567159c4dfcc9276164bd18b.

Revert "bgpd: rename router bgp callbacks"

This reverts commit feded4118b89eba5a1629f45cfe623277612e404.

Revert "bgpd: fix crash in bgp instance creation"

This reverts commit a5ab756f2483594a19837e0c30f6184cd966940f.

Revert "bgpd: use common api in bgp_get"

This reverts commit bdaadb91e8adc103197d8f57a6a3358ff313a23c.

Revert "yang: update bgp module afi-safi parameters"

This reverts commit 3b66fcef5833da4eeb2dd19922778b108a1c63c1.

Revert "bgpd: convert addr-family clis to transactional clis"

This reverts commit 37a87b8f9865fe17a235b9a76a0dc47f03aae3df.

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

Revert "bgpd: add match-med nb support in aggregate cmd"

This reverts commit fa423774ff303ffc62db91f8aa09995711e81d50.

Revert "bgpd: add nb support for suppress-map in aggregate cmd"

This reverts commit 90e21f3535ff2d887cbb52f0e4e6fe4907095d70.

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

Revert "bgpd: neighbor peergrp config to transactional cli"

This reverts commit f4b8ec07219cce13c8627efe311247ee06f3a8ab.

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

Revert "*: shorten name of prefix-limit options"

This reverts commit afe03f6094656c9f96150223c1ed5ee8e9a4ba93.

Revert "bgpd: convert max prefix clis to transactional"

This reverts commit 04261decb9901c931f1800f27c459f3a0483776c.

Revert "bgpd: convert neighbor peer config to transactional"

This reverts commit 8611c7f3b6f33cb5ef8687bc126354d06eadbfcf.

Revert "bgpd: shorten names for default originate nb callbacks"

This reverts commit cc4440c3177f312294ec0cec836f41e453b367fb.

Revert "bgpd: default originate transactional callbacks"

This reverts commit 7c54def2926e85d760a7ab42f879a6545630878f.

Revert "bgpd: Actually return the group peer"

This reverts commit e5209541f2f425c25ef016be2cade323b327ea17.

Revert "bgpd: Check if peer is not NULL before calling peer_as_change()"

This reverts commit 7a68370285ed4d64e1ed9dcab60b17bfb26a0ba4.

Revert "bgpd: Respect `-e X` value for multipath"

This reverts commit 76bd07b6881add1fbeafa5dc11c48d9ad6a00f14.

Revert "bgpd: fix compiler warnings in nbr cmd"

This reverts commit 555c8ab7ad7a9f60c8d6cc9dd48705f0ee364797.

Revert "bgpd: convert to yang modeled nbr cmds"

This reverts commit 232c75cd032a0ff8a6540f03a10cb1ae3143af06.

Revert "bgpd: Prevent null deref"

This reverts commit e80bc08944be7cbe4085c03653818139ccd616d1.

Revert "bgpd: convert neighbor plist to transactional cli"

This reverts commit d9986d26c0a48f9cc2afeaa15fe304b094c041e4.

Revert "yang: nbr plist nb conversion for more afi-safis"

This reverts commit f4eac84c5ac136bc7dd37f2c950d375e94616bc4.

Revert "bgpd: convert nbr rmap transactional cli"

This reverts commit c6685575336566680aeb537e8ed5abb164be0268.

Revert "bgpd: fix unsetting of multipath as-set"

This reverts commit 174c5be33c433b9cc586d3aeee178407d282d97f.

Revert "bgpd: fix removal of rd vpn export"

This reverts commit c576dc6a4f5c62aeeb5e77e41c0f8efbf7f5797f.

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

This reverts commit 2a059a5448c37699eabd8802b44427978ec645ac.

Revert "bgpd: Remove hidden `neighbor X route-map Y <in|out>` command"

This reverts commit 4c3e9f072a415aa02f367aaff1cc90dd6fda5d06.

Revert "bgpd: fix 'neighbor X prefix-list Y in' SAFI VPN"

This reverts commit 9fa7a8bf3769914620225826e4ca8ef3f8182106.

Revert "bgpd: don't rely on northbound control plane protocol vrf pointer"

This reverts commit 3fa607bebacd4881053414d641e8376b4e4f6e83.

Revert "bgpd: convert send-community to transactional cli"

This reverts commit e48a9a90e07ccdeae8f6bcdff9c433f5d6ea0b39.

Revert "bgpd: remove duplicated checks from CLI"

This reverts commit 45d1d7c9a2b7b41125e565af566e10d470349122.

Revert "bgpd: don't use operational state in "router bgp" command"

This reverts commit 4062b455a6eacfe3f5107daa68351b18cc0ca7e7.

Revert "bgpd: convert l2vpn evpn nbr route-map to nb"

This reverts commit 8be861e60b7f6069c8a1c2aab75bbbbed3b57829.