Merge pull request #10235 from proelbtn/srv6-soft-reconf

SRv6 routes aren't inserted into data plane correctly with soft-reconfiguration

Merge pull request #10212 from mobash-rasool/mld-pim6-dev-prefix

Merge pull request #10313 from anlancs/fix-ospf-check

ospfd,ospf6d: fix wrong comparison of routemap name

pimd: Modifying in_addr to pim_addr in struct pim_neighbor and gm_sock

Changed struct in_addr source_addr to struct PIM_ADDR source_addr
which is to be used in both IPv4 and IPv6(Both MLD and IGMP).
Reviewed-by: Mobashshera Rasool <mrasool@vmware.com>
Signed-off-by: sarita patra <saritap@vmware.com>

pimd: Modifying in_addr to pim_addr in struct pim_upstream for IPv6.

Changed struct in_addr upstream_addr and struct in_addr upstream_register
to struct PIM_ADDR upstream_addr and struct PIM_ADDR upstream_register
which are to be used in both IPv4 and IPv6(Both MLD and IGMP).

Reviewed-by: Mobashshera Rasool <mrasool@vmware.com>
Signed-off-by: sarita patra <saritap@vmware.com>

pimd: Modifying in_addr to pim_addr in struct pim_nexthop for IPv6.

Changed struct in_addr last_lookup to struct PIM_ADDR last_lookup
which is to be used in both IPv4 and IPv6(Both MLD and IGMP).

Reviewed-by: Mobashshera Rasool <mrasool@vmware.com>
Signed-off-by: sarita patra <saritap@vmware.com>

pimd: Modifying in_addr to pim_addr in struct igmp_group for IPv6.

Changed struct in_addr group_addr to struct PIM_ADDR group_addr
in struct igmp_group which is to be used in
both IPv4 and IPv6(Both MLD and IGMP).

Reviewed-by: Mobashshera Rasool <mrasool@vmware.com>
Signed-off-by: sarita patra <saritap@vmware.com>

pimd: Modifying in_addr to pim_addr in struct pim_iface_upstream_switch

Changed struct in_addr address to struct pim_addr in struct
pim_iface_upstream_switch which is to be used in both IPv4
and IPv6(Both MLD and IGMP).

Reviewed-by: Mobashshera Rasool <mrasool@vmware.com>
Signed-off-by: sarita patra <saritap@vmware.com>

pimd: Modifying in_addr to pim_addr in struct pim_jp_agg_group for IPv6

Changed struct in_addr group to struct pim_addr group
which is to be used in both IPv4 and IPv6(Both MLD and IGMP).
Reviewed-by: Sarita Patra <saritap@vmware.com>
Signed-off-by: Mobashshera Rasool <mrasool@vmware.com>

pimd: Modifying in_addr to pim_addr in igmp_source for IPv6.

Changed struct in_addr source_addr to pim_addr source_addr
in struct igmp_source which is to be used in
both IPv4 and IPv6(Both MLD and IGMP).

Reviewed-by: Sarita Patra <saritap@vmware.com>
Signed-off-by: Mobashshera Rasool <mrasool@vmware.com>

pimd: Modifying in_addr to pim_addr in igmp_join for IPv6.

Changed struct in_addr source_addr to pim_addr source_addr
in struct igmp_join which is to be used in
both IPv4 and IPv6(Both MLD and IGMP).
Reviewed-by: Sarita Patra <saritap@vmware.com>
Signed-off-by: Mobashshera Rasool <mrasool@vmware.com>

pimd: Modifying in_addr to pim_addr in struct pim_interface for IPv6

Based on compiler option, pim_addr will be changed to in_addr
or in6_addr for pimd and pim6d respectively.
Reviewed-by: Sarita Patra <saritap@vmware.com>
Signed-off-by: Mobashshera Rasool <mrasool@vmware.com>

pimd: Modify in_addr to pim_addr in pim_assert_metric

This change is to accomodate IPv6 and IPv4 in the same code.
Based on pimd or pim6d, this will be compiled.
Reviewed-by: Sarita Patra <saritap@vmware.com>
Signed-off-by: Mobashshera Rasool <mrasool@vmware.com>

pimd: Modifying in_addr to pim_addr in struct pim_ifchannel for IPv6.

Changed struct in_addr ifassert_winner to pim_addr
which will be used in both IPv4 and IPv6(Both MLD and IGMP).
Reviewed-by: Sarita Patra <saritap@vmware.com>
Signed-off-by: Mobashshera Rasool <mrasool@vmware.com>

ospfd,ospf6d: make clear the comparison of routemap name

Comparison of the two pointer is confusing, they have no relevance
except the time both of them are empty.

Additionly modify one variable name and correct some comment words, they
are same in both ospfd and ospf6d.

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

Merge pull request #10299 from donaldsharp/various_stuff

Various stuff

Merge pull request #10309 from mobash-rasool/pim6-merge

pimd: API changes to accomodate IPv4 and IPv6

Added apis which will be decided on compile time
for pimd and pim6d daemon
Reviewed-by: Sarita Patra <saritap@vmware.com>
Signed-off-by: Mobashshera Rasool <mrasool@vmware.com>

doc: Adding the Design document for PIM IPv6

Added the High Level design for implementing MLD and PIMv6.

Added a PPT as well in case someone wants to translate it
to some other language.

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

pimd: Adding pim_addr as common address for IPv4 and IPv6

PIM_ADDR will be controlled at compile time for IPv4 and IPv6.
in_addr and in6_addr will be compiled for the respective daemons.

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

Merge pull request #10030 from anlancs/cleanup-reload-null0

tools: cleanup convertion of "Null0"

Merge pull request #9517 from anlancs/reload-add-str

tools: fix wrong get_contexts() of Config class.

Merge pull request #10191 from iqras23/bgp_gr

bgpd: VRF-Lite fix to clear stale leaked routes

tests: BGP : Dynamic route leak VRF lite (BGP-GR)

Authored-by: Shreenidhi A R <rshreenidhi@vmware.com>
Signed-off-by: Shreenidhi A R <rshreenidhi@vmware.com>

bgpd: VRF-Lite fix to clear stale leaked routes

Description:
Change is intended for fixing the issue related to
clearing of stale leaked routes:
- Whenever BGP goes down,
  after bringing down tcp connection and renegotiating capabilities,
  once we reestablish connection,
  we are not handling clear of VRF leaked route in the bgp_clear_stale_route.

- While bgp is clearing stale routes,
  we need to handle withdraw of routes for VRF route-leaking.

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

Merge pull request #10288 from ton31337/fix/rfc7300

bgpd: Add a warning if we try to use the last reserved ASN

Merge pull request #10289 from ton31337/fix/rfc9003

bgpd: Extended BGP Administrative Shutdown Communication

Merge pull request #10306 from anlancs/bfd-debug-info

lib: small debug adjustment for bfd

Merge pull request #10242 from anlancs/fix-service-time

build: add "--with-service-timeout" in configure.ac

pimd: Cleanup weird indentation

The zlog_warn used to be bounded by a debug guard
but the debug guard was removed but the code was
never fixed up to remove the open and close paranthesis.

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

bfdd: Clean up some white space snafu's

Found some extra spaces during code inspection.  Let's
get them cleaned up.

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

Merge pull request #10305 from mobash-rasool/minor-fixes

pimd: remove redundant header inclusion

lib: small debug adjustment for bfd

Just use `__func__` to display function name.

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

pimd: remove redundant header inclusion

Just found while code inspection, pim_str.h is included twice.

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

tests: Adopt bgp_shutdown_message test to a proper encoding

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

doc: Add rfc9003 to supported BGP RFCs

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

bgpd: Use correct encoding before printing shutdown msg

Using `bgp shutdown message MSG...`.

Length should be decoded from the first byte, but it's decoded from the data
instead.

Before:

```
%NOTIFICATION: sent to neighbor 192.168.0.2 6/2 (Cease/Administratively Shutdown) 70 bytes 5b 54 49 43 4b 45 54 2d 31 2d 31 34 33 38 33 36 37 33 39 30 5d 20 73
```

After:

```
%NOTIFICATION: sent to neighbor 192.168.0.2 6/2 (Cease/Administratively Shutdown) "[TICKET-1-1438367390] software upgrade; Expected downtime for 2 hours;"
```

On receiving side:

```
"[TICKET-1-1438367390] software upgrade; Expected downtime for 2 hours;"
```

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

bgpd: Limit shutdown message size to max 255 characters

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

bgpd: Increase administrative shutdown message size to 255

Extended BGP Administrative Shutdown Communication (rfc9003):

Basically, shutdown message size is increased to 255 from 128.

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

bgpd: Add a warning if we try to use the last reserved ASN

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

Merge pull request #10301 from donaldsharp/pim_multicast_fix

tools: Give longer for interface traffic in pim to work

Merge pull request #10290 from donaldsharp/nhrp_topo_queries

Nhrp topo queries

Merge pull request #10300 from donaldsharp/support_bundle_fix

tools: Run formatter over generate_support_bundle.py

tests: Ensure packets have a chance to arrive in test_multicast_pim_sm_topo4.py

The test is doing this:

a) gather interface data about packets sent
b) shut interface
c) no shut interface
d) gather interface data about packets sent
e) compare a to d and fail if packets sent/received has not incremented

The problem is, of course, that under heavy system load insufficient time
might not have passed for packets to be sent between c and d.  Add up to
35 seconds of looking for packet data being incremented else heavily
loaded systems may never show that data is being sent.

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

test: Cleanup via black the test_multicast_pim_sm_topo4.py

The test needed some cleanup via black formatting.

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

tests: Rename poorly named function

verify_pim_interface_traffic *fetches* the pim
traffic data.  Rename the function to what it
actually does

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

Merge pull request #10286 from ton31337/fix/rfc6938

bgpd: Deprecate DPA, ADVERTISER and RCID_PATH path attributes

bgpd: Deprecate DPA, ADVERTISER and RCID_PATH path attributes

rfc6938

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

Merge pull request #10283 from sworleys/Alpline-Git-Rev

docker: update alpine build enable set own version

tools: Run formatter over generate_support_bundle.py

The generate_support_bundle.py script needs some
reformating to meet our standards

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

Merge pull request #10226 from bisdn/jogo_fix_support_bundle_backup

tools: fix backing up previous logs in generate_support_bundle.py

Merge pull request #10284 from ton31337/fix/adjust_rfc4486

bgpd: Adjust symbolic names for cease notifications according to rfc4486

build: add "--with-service-timeout" in configure.ac

On lower CPU with lots of static routes, it will cost more than 2
minutes.

2 minutes is the default timeout value, we can adjust it by configure:
./configure --with-service-timeout=<digit>

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

Merge pull request #10287 from ton31337/fix/rfc7196

bgpd: Increase maximum supress threshold for dampening to 50,000

tests: Fixup output that was incorrect in nhrp_topo

The nhrp_topo test sets up some infrastructure and
was displaying the commands it was outputting
incorrectly.  Fix this.

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

Merge pull request #10296 from anlancs/fix-doc-version

doc: minor grammar correction

Merge pull request #10297 from ton31337/fix/gr_can_be_0

bgpd: Graceful Restart restart-time can be 0

bgpd: Graceful Restart restart-time can be 0

Using with LLGR, this should be allowed setting GR restart-time timer to 0,
to immediately start LLGR timers.

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

doc: minor grammar correction

Need one empty line.

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

Merge pull request #10291 from mjstapp/fix_topo_doc_reqs

docs: clarify topotest requirements

bgpd: Increase maximum supress threshold for dampening to 50,000

rfc7196 recommends:

In addition, BGP implementations have an internal constant, which we
   will call the 'maximum penalty', and the current computed penalty may
   not exceed it.

Router Maximum Penalty:  The internal constant for the maximum
      penalty value MUST be raised to at least 50,000.

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

bgpd: Adjust symbolic names for cease notifications according to rfc4486

The following subcodes are defined for the Cease NOTIFICATION
   message:

      Subcode     Symbolic Name

         1        Maximum Number of Prefixes Reached
         2        Administrative Shutdown
         3        Peer De-configured
         4        Administrative Reset
         5        Connection Rejected
         6        Other Configuration Change
         7        Connection Collision Resolution
         8        Out of Resources

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

Merge pull request #10294 from opensourcerouting/topotests-nhrp-require-5.0

topotests: require Linux 5.0 for NHRP

topotests: require Linux 5.0 for NHRP

It fails on 4.19, so let's go minimum 5.0 for the time being.

Signed-off-by: David Lamparter <equinox@opensourcerouting.org>

docs: clarify topotest requirements

The developer docs weren't clear about some pre-reqs.

Signed-off-by: Mark Stapp <mstapp@nvidia.com>

Merge pull request #10208 from adrianomarto/master

Babel configuration parameter fixes

Merge pull request #10260 from ton31337/feature/bgp_llgr_helper_mode

bgpd: Implement LLGR helper mode

docker: update alpine build enable set own version

Add ability to set your own env for the version of the docker
container alpine image. This is useful for applications like GNS3
who pin a specific version to look for when they boot up. When you build
locally to test your code you can just set the version to 0 so you don't
have to update configs/scripts looking for a specific image version.

Also fix a shebang in docker start for alpine.

Signed-off-by: Stephen Worley <sworley@nvidia.com>

Merge pull request #10274 from anlancs/fix-spell-error-bfd

bfdd: correct one spelling error of comment

bfdd: correct one spelling error of comment

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

tools: cleanup route convertion from null0/Null0 to blackhole

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

Merge pull request #10219 from donaldsharp/l3vpn_to_bgp_vrf_fixes

tests: Further fix bgp_l3vpn_to_bgp_vrf

tests: Add basic BGP Long-lived Graceful restart tests

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

bgpd: Change default long-lived graceful restart stale timer to 0 seconds

That means the feature is off by default.

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

bgpd: Implement LLGR helper mode

Tested between GoBGP and FRR (this commit).

```
┌───────────┐             ┌────────────┐
│           │             │            │
│ GoBGPD    │             │ FRRouting  │
│ (restart) │             │            │
│           │             │            │
└──────┬────┘             └───────┬────┘
       │                          │
       │                          │
       │                          │
       │     ┌───────────┐        │
       │     │           │        │
       │     │           │        │
       └─────┤ FRRouting ├────────┘
             │ (helper)  │
             │           │
             └───────────┘

// GoBGPD
% cat /etc/gobgp/config.toml
[global.config]
    as = 65002
    router-id = "2.2.2.2"
    port = 179

[[neighbors]]
    [neighbors.config]
        peer-as = 65001
        neighbor-address = "2a02:abc::123"
    [neighbors.graceful-restart.config]
        enabled = true
        restart-time = 3
        long-lived-enabled = true
    [[neighbors.afi-safis]]
        [neighbors.afi-safis.config]
            afi-safi-name = "ipv6-unicast"
        [neighbors.afi-safis.mp-graceful-restart.config]
            enabled = true
        [neighbors.afi-safis.long-lived-graceful-restart.config]
            enabled = true
            restart-time = 10
    [[neighbors.afi-safis]]
        [neighbors.afi-safis.config]
            afi-safi-name = "ipv4-unicast"
        [neighbors.afi-safis.mp-graceful-restart.config]
            enabled = true
        [neighbors.afi-safis.long-lived-graceful-restart.config]
            enabled = true
            restart-time = 20

% ./gobgp global rib add -a ipv6 2001:db8:4::/64
% ./gobgp global rib add -a ipv6 2001:db8:5::/64 community 65535:7
% ./gobgp global rib add -a ipv4 100.100.100.100/32
% ./gobgp global rib add -a ipv4 100.100.100.200/32 community 65535:7
```

1. When killing GoBGPD, graceful restart timer starts in FRR helper router;
2. When GR timer expires in helper router:
   a) LLGR_STALE community is attached to routes to be retained;
   b) Clear stale routes that have NO_LLGR community attached;
   c) Start LLGR timer per AFI/SAFI;
   d) Recompute bestpath and reannounce routes to peers;
   d) When LLGR timer expires, clear all routes on particular AFI/SAFI.

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

Merge pull request #10254 from ton31337/fix/typo

bgpd: Fix typo in bgp_aggr_community_hash_alloc()

Merge pull request #10261 from ton31337/fix/remove_if0

Merge pull request #10259 from anlancs/add-assert-ospf6

ospf6d: clean coverity warning of possible null pointer

bgpd: Drop `if 0` blocks

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

Merge pull request #10256 from anlancs/cleanup-zebra_evpn_mac_add

zebra: cleanup checking zebra_evpn_mac_add function's return value

ospf6d: clean coverity warning of possible null pointer

Add assert to protect it.

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

zebra: cleanup checking zebra_evpn_mac_add function's return value

This function is sure to return correct value by "assert", so the
checking its return value should be removed.

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

doc: Minor grammar correction

No comma needed.

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

babeld: C-style comments

Replacing C++-style comments with C-style comments.

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

doc: Updating babel default configuration parameters

Updating babel default configuration parameters rtt-min and
max-rtt-penalty according to the actual implementation.

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

babeld: Presenting interface configuration parameters

* Presenting the configuration parameters enable-timestamps,
max-rtt-penalty, rtt-min, and rtt-max.
* Using #defines for the default configuration values instead of magic
numbers.
* rtt-max and rtt-min are entered and presented in milliseconds, but
stored and internally used in microseconds.

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

Merge pull request #8494 from donaldsharp/wfi_failures

bgpd, tests: Add code to handle failed installations

Merge pull request #10255 from qlyoung/fix-ldpd-doc-option-typo

doc: fix typo in ldpd docs

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

bgpd: Fix typo in bgp_aggr_community_hash_alloc()

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

Merge pull request #10216 from opensourcerouting/lib-assorted-20211214

Merge pull request #10228 from opensourcerouting/bgp-agg-aspath

bgpd: fix aggregate route AS Path attribute

Merge pull request #9750 from mjstapp/zebra_installed_nhg_id

zebra: add installed nexthop-group id value

Merge pull request #10182 from mjstapp/fix_clang_break_opers

tools: clang-format break after operators

bgpd: fix aggregate route AS Path attribute

Always free the locally allocated attribute not the one we are using for
return. This fixes a memory leak and a crash when AS Path is set with
route-map.

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

Merge pull request #10098 from opensourcerouting/ospf-gr-topotest-fix

ospfd: fix incorrect detection of topology changes in helper mode

Merge pull request #10250 from ton31337/fix/no_need_to_test_for_BGP_TIMER_OFF

bgpd: No need to test if a thread is running for BGP_TIMER_OFF

bgpd: No need to test if a thread is running for BGP_TIMER_OFF

Handles that inside the macro.

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

Merge pull request #10245 from anlancs/fix-spell-error

zebra: correct one spell error

Merge pull request #10181 from ton31337/fix/default-originate_route-map

doc: Add `default-originate route-map WORD` for bgpd