summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* nhrp, lib, zebra: add/del neighbor entry possible from nhrpPhilippe Guibert2021-04-0910-40/+154
| | | | | | | | | a zebra api is extended to offer ability to add or remove neighbor entry from daemon. Also this extension makes possible to add neigh entry, not only between IPs and macs, but also between IPs and NBMA IPs. This API supports configuring ipv6/ipv4 entries with ipv4/ipv6 lladdr. Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
* nhrpd: link layer registration to notificationsPhilippe Guibert2021-04-093-103/+91
| | | | | | | | | neighbor notifications are done in zebra. so, instead of relying on nhrp, rely on zebra by using zebra api interface. Consequently, the code originally used in nhrp for netlink neighor notification is no more used. Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
* zebra: link layer config and notification, implementation in zebraPhilippe Guibert2021-04-099-2/+134
| | | | | | | | | | | | | zebra implements zebra api for configuring link layer information. that can be an arp entry (for ipv4) or ipv6 neighbor discovery entry. This can also be an ipv4/ipv6 entry associated to an underlay ipv4 address, as it is used in gre point to multipoint interfaces. this api will also be used as monitoring. an hash list is instantiated into zebra (this is the vrf bitmap). each client interested in those entries in a specific vrf, will listen for following messages: entries added, removed, or who-has messages. Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
* lib: link layer neighbor registration and notification, define API msgsPhilippe Guibert2021-04-093-1/+26
| | | | | | | | | This patch implements new zapi api to get neighbor information that zebra knows and that other daemons may need to know. Actually, nhrp daemons is interested in getting the neighbor information on gre interfaces, and the API will be used for that. Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
* nhrpd: use CALLOC instead of MALLOCPhilippe Guibert2021-04-091-1/+1
| | | | | | | in order to apply a memset to a non initialised structure, use CALLOC instead of MALLOC Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
* nhrpd: lookup appropriate ipsec pathPhilippe Guibert2021-04-091-0/+39
| | | | | | | | | | | | | | | | | | | | | lookup appropriate ipsec path. there are systems where the path where the charon.vici file is not in standard paths. For that, 'ipsec --piddir' may help in solving the path. result of ipsec --piddir is as follow for example: ' /etc/ike/ipsec.d/run ' Note that the assumption is done that even if there are several instances of strongswan across the vrfs, the charon.vici path file is the same across vrfs. Consequently, as there is a thread per vrf that performs vici initialisation, and file path retrieval is part of the vici initialisation procedure, in order to avoid intempestive system calls, use a boolean 'vici_charon_filepath_done' to avoid doing unnecessary calls. Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
* Merge pull request #8432 from ton31337/fix/use_bool_for_use32bit_asnQuentin Young2021-04-091-1/+1
|\ | | | | bgpd: Use bool type to check if use32bit ASN
| * bgpd: Use bool type to check if use32bit ASNDonatas Abraitis2021-04-091-1/+1
| | | | | | | | Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
* | Merge pull request #8428 from idryzhov/show-memory-daemonDonald Sharp2021-04-092-4/+10
|\ \ | | | | | | vtysh: add "show memory" for a single daemon
| * | doc: add "show memory [DAEMON]"Igor Ryzhov2021-04-091-2/+3
| | | | | | | | | | | | Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>
| * | vtysh: add "show memory" for a single daemonIgor Ryzhov2021-04-081-2/+7
| | | | | | | | | | | | Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>
* | | Merge pull request #8433 from ton31337/fix/mpls_prd_dead_codeDonald Sharp2021-04-091-7/+0
|\ \ \ | | | | | | | | bgpd: Remove dead code regarding prefix_rd allocation
| * | | bgpd: Remove dead code regarding prefix_rd allocationDonatas Abraitis2021-04-091-7/+0
| | |/ | |/| | | | | | | Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
* | | Merge pull request #8422 from idryzhov/fix-show-ip-ospfDonald Sharp2021-04-092-2/+4
|\ \ \ | |/ / |/| | ospfd: fix crash on "show ip ospf neighbor detail"
| * | ospfd: fix crash on "show ip ospf neighbor detail"Igor Ryzhov2021-04-092-2/+4
| | | | | | | | | | | | | | | | | | Fixes #8419. Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>
* | | Merge pull request #8430 from ↵Patrick Ruddy2021-04-098-155/+0
|\ \ \ | | | | | | | | | | | | | | | | ton31337/fix/revert_0b6d35324ef3fd2b4d5997103795cad73eb72954 Revert "bgpd: BGP session not established for ipv6 link local address…
| * | | Revert "bgpd: BGP session not established for ipv6 link local address with ↵Donatas Abraitis2021-04-098-155/+0
| | | | | | | | | | | | | | | | | | | | | | | | vrf config" This reverts commit 0b6d35324ef3fd2b4d5997103795cad73eb72954.
* | | | Merge pull request #8424 from taspelund/allow_no_advertise_evpnDonatas Abraitis2021-04-091-13/+7
|\ \ \ \ | |/ / / |/| | | bgpd: always allow no advertise-svi-ip/default-gw
| * | | bgpd: always allow no advertise-svi-ip/default-gwTrey Aspelund2021-04-081-13/+7
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | Current behavior has an EVPN_ENABLED check for both standard and 'no' forms of 'advertise-svi-ip' and 'advertise-default-gw'. This prevents a user from removing either command from running config if 'advertise-all-vni' is not present. This commit removes/adjusts the EVPN_ENABLED checks to always allow the 'no' command so config doesn't get stuck. Signed-off-by: Trey Aspelund <taspelund@nvidia.com>
* | | Merge pull request #8402 from opensourcerouting/debian10-topotest-fixPatrick Ruddy2021-04-087-80/+71
|\ \ \ | |/ / |/| | tests: fix for l3mdev topotests on kernel 4.19
| * | tests: fix python3 backported iproute2 vrf checkDavid Schweizer2021-04-051-2/+1
| | | | | | | | | | | | Signed-off-by: David Schweizer <dschweizer@opensourcerouting.org>
| * | tests: fix l3mdev config in isis-topo1-vrf topotestDavid Schweizer2021-04-051-25/+6
| | | | | | | | | | | | Signed-off-by: David Schweizer <dschweizer@opensourcerouting.org>
| * | tests: fix l3mdev config in bgp_vrf_lite_ipv6_rtadv topotestDavid Schweizer2021-04-051-25/+3
| | | | | | | | | | | | Signed-off-by: David Schweizer <dschweizer@opensourcerouting.org>
| * | tests: fix l3mdev config in bgp_l3vpn_to_bgp_vrf topotestDavid Schweizer2021-04-052-18/+9
| | | | | | | | | | | | Signed-off-by: David Schweizer <dschweizer@opensourcerouting.org>
| * | tests: fix l3mdev config in bgp_evpn_rt5 topotestDavid Schweizer2021-04-051-10/+2
| | | | | | | | | | | | Signed-off-by: David Schweizer <dschweizer@opensourcerouting.org>
| * | tests: add l3mdev config helpers for VRF topotestsDavid Schweizer2021-04-051-0/+50
| | | | | | | | | | | | Signed-off-by: David Schweizer <dschweizer@opensourcerouting.org>
* | | Merge pull request #7434 from sudhanshukumar22/bgp-link-local-addressDonatas Abraitis2021-04-088-0/+155
|\ \ \ | | | | | | | | bgpd: BGP session not established for ipv6 link local address with vrf config
| * | | bgpd: BGP session not established for ipv6 link local address with vrf configsudhanshukumar222021-04-018-0/+155
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Description: BGP session not established for ipv6 link local address with vrf config Problem Description/Summary : BGP session not established for ipv6 link local address with vrf configyy 1.Configure ipv6 link-local address fe80::1234/64 on dut1 and fe80::4567/64 on dut2 2.Configure BGP neighbors for ipv6 link-local on both dut1 and dut2 3.Verify BGP session is UP over link-local ipv6 address 4.Observed that bgp session not established for ipv6 link local address Expected Behavior : BGP session should be established for ipv6 link local address with vrf config Signed-off-by: sudhanshukumar22 <sudhanshu.kumar@broadcom.com>
* | | | Merge pull request #8365 from ↵Mark Stapp2021-04-076-338/+358
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | aldobrrrr/fix_parse_topology_function_in_isis_topo1_vrf_test topotests: fix 'parse_topology' function in test_isis_topo1_vrf.py
| * | | | topotests: fix 'parse_topology' function in test_isis_topo1_vrf.pyEmanuele Altomare2021-03-296-338/+358
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now the parse_topology function handles well the vertex type identifier. Signed-off-by: Emanuele Altomare <emanuele@common-net.org>
* | | | | Merge pull request #8415 from mjstapp/fix_ospf_keyid_configIgor Ryzhov2021-04-071-4/+3
|\ \ \ \ \ | | | | | | | | | | | | ospfd: permit over-write of message-digest-key
| * | | | | ospfd: permit over-write of message-digest-keyMark Stapp2021-04-061-4/+3
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow over-write of message-digest-key interface config. Most attributes handle multi-instance by ... ignoring instances, and tolerating repeated config: do the same for md5 auth. Signed-off-by: Mark Stapp <mjs@voltanet.io>
* | | | | Merge pull request #8413 from mjstapp/zebra_doc_debugsQuentin Young2021-04-071-0/+65
|\ \ \ \ \ | | | | | | | | | | | | docs: add zebra debug docs
| * | | | | docs: add zebra debug docsMark Stapp2021-04-061-0/+65
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | Add docs for the zebra debug commands. Signed-off-by: Mark Stapp <mjs@voltanet.io>
* | | | | Merge pull request #8411 from donaldsharp/nhrp_reallyJafar Al-Gharaibeh2021-04-062-6/+3
|\ \ \ \ \ | |/ / / / |/| | | | nhrpd: Fixup tab usage and sockunion2str usage
| * | | | nhrpd: Fixup tab usage and sockunion2str usageDonald Sharp2021-04-062-6/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For some reason the usage of tabs in a string snuck in as well as using a sockunion2str instead of %pSU. Fix. Signed-off-by: Donald Sharp <sharpd@nvidia.com>
* | | | | Merge pull request #8405 from mjstapp/skip_backup_lspsRuss White2021-04-063-8/+40
|\ \ \ \ \ | | | | | | | | | | | | zebra: don't process LSPs with backups immediately
| * | | | | zebra: be more selective about processing LSPsMark Stapp2021-04-051-0/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When certain events occur (connected route changes e.g.) zebra examines LSPs to see if they might have been affected. For LSPs with backup nhlfes, skip this immediate processing and wait for the owning protocol daemon to react. Signed-off-by: Mark Stapp <mjs@voltanet.io>
| * | | | | zebra: add 'detail' mpls debug settingMark Stapp2021-04-052-8/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add setting and cli for 'debug zebra mpls detail'. Signed-off-by: Mark Stapp <mjs@voltanet.io>
* | | | | | Merge pull request #8407 from ↵Russ White2021-04-068-0/+157
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | | | | | | | | | | | | | ton31337/fix/nexthop_should_not_be_changed_if_peer_shares_common_subnet tests: Nexthop should not be changed if peer X shares a common subnet
| * | | | | tests: Nexthop should not be changed if peer X shares a common subnetDonatas Abraitis2021-04-068-0/+157
| | |_|_|/ | |/| | | | | | | | | | | | | Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
* | | | | Merge pull request #8295 from opensourcerouting/ospf6-topo-stabilizeMark Stapp2021-04-0610-94/+113
|\ \ \ \ \ | | | | | | | | | | | | topotest: stabilize OSPFv3 topology
| * | | | | topotests: improve OSPF(v2|v3) convergence testRafael Zalamena2021-03-231-0/+72
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Wait until all routers are actually `Full` instead of just looking at routes. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
| * | | | | topotests: remove old configuration filesRafael Zalamena2021-03-235-39/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We version the tests with FRR so it is no longer necessary to support old versions. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
| * | | | | topotest: stabilize OSPFv3 topologyRafael Zalamena2021-03-235-55/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Changes: - Decrease hello interval to avoid packet loss slow downs - Decrease dead interval to converge faster - Remove previous 'Full' state check that wasn't checking for all peers (only one per router) Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
* | | | | | Merge pull request #8360 from kuldeepkash/bgp_graceful_restartMartin Winter2021-04-064-25/+184
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | tests: Fix for BGP_GR test failures
| * | | | | tests: Fix for BGP_GR test failuresKuldeep Kashyap2021-04-054-25/+184
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | Few tests were failing in BGP-GR intermittently. Fixing the failures. Signed-off-by: Kuldeep Kashyap <kashyapk@vmware.com>
* | | | | Merge pull request #8391 from idryzhov/fix-bgp-nb-prefix-listRenato Westphal2021-04-061-8/+5
|\ \ \ \ \ | | | | | | | | | | | | bgpd: prefix-list and route-map nb fixes
| * | | | | bgpd: prefix-list and route-map nb fixesIgor Ryzhov2021-04-011-8/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix missing/incorrect callbacks. Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>
* | | | | | Merge pull request #8384 from volta-networks/fix_no_srb_blockQuentin Young2021-04-061-0/+10
|\ \ \ \ \ \ | | | | | | | | | | | | | | tools: do in-place SRGB/SRLB changes