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.

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: When deleting a neighbor from a peer-group the PGNAME is optional"

This reverts commit 2cbd181ac99801f2fb6b5b820ad66626c0f7168a.

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.

Revert "bgpd: prefix-list and route-map nb fixes"

This reverts commit 68a57089122b22414b04ebc28bd0db81051fdcbd.

Revert "bgpd: vrf route leaking, fix the bgp instance delete and re-add"

This reverts commit ef7c53e244d5d0cd7c0ba2536fc931abf28df656.

Revert "bgpd: fix pending commit check"

This reverts commit ea1a5c00df70d7204ccf8716222dc355e2501723.

Revert "bgpd: fix bgp_get_vty return values"

This reverts commit d09328e5991c9f657758921264492825e7081175.

Revert "bgpd: fix crash when as/type mismatches in config"

This reverts commit 0d6d0208a55c786513f472ad690bae1788e173fa.

Revert "bgpd: simplify bgp_global_local_as_modify code"

This reverts commit debb7b1931cdf5b3e06ca244bf09bb009b5a8677.

Revert "bgpd: forbid modification of bgp instance type"

This reverts commit d3e3677096e5cf30116ca63879caf44e25b080ad.

Revert "bgpd: don't return error for GR no-op commands"

This reverts commit 895b0d0ad6e39ad54e82e5c3d9060aa390c8bdd8.

build: Update configure.ac to reflect proper version

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

Merge pull request #8608 from Fredi-raspall/pr_isis_sr_anycast

isisd: clear the N-flag in ext. reachability TLVs

Merge pull request #8133 from gpnaveen/ospf_basic_testcases

tests: adding ospf basic 2 testcases.

Merge pull request #8673 from LabNConsulting/chopps/ly2-again

lib: libyang2 add missed conversion

Merge pull request #8646 from chiragshah6/mdev

zebra: evpn check vni oper state in svi up/down event

isisd: simplify node Sid handling

Centralize the n-flag-clear processing to a single point.

Signed-off-by: Fredi Raspall <fredi@voltanet.io>

isisd: clear the N-flag in ext. reachability TLVs

If the n-flag-clear option is set in the configuration of a prefix
segment, clear the flag in the extended ip reachability TLVs.

RFCs 7794 and 8667 are not too strict on the setting / clearing the
N-flag in prefix SIDs. However, if there exists a cmd line option
to clear it, it should be cleared in the TLVs announced, as other
vendors do.

Signed-off-by: Fredi Raspall <fredi@voltanet.io>

Merge pull request #8535 from opensourcerouting/zlog-rnode

zebra: replace _rnode_zlog with %pZN ext

tests: adding ospf basic 2  testcases.

Test cases included are 1 ospf cost, 1 ospf mtu.

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

lib: libyang2 add missed conversion

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

Merge pull request #8680 from idryzhov/rip-ripng-fix-crash

ripd, ripngd: fix interface wakeup after shutdown

Merge pull request #8643 from icosahedral/master

bgpd: modify path selection for EVPN type-5 routes

Merge pull request #8676 from idryzhov/test-gitignore

tests: fix missing gitignore entry

ripngd: fix interface wakeup after shutdown

RIPNG schedules a call to `ripng_interface_wakeup` in 1 second after
receiving the interface UP event from zebra. The function is called even
if the interface was shut down during this interval.

This is incorrect and also leads to a crash in the following scenario:
```
vtysh -c "conf" -c "router ripng vrf red" -c "network enp2s0"
ip link add red type vrf table 1
ip link set enp2s0 vrf red
ip link set enp2s0 down
ip link set enp2s0 up && sleep 2 && ip link del red
```

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

ripd: fix interface wakeup after shutdown

RIP schedules a call to `rip_interface_wakeup` in 1 second after
receiving the interface UP event from zebra. The function is called even
if the interface was shut down during this interval.

This is incorrect and also leads to a crash in the following scenario:
```
vtysh -c "conf" -c "router rip vrf red" -c "network enp2s0"
ip link add red type vrf table 1
ip link set enp2s0 vrf red
ip link set enp2s0 down
ip link set enp2s0 up && ip link del red
```

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

tests: fix missing gitignore entry

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

Merge pull request #8672 from qlyoung/fix-bgp-timer-display

bgpd: fix display of timers when only 1 is changed

When only one of the keepalive or hold timers is changed from the
default, bgp won't print the timers command in the config.

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

Merge pull request #8556 from donaldsharp/bgp_pbr_weird

Bgp flowspec cleanups

Merge pull request #8144 from LabNConsulting/chopps/ly2

libyang2

lib: adapt to version 2 of libyang

Compile with v2.0.0 tag of `libyang2` branch of:
https://github.com/CESNET/libyang

staticd init load time of 10k routes now 6s vs ly1 time of 150s

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

Merge pull request #8658 from gromit1811/bugfix_8567

Fix #8567 OSPFv3-26.13 ANVL failure

Merge pull request #8649 from ton31337/fix/unify-naming-for-topotests-directory

tests: Rename tests/topotests directories to be consistent

Merge pull request #8659 from mjstapp/fix_connected_multi

lib,zebra: Use a flag to track down status for connected addrs

Merge pull request #8663 from donaldsharp/pretty_function

Use __func__ instead of __PRETTY_FUNCTION__

Merge pull request #8544 from donaldsharp/weird_stuff_in_topo

tests: Fix pylint issues in test_bgp_recursive_route_ebgp_multi_hop.py

pimd: Use __func__ instead of __PRETTY_FUNCTION__

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

ospfd: Use _func__ instead of __PRETTY_FUNCTION__

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

zebra: Use __func__ instead of __PRETTY_FUNCTION__

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

bgpd: use __func__ instead of __PRETTY_FUNCTION__

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

lib,zebra: Use a flag to track down status for connected addrs

Track 'down' state of connected addresses with a new flag. We
may have multiple addresses on an interface that share a prefix;
in those cases, we need to determine when the first address
is valid, to install a connected route, and similarly detect
when the last address goes 'down', to remove the connected
route.

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

Merge pull request #8661 from donaldsharp/more_privs

lib: Add ZCAP_IPC_LOCK

tests: Add pytestmark to the flowspec topotest

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

bgpd: Remove usage of prefix2str and use builtin in bgp_zebra.c

Convert over from prefix2str explicit call and use the builtin
%pFX we have now.

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

bgpd: reduce cut-n-paste of bgp_zebra_announce_default for install

This bit of code was cut-n-pasted all over the place:

               if (!bpa->installed && !bpa->install_in_progress) {
                       bgp_send_pbr_rule_action(bpa, NULL, true);
                       bgp_zebra_announce_default(bgp, nh,
                                                  bpa->afi,
                                                  bpa->table_id, true);
               }

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

bgpd: reduce cut-n-paste code in bgp_pbr.c

Create a function bgp_bpr_bpa_remove that is this cut-n-paste code:

       if (bpa->refcnt == 0) {
               if (bpa->installed && bpa->table_id != 0) {
                       bgp_send_pbr_rule_action(bpa, NULL, false);
                       bgp_zebra_announce_default(bpa->bgp, &(bpa->nh),
                                                  AFI_IP,
                                                  bpa->table_id,
                                                  false);
                       bpa->installed = false;
               }
       }

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

tests: Fix pylint issues in test_bgp_recursive_route_ebgp_multi_hop.py

Tests had format for strings with 2 variables but 1 place to put the data.

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

Merge pull request #8660 from qlyoung/fix-bgp-conditional-advertisement-deconfig-removing-unrelated-filters

bgpd: fix deconfig of conditional advertisement

Merge pull request #8629 from donaldsharp/parse_rtattr

Parse rtattr

Merge pull request #8509 from volta-networks/pathd_ls_client

pathd: Traffic Engineering Database support

tests: Skip example_topojson_test and example_test from being tested

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

Merge pull request #8622 from volta-networks/fix_ospf6_max_if_addr

ospf6d: Limit the number of interface addresses being supported in ospfv3

zebra: Consolidate on 1 function netlink_parse_rattr_nested

if_netlink.c created it's on nested parsing #define which
is identical to netlink_parse_rtattr_nested.  Consolidate
on one instead of having this duality.

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

zebra: memset the `struct rtattr *tb[SIZE]` in setting function

In order to parse the netlink message into the
`struct rtattr *tb[size]` it is assumed that the buffer is
memset to 0 before the parsing.  As such if you attempt
to read a value that was not returned in the message
you will not crash when you test for it.

The code has places were we memset it and places where we don't.
This *will* lead to crashes when the kernel changes.  In
our parsing routines let's have them memset instead of having
to remember to do it pre pass in to the parser.

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

lib: Add ZCAP_IPC_LOCK

We'll need ZCAP_IPC_LOCK for future work coming down the pike
related to dataplane work being done.

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

bgpd: fix deconfig of conditional advertisement

Deconfiguring conditional advertisement resulted in all other policy
settings on the peer getting removed due to an excessively large memset.

This also created a desync with the northbound config tree, which caused
its own set of problems...

Fix the memset to just remove the conditional advertisement config.

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

ospf6d: document the interface address limits for ospfv3

Signed-off-by: Lynne Morrison <lynne@voltanet.io>

ospf6d: Limit the number of interface addresses being supported

The code had no limits on addresses configured on an interface running
ospf6d.  The code would crash when more than 100 addresses were added.
This change limits the number of interface address to 100 if mtu is set
to the default value.  If the mtu is set to a jumbo packet size or larger
we will support 200 interface addresses.

Signed-off-by: Lynne Morrison <lynne@voltanet.io>

Merge pull request #8652 from ton31337/fix/doc_bgp_alias

doc: Move BGP community alias under a separate section

Merge pull request #8589 from idryzhov/bgp-cli-nb-fixes

bgp cli/nb fixes

Merge pull request #8650 from idryzhov/bgp-fix-redist

bgpd: fix redistribution in vrf

tools: Ignore mass renaming of topotests for git blame

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

tests: Unify directory naming for topotests

Change every `-` to `_` in directory names. This is to avoid mixing _ and -.

Just for consistency and directory sorting properly.

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

tools: Fix topotest directory for Travis

ospf1-topo1 => ospf1_topo1

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