-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
zebra: netlink FPM interface using zebra data plane #5510
zebra: netlink FPM interface using zebra data plane #5510
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your contribution to FRR!
Click for style suggestions
To apply these suggestions:
curl -s https://gist.githubusercontent.com/polychaeta/578b311aa65bbcc21c3ae592df5309be/raw/7841e7ea7690fee10e7b5e05785b805b0e53a776/cr_5510_1575920414.diff | git apply
diff --git a/zebra/rt_netlink.c b/zebra/rt_netlink.c
index 96064be6a..ad5dee43a 100644
--- a/zebra/rt_netlink.c
+++ b/zebra/rt_netlink.c
@@ -1258,8 +1258,8 @@ static void _netlink_route_build_singlepath(const char *routedesc, int bytelen,
}
if (IS_ZEBRA_DEBUG_KERNEL)
- zlog_debug("%s: (%s): nexthop via if %u(%u)",
- __func__, routedesc, nexthop->ifindex,
+ zlog_debug("%s: (%s): nexthop via if %u(%u)", __func__,
+ routedesc, nexthop->ifindex,
nexthop->vrf_id);
}
}
@@ -1356,8 +1356,8 @@ static void _netlink_route_build_multipath(const char *routedesc, int bytelen,
if (IS_ZEBRA_DEBUG_KERNEL)
zlog_debug("%s: 5549 (%s): nexthop via %s %s if %u",
- __func__, routedesc, ipv4_ll_buf, label_buf,
- nexthop->ifindex);
+ __func__, routedesc, ipv4_ll_buf, label_buf,
+ nexthop->ifindex);
return;
}
@@ -1374,7 +1374,7 @@ static void _netlink_route_build_multipath(const char *routedesc, int bytelen,
if (IS_ZEBRA_DEBUG_KERNEL) {
inet_ntop(AF_INET, &nexthop->gate.ipv4, addrstr,
sizeof(addrstr));
- zlog_debug( "%s: (%s): nexthop via %s %s if %u",
+ zlog_debug("%s: (%s): nexthop via %s %s if %u",
__func__, routedesc, addrstr, label_buf,
nexthop->ifindex);
}
@@ -1393,7 +1393,7 @@ static void _netlink_route_build_multipath(const char *routedesc, int bytelen,
if (IS_ZEBRA_DEBUG_KERNEL) {
inet_ntop(AF_INET, &nexthop->gate.ipv6, addrstr,
sizeof(addrstr));
- zlog_debug( "%s: (%s): nexthop via %s %s if %u",
+ zlog_debug("%s: (%s): nexthop via %s %s if %u",
__func__, routedesc, addrstr, label_buf,
nexthop->ifindex);
}
@@ -1415,8 +1415,8 @@ static void _netlink_route_build_multipath(const char *routedesc, int bytelen,
*src = &nexthop->src;
if (IS_ZEBRA_DEBUG_KERNEL)
- zlog_debug("%s: (%s): nexthop via if %u",
- __func__, routedesc, nexthop->ifindex);
+ zlog_debug("%s: (%s): nexthop via if %u", __func__,
+ routedesc, nexthop->ifindex);
}
}
@@ -1586,10 +1586,9 @@ ssize_t netlink_route_multipath(int cmd, struct zebra_dplane_ctx *ctx,
}
if (IS_ZEBRA_DEBUG_KERNEL)
- zlog_debug(
- "%s: %s %pFX vrf %u(%u)", __func__,
- nl_msg_type_to_str(cmd), p, dplane_ctx_get_vrf(ctx),
- table_id);
+ zlog_debug("%s: %s %pFX vrf %u(%u)", __func__,
+ nl_msg_type_to_str(cmd), p, dplane_ctx_get_vrf(ctx),
+ table_id);
/*
* If we are not updating the route and we have received
@@ -2150,8 +2149,8 @@ enum zebra_dplane_result kernel_route_update(struct zebra_dplane_ctx *ctx)
* route should cause us to withdraw from
* the kernel the old non-system route
*/
- if (RSYSTEM_ROUTE(dplane_ctx_get_type(ctx)) &&
- !RSYSTEM_ROUTE(dplane_ctx_get_old_type(ctx))) {
+ if (RSYSTEM_ROUTE(dplane_ctx_get_type(ctx))
+ && !RSYSTEM_ROUTE(dplane_ctx_get_old_type(ctx))) {
netlink_route_multipath(RTM_DELROUTE, ctx,
nl_pkt, sizeof(nl_pkt));
netlink_talk_info(netlink_talk_filter,
@@ -2881,9 +2880,8 @@ int netlink_macfdb_read_specific_mac(struct zebra_ns *zns,
/*
* Netlink-specific handler for MAC updates using dataplane context object.
*/
-ssize_t
-netlink_macfdb_update_ctx(struct zebra_dplane_ctx *ctx, uint8_t *data,
- size_t datalen)
+ssize_t netlink_macfdb_update_ctx(struct zebra_dplane_ctx *ctx, uint8_t *data,
+ size_t datalen)
{
uint8_t protocol = RTPROT_ZEBRA;
struct {
@@ -2956,8 +2954,8 @@ netlink_macfdb_update_ctx(struct zebra_dplane_ctx *ctx, uint8_t *data,
nl_family_to_str(req->ndm.ndm_family),
dplane_ctx_get_ifname(ctx),
dplane_ctx_get_ifindex(ctx), vid_buf,
- dplane_ctx_mac_is_sticky(ctx) ? "sticky " : "",
- buf, dst_buf);
+ dplane_ctx_mac_is_sticky(ctx) ? "sticky " : "", buf,
+ dst_buf);
}
return NLMSG_ALIGN(req->n.nlmsg_len);
@@ -3402,10 +3400,10 @@ enum zebra_dplane_result kernel_mac_update_ctx(struct zebra_dplane_ctx *ctx)
return ZEBRA_DPLANE_REQUEST_FAILURE;
rv = netlink_talk_info(netlink_talk_filter, (struct nlmsghdr *)nl_buf,
- dplane_ctx_get_ns(ctx), 0);
+ dplane_ctx_get_ns(ctx), 0);
- return rv == 0 ?
- ZEBRA_DPLANE_REQUEST_SUCCESS : ZEBRA_DPLANE_REQUEST_FAILURE;
+ return rv == 0 ? ZEBRA_DPLANE_REQUEST_SUCCESS
+ : ZEBRA_DPLANE_REQUEST_FAILURE;
}
enum zebra_dplane_result kernel_neigh_update_ctx(struct zebra_dplane_ctx *ctx)
diff --git a/zebra/zebra_dplane.c b/zebra/zebra_dplane.c
index 30271f0c2..0975b0ed3 100644
--- a/zebra/zebra_dplane.c
+++ b/zebra/zebra_dplane.c
@@ -2402,8 +2402,8 @@ enum zebra_dplane_result dplane_mac_add(const struct interface *ifp,
enum zebra_dplane_result result;
/* Use common helper api */
- ctx = mac_update_internal(DPLANE_OP_MAC_INSTALL, ifp, bridge_ifp,
- vid, mac, vtep_ip, sticky);
+ ctx = mac_update_internal(DPLANE_OP_MAC_INSTALL, ifp, bridge_ifp, vid,
+ mac, vtep_ip, sticky);
/* Enqueue for processing on the dplane pthread */
result = dplane_update_enqueue(ctx);
@@ -2437,8 +2437,8 @@ enum zebra_dplane_result dplane_mac_del(const struct interface *ifp,
enum zebra_dplane_result result;
/* Use common helper api */
- ctx = mac_update_internal(DPLANE_OP_MAC_DELETE, ifp, bridge_ifp,
- vid, mac, vtep_ip, false);
+ ctx = mac_update_internal(DPLANE_OP_MAC_DELETE, ifp, bridge_ifp, vid,
+ mac, vtep_ip, false);
/* Enqueue for processing on the dplane pthread */
result = dplane_update_enqueue(ctx);
@@ -2463,12 +2463,9 @@ enum zebra_dplane_result dplane_mac_del(const struct interface *ifp,
* Common helper api for MAC address/vxlan updates
*/
struct zebra_dplane_ctx *
-mac_update_internal(enum dplane_op_e op,
- const struct interface *ifp,
- const struct interface *br_ifp,
- vlanid_t vid,
- const struct ethaddr *mac,
- struct in_addr vtep_ip,
+mac_update_internal(enum dplane_op_e op, const struct interface *ifp,
+ const struct interface *br_ifp, vlanid_t vid,
+ const struct ethaddr *mac, struct in_addr vtep_ip,
bool sticky)
{
struct zebra_dplane_ctx *ctx = NULL;
diff --git a/zebra/zebra_dplane.h b/zebra/zebra_dplane.h
index f6352b27d..a6ac983f2 100644
--- a/zebra/zebra_dplane.h
+++ b/zebra/zebra_dplane.h
@@ -430,11 +430,11 @@ enum zebra_dplane_result dplane_intf_addr_unset(const struct interface *ifp,
/*
* Enqueue evpn mac operations for the dataplane.
*/
-extern struct zebra_dplane_ctx *mac_update_internal(
- enum dplane_op_e op, const struct interface *ifp,
- const struct interface *br_ifp,
- vlanid_t vid, const struct ethaddr *mac,
- struct in_addr vtep_ip, bool sticky);
+extern struct zebra_dplane_ctx *
+mac_update_internal(enum dplane_op_e op, const struct interface *ifp,
+ const struct interface *br_ifp, vlanid_t vid,
+ const struct ethaddr *mac, struct in_addr vtep_ip,
+ bool sticky);
enum zebra_dplane_result dplane_mac_add(const struct interface *ifp,
const struct interface *bridge_ifp,
diff --git a/zebra/zebra_vxlan_private.h b/zebra/zebra_vxlan_private.h
index 73ebb1734..1b0193b70 100644
--- a/zebra/zebra_vxlan_private.h
+++ b/zebra/zebra_vxlan_private.h
@@ -303,7 +303,7 @@ struct zebra_mac_t_ {
/* remote VTEP advertised MAC as default GW */
#define ZEBRA_MAC_REMOTE_DEF_GW 0x40
#define ZEBRA_MAC_DUPLICATE 0x80
-#define ZEBRA_MAC_FPM_SENT 0x100 /* whether or not this entry was sent. */
+#define ZEBRA_MAC_FPM_SENT 0x100 /* whether or not this entry was sent. */
/* back pointer to zvni */
zebra_vni_t *zvni;
If you are a new contributor to FRR, please see our contributing guidelines.
💚 Outdated resultsBasic BGPD CI results: SUCCESS, 0 tests failed
For details, please contact louberger |
Continuous Integration Result: FAILEDContinuous Integration Result: FAILEDSee below for issues. This is a comment from an automated CI system. Get source / Pull Request: SuccessfulBuilding Stage: FailedUbuntu 16.04 i386 build: Failed (click for details)Make failed for Ubuntu 16.04 i386 build:
Ubuntu 16.04 i386 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10026/artifact/U1604I386/config.status/config.status Ubuntu 18.04 ppc64le build: Failed (click for details)Ubuntu 18.04 ppc64le build: config.log output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10026/artifact/U1804PPC64LEBUILD/config.log/ Ubuntu 18.04 ppc64le build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10026/artifact/U1804PPC64LEBUILD/config.status/config.statusPackage building failed for Ubuntu 18.04 ppc64le build:
NetBSD 7 amd64 build: Failed (click for details)Make failed for NetBSD 7 amd64 build:
NetBSD 7 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10026/artifact/CI012BUILD/config.status/config.status CentOS 7 amd64 build: Failed (click for details)Package building failed for CentOS 7 amd64 build Fedora 29 amd64 build: Failed (click for details)Fedora 29 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10026/artifact/F29BUILD/config.status/config.statusPackage building failed for Fedora 29 amd64 build Debian 10 amd64 build: Failed (click for details)Debian 10 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10026/artifact/DEB10BUILD/config.status/config.statusPackage building failed for Debian 10 amd64 build:
OpenBSD 6 amd64 build: Failed (click for details)Make failed for OpenBSD 6 amd64 build:
OpenBSD 6 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10026/artifact/CI011BUILD/config.status/config.status FreeBSD 11 amd64 build: Failed (click for details)Make failed for FreeBSD 11 amd64 build:
FreeBSD 11 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10026/artifact/CI009BUILD/config.status/config.status NetBSD 6 amd64 build: Failed (click for details)Make failed for NetBSD 6 amd64 build:
NetBSD 6 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10026/artifact/CI007BUILD/config.status/config.status Ubuntu 18.04 amd64 build: Failed (click for details)Package building failed for Ubuntu 18.04 amd64 build:
Ubuntu 18.04 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10026/artifact/U1804AMD64/config.status/config.status FreeBSD 12 amd64 build: Failed (click for details)FreeBSD 12 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10026/artifact/FBSD12AMD64/config.status/config.statusMake failed for FreeBSD 12 amd64 build:
Successful on other platforms
Warnings Generated during build:Checkout code: Successful with additional warningsUbuntu 16.04 i386 build: Failed (click for details)Make failed for Ubuntu 16.04 i386 build:
Ubuntu 16.04 i386 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10026/artifact/U1604I386/config.status/config.status Ubuntu 18.04 ppc64le build: Failed (click for details)Ubuntu 18.04 ppc64le build: config.log output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10026/artifact/U1804PPC64LEBUILD/config.log/ Ubuntu 18.04 ppc64le build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10026/artifact/U1804PPC64LEBUILD/config.status/config.statusPackage building failed for Ubuntu 18.04 ppc64le build:
NetBSD 7 amd64 build: Failed (click for details)Make failed for NetBSD 7 amd64 build:
NetBSD 7 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10026/artifact/CI012BUILD/config.status/config.status CentOS 7 amd64 build: Failed (click for details)Package building failed for CentOS 7 amd64 build Fedora 29 amd64 build: Failed (click for details)Fedora 29 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10026/artifact/F29BUILD/config.status/config.statusPackage building failed for Fedora 29 amd64 build Debian 10 amd64 build: Failed (click for details)Debian 10 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10026/artifact/DEB10BUILD/config.status/config.statusPackage building failed for Debian 10 amd64 build:
OpenBSD 6 amd64 build: Failed (click for details)Make failed for OpenBSD 6 amd64 build:
OpenBSD 6 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10026/artifact/CI011BUILD/config.status/config.status FreeBSD 11 amd64 build: Failed (click for details)Make failed for FreeBSD 11 amd64 build:
FreeBSD 11 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10026/artifact/CI009BUILD/config.status/config.status NetBSD 6 amd64 build: Failed (click for details)Make failed for NetBSD 6 amd64 build:
NetBSD 6 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10026/artifact/CI007BUILD/config.status/config.status Ubuntu 18.04 amd64 build: Failed (click for details)Package building failed for Ubuntu 18.04 amd64 build:
Ubuntu 18.04 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10026/artifact/U1804AMD64/config.status/config.status FreeBSD 12 amd64 build: Failed (click for details)FreeBSD 12 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10026/artifact/FBSD12AMD64/config.status/config.statusMake failed for FreeBSD 12 amd64 build:
|
Ops: CI needs to handle the new module, but lets wait this PR becomes mergeable before doing any changes there. |
@rzalamena what information do you need regarding "I would like to receive reports from FPM netlink users that use RMAC since I don't have the appropriate tools to test it." ?? We (VMWare) do use FPM interface and also RMAC. |
@srimohans I'm interested to hear if this new FPM module works for you. Start
To run with the new module you have to use:
|
@rzalamena Sure. Will ping you on slack for more details. Thanks. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had some questions - I'll take a look and see if I can help make the dplane apis work a bit better for you.
zebra/dplane_fpm_nl.c
Outdated
} | ||
|
||
/* Enqueue route install. */ | ||
ctx = dplane_ctx_alloc(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
so let's come up with a way for you to re-use a context object in this loop, so we don't have to do so many alloc/free cycles. the code's all there to init / re-init the objectg - it's just a matter of making it available in the right way so it's easy for you to use?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you can manage it in a easy way it would be great. I did not want to touch this part because it seem to do more allocations in the opaque struct zebra_dplane_ctx
and I didn't want to add more complexity to this PR.
enum dplane_op_e op, | ||
struct route_node *rn, | ||
struct route_entry *re) | ||
int dplane_ctx_route_init(struct zebra_dplane_ctx *ctx, enum dplane_op_e op, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm good with exposing this, but I think we could make the path more useful by ensuring we get a memset in the path, so we can reuse a ctx in the rib walk.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What happens with the allocated data in that function?
Example:
/* Copy nexthops; recursive info is included too */
copy_nexthops(&(ctx->u.rinfo.zd_ng.nexthop),
re->nhe->nhg->nexthop, NULL);
Continuous Integration Result: FAILEDContinuous Integration Result: FAILEDSee below for issues. This is a comment from an automated CI system. Get source / Pull Request: SuccessfulBuilding Stage: FailedFedora 29 amd64 build: Failed (click for details)Fedora 29 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10070/artifact/F29BUILD/config.status/config.statusPackage building failed for Fedora 29 amd64 build FreeBSD 12 amd64 build: Failed (click for details)FreeBSD 12 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10070/artifact/FBSD12AMD64/config.status/config.statusMake failed for FreeBSD 12 amd64 build:
Debian 10 amd64 build: Failed (click for details)Debian 10 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10070/artifact/DEB10BUILD/config.status/config.statusPackage building failed for Debian 10 amd64 build:
Ubuntu 16.04 i386 build: Failed (click for details)Make failed for Ubuntu 16.04 i386 build:
Ubuntu 16.04 i386 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10070/artifact/U1604I386/config.status/config.status Ubuntu 18.04 amd64 build: Failed (click for details)Package building failed for Ubuntu 18.04 amd64 build:
Ubuntu 18.04 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10070/artifact/U1804AMD64/config.status/config.status FreeBSD 11 amd64 build: Failed (click for details)Make failed for FreeBSD 11 amd64 build:
FreeBSD 11 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10070/artifact/CI009BUILD/config.status/config.status NetBSD 6 amd64 build: Failed (click for details)Make failed for NetBSD 6 amd64 build:
NetBSD 6 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10070/artifact/CI007BUILD/config.status/config.status CentOS 7 amd64 build: Failed (click for details)Package building failed for CentOS 7 amd64 build Ubuntu 18.04 ppc64le build: Failed (click for details)Ubuntu 18.04 ppc64le build: config.log output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10070/artifact/U1804PPC64LEBUILD/config.log/ Ubuntu 18.04 ppc64le build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10070/artifact/U1804PPC64LEBUILD/config.status/config.statusPackage building failed for Ubuntu 18.04 ppc64le build:
OpenBSD 6 amd64 build: Failed (click for details)Make failed for OpenBSD 6 amd64 build:
OpenBSD 6 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10070/artifact/CI011BUILD/config.status/config.status NetBSD 8 amd64 build: Failed (click for details)Make failed for NetBSD 8 amd64 build:
NetBSD 8 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10070/artifact/CI012BUILD/config.status/config.status Successful on other platforms
Warnings Generated during build:Checkout code: Successful with additional warningsFedora 29 amd64 build: Failed (click for details)Fedora 29 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10070/artifact/F29BUILD/config.status/config.statusPackage building failed for Fedora 29 amd64 build FreeBSD 12 amd64 build: Failed (click for details)FreeBSD 12 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10070/artifact/FBSD12AMD64/config.status/config.statusMake failed for FreeBSD 12 amd64 build:
Debian 10 amd64 build: Failed (click for details)Debian 10 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10070/artifact/DEB10BUILD/config.status/config.statusPackage building failed for Debian 10 amd64 build:
Ubuntu 16.04 i386 build: Failed (click for details)Make failed for Ubuntu 16.04 i386 build:
Ubuntu 16.04 i386 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10070/artifact/U1604I386/config.status/config.status Ubuntu 18.04 amd64 build: Failed (click for details)Package building failed for Ubuntu 18.04 amd64 build:
Ubuntu 18.04 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10070/artifact/U1804AMD64/config.status/config.status FreeBSD 11 amd64 build: Failed (click for details)Make failed for FreeBSD 11 amd64 build:
FreeBSD 11 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10070/artifact/CI009BUILD/config.status/config.status NetBSD 6 amd64 build: Failed (click for details)Make failed for NetBSD 6 amd64 build:
NetBSD 6 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10070/artifact/CI007BUILD/config.status/config.status CentOS 7 amd64 build: Failed (click for details)Package building failed for CentOS 7 amd64 build Ubuntu 18.04 ppc64le build: Failed (click for details)Ubuntu 18.04 ppc64le build: config.log output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10070/artifact/U1804PPC64LEBUILD/config.log/ Ubuntu 18.04 ppc64le build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10070/artifact/U1804PPC64LEBUILD/config.status/config.statusPackage building failed for Ubuntu 18.04 ppc64le build:
OpenBSD 6 amd64 build: Failed (click for details)Make failed for OpenBSD 6 amd64 build:
OpenBSD 6 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10070/artifact/CI011BUILD/config.status/config.status NetBSD 8 amd64 build: Failed (click for details)Make failed for NetBSD 8 amd64 build:
NetBSD 8 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10070/artifact/CI012BUILD/config.status/config.status
|
💚 Outdated resultsBasic BGPD CI results: SUCCESS, 0 tests failed
For details, please contact louberger |
34ce451
to
c8485a4
Compare
Continuous Integration Result: FAILEDContinuous Integration Result: FAILEDSee below for issues. This is a comment from an automated CI system. Get source / Pull Request: SuccessfulBuilding Stage: FailedOpenBSD 6 amd64 build: Failed (click for details)Make failed for OpenBSD 6 amd64 build:
OpenBSD 6 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10072/artifact/CI011BUILD/config.status/config.status FreeBSD 11 amd64 build: Failed (click for details)Make failed for FreeBSD 11 amd64 build:
FreeBSD 11 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10072/artifact/CI009BUILD/config.status/config.status Fedora 29 amd64 build: Failed (click for details)Fedora 29 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10072/artifact/F29BUILD/config.status/config.statusPackage building failed for Fedora 29 amd64 build Debian 10 amd64 build: Failed (click for details)Debian 10 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10072/artifact/DEB10BUILD/config.status/config.statusPackage building failed for Debian 10 amd64 build:
FreeBSD 12 amd64 build: Failed (click for details)FreeBSD 12 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10072/artifact/FBSD12AMD64/config.status/config.statusMake failed for FreeBSD 12 amd64 build:
NetBSD 8 amd64 build: Failed (click for details)Make failed for NetBSD 8 amd64 build:
NetBSD 8 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10072/artifact/CI012BUILD/config.status/config.status Ubuntu 16.04 i386 build: Failed (click for details)Make failed for Ubuntu 16.04 i386 build:
Ubuntu 16.04 i386 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10072/artifact/U1604I386/config.status/config.status Ubuntu 18.04 amd64 build: Failed (click for details)Package building failed for Ubuntu 18.04 amd64 build:
Ubuntu 18.04 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10072/artifact/U1804AMD64/config.status/config.status NetBSD 6 amd64 build: Failed (click for details)Make failed for NetBSD 6 amd64 build:
NetBSD 6 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10072/artifact/CI007BUILD/config.status/config.status CentOS 7 amd64 build: Failed (click for details)Package building failed for CentOS 7 amd64 build Ubuntu 18.04 ppc64le build: Failed (click for details)Ubuntu 18.04 ppc64le build: config.log output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10072/artifact/U1804PPC64LEBUILD/config.log/ Ubuntu 18.04 ppc64le build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10072/artifact/U1804PPC64LEBUILD/config.status/config.statusPackage building failed for Ubuntu 18.04 ppc64le build:
Successful on other platforms
Warnings Generated during build:Checkout code: Successful with additional warningsOpenBSD 6 amd64 build: Failed (click for details)Make failed for OpenBSD 6 amd64 build:
OpenBSD 6 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10072/artifact/CI011BUILD/config.status/config.status FreeBSD 11 amd64 build: Failed (click for details)Make failed for FreeBSD 11 amd64 build:
FreeBSD 11 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10072/artifact/CI009BUILD/config.status/config.status Fedora 29 amd64 build: Failed (click for details)Fedora 29 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10072/artifact/F29BUILD/config.status/config.statusPackage building failed for Fedora 29 amd64 build Debian 10 amd64 build: Failed (click for details)Debian 10 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10072/artifact/DEB10BUILD/config.status/config.statusPackage building failed for Debian 10 amd64 build:
FreeBSD 12 amd64 build: Failed (click for details)FreeBSD 12 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10072/artifact/FBSD12AMD64/config.status/config.statusMake failed for FreeBSD 12 amd64 build:
NetBSD 8 amd64 build: Failed (click for details)Make failed for NetBSD 8 amd64 build:
NetBSD 8 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10072/artifact/CI012BUILD/config.status/config.status Ubuntu 16.04 i386 build: Failed (click for details)Make failed for Ubuntu 16.04 i386 build:
Ubuntu 16.04 i386 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10072/artifact/U1604I386/config.status/config.status Ubuntu 18.04 amd64 build: Failed (click for details)Package building failed for Ubuntu 18.04 amd64 build:
Ubuntu 18.04 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10072/artifact/U1804AMD64/config.status/config.status NetBSD 6 amd64 build: Failed (click for details)Make failed for NetBSD 6 amd64 build:
NetBSD 6 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10072/artifact/CI007BUILD/config.status/config.status CentOS 7 amd64 build: Failed (click for details)Package building failed for CentOS 7 amd64 build Ubuntu 18.04 ppc64le build: Failed (click for details)Ubuntu 18.04 ppc64le build: config.log output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10072/artifact/U1804PPC64LEBUILD/config.log/ Ubuntu 18.04 ppc64le build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10072/artifact/U1804PPC64LEBUILD/config.status/config.statusPackage building failed for Ubuntu 18.04 ppc64le build:
|
💚 Outdated resultsBasic BGPD CI results: SUCCESS, 0 tests failed
For details, please contact louberger |
💚 Outdated resultsBasic BGPD CI results: SUCCESS, 0 tests failed
For details, please contact louberger |
Continuous Integration Result: FAILEDContinuous Integration Result: FAILEDSee below for issues. This is a comment from an automated CI system. Get source / Pull Request: SuccessfulBuilding Stage: FailedUbuntu 18.04 amd64 build: Failed (click for details)Package building failed for Ubuntu 18.04 amd64 build:
Ubuntu 18.04 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10084/artifact/U1804AMD64/config.status/config.status FreeBSD 12 amd64 build: Failed (click for details)FreeBSD 12 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10084/artifact/FBSD12AMD64/config.status/config.statusMake failed for FreeBSD 12 amd64 build:
Ubuntu 16.04 i386 build: Failed (click for details)Make failed for Ubuntu 16.04 i386 build:
Ubuntu 16.04 i386 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10084/artifact/U1604I386/config.status/config.status FreeBSD 11 amd64 build: Failed (click for details)Make failed for FreeBSD 11 amd64 build:
FreeBSD 11 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10084/artifact/CI009BUILD/config.status/config.status Ubuntu 18.04 ppc64le build: Failed (click for details)Ubuntu 18.04 ppc64le build: config.log output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10084/artifact/U1804PPC64LEBUILD/config.log/ Ubuntu 18.04 ppc64le build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10084/artifact/U1804PPC64LEBUILD/config.status/config.statusPackage building failed for Ubuntu 18.04 ppc64le build:
OpenBSD 6 amd64 build: Failed (click for details)Make failed for OpenBSD 6 amd64 build:
OpenBSD 6 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10084/artifact/CI011BUILD/config.status/config.status NetBSD 8 amd64 build: Failed (click for details)Make failed for NetBSD 8 amd64 build:
NetBSD 8 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10084/artifact/CI012BUILD/config.status/config.status NetBSD 6 amd64 build: Failed (click for details)Make failed for NetBSD 6 amd64 build:
NetBSD 6 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10084/artifact/CI007BUILD/config.status/config.status CentOS 7 amd64 build: Failed (click for details)Package building failed for CentOS 7 amd64 build Debian 10 amd64 build: Failed (click for details)Debian 10 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10084/artifact/DEB10BUILD/config.status/config.statusPackage building failed for Debian 10 amd64 build:
Fedora 29 amd64 build: Failed (click for details)Fedora 29 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10084/artifact/F29BUILD/config.status/config.statusPackage building failed for Fedora 29 amd64 build Successful on other platforms
Warnings Generated during build:Checkout code: Successful with additional warningsUbuntu 18.04 amd64 build: Failed (click for details)Package building failed for Ubuntu 18.04 amd64 build:
Ubuntu 18.04 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10084/artifact/U1804AMD64/config.status/config.status FreeBSD 12 amd64 build: Failed (click for details)FreeBSD 12 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10084/artifact/FBSD12AMD64/config.status/config.statusMake failed for FreeBSD 12 amd64 build:
Ubuntu 16.04 i386 build: Failed (click for details)Make failed for Ubuntu 16.04 i386 build:
Ubuntu 16.04 i386 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10084/artifact/U1604I386/config.status/config.status FreeBSD 11 amd64 build: Failed (click for details)Make failed for FreeBSD 11 amd64 build:
FreeBSD 11 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10084/artifact/CI009BUILD/config.status/config.status Ubuntu 18.04 ppc64le build: Failed (click for details)Ubuntu 18.04 ppc64le build: config.log output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10084/artifact/U1804PPC64LEBUILD/config.log/ Ubuntu 18.04 ppc64le build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10084/artifact/U1804PPC64LEBUILD/config.status/config.statusPackage building failed for Ubuntu 18.04 ppc64le build:
OpenBSD 6 amd64 build: Failed (click for details)Make failed for OpenBSD 6 amd64 build:
OpenBSD 6 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10084/artifact/CI011BUILD/config.status/config.status NetBSD 8 amd64 build: Failed (click for details)Make failed for NetBSD 8 amd64 build:
NetBSD 8 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10084/artifact/CI012BUILD/config.status/config.status NetBSD 6 amd64 build: Failed (click for details)Make failed for NetBSD 6 amd64 build:
NetBSD 6 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10084/artifact/CI007BUILD/config.status/config.status CentOS 7 amd64 build: Failed (click for details)Package building failed for CentOS 7 amd64 build Debian 10 amd64 build: Failed (click for details)Debian 10 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10084/artifact/DEB10BUILD/config.status/config.statusPackage building failed for Debian 10 amd64 build:
Fedora 29 amd64 build: Failed (click for details)Fedora 29 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10084/artifact/F29BUILD/config.status/config.statusPackage building failed for Fedora 29 amd64 build
|
Continuous Integration Result: FAILEDContinuous Integration Result: FAILEDSee below for issues. This is a comment from an automated CI system. Get source / Pull Request: SuccessfulBuilding Stage: FailedUbuntu 18.04 ppc64le build: Failed (click for details)Ubuntu 18.04 ppc64le build: config.log output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10096/artifact/U1804PPC64LEBUILD/config.log/ Ubuntu 18.04 ppc64le build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10096/artifact/U1804PPC64LEBUILD/config.status/config.statusPackage building failed for Ubuntu 18.04 ppc64le build:
Ubuntu 16.04 i386 build: Failed (click for details)Make failed for Ubuntu 16.04 i386 build:
Ubuntu 16.04 i386 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10096/artifact/U1604I386/config.status/config.status Fedora 29 amd64 build: Failed (click for details)Fedora 29 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10096/artifact/F29BUILD/config.status/config.statusPackage building failed for Fedora 29 amd64 build NetBSD 6 amd64 build: Failed (click for details)Make failed for NetBSD 6 amd64 build:
NetBSD 6 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10096/artifact/CI007BUILD/config.status/config.status CentOS 7 amd64 build: Failed (click for details)Package building failed for CentOS 7 amd64 build Debian 10 amd64 build: Failed (click for details)Debian 10 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10096/artifact/DEB10BUILD/config.status/config.statusPackage building failed for Debian 10 amd64 build:
FreeBSD 12 amd64 build: Failed (click for details)FreeBSD 12 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10096/artifact/FBSD12AMD64/config.status/config.statusMake failed for FreeBSD 12 amd64 build:
OpenBSD 6 amd64 build: Failed (click for details)Make failed for OpenBSD 6 amd64 build:
OpenBSD 6 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10096/artifact/CI011BUILD/config.status/config.status NetBSD 8 amd64 build: Failed (click for details)Make failed for NetBSD 8 amd64 build:
NetBSD 8 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10096/artifact/CI012BUILD/config.status/config.status FreeBSD 11 amd64 build: Failed (click for details)Make failed for FreeBSD 11 amd64 build:
FreeBSD 11 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10096/artifact/CI009BUILD/config.status/config.status Ubuntu 18.04 amd64 build: Failed (click for details)Package building failed for Ubuntu 18.04 amd64 build:
Ubuntu 18.04 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10096/artifact/U1804AMD64/config.status/config.status Successful on other platforms
Warnings Generated during build:Checkout code: Successful with additional warningsUbuntu 18.04 ppc64le build: Failed (click for details)Ubuntu 18.04 ppc64le build: config.log output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10096/artifact/U1804PPC64LEBUILD/config.log/ Ubuntu 18.04 ppc64le build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10096/artifact/U1804PPC64LEBUILD/config.status/config.statusPackage building failed for Ubuntu 18.04 ppc64le build:
Ubuntu 16.04 i386 build: Failed (click for details)Make failed for Ubuntu 16.04 i386 build:
Ubuntu 16.04 i386 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10096/artifact/U1604I386/config.status/config.status Fedora 29 amd64 build: Failed (click for details)Fedora 29 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10096/artifact/F29BUILD/config.status/config.statusPackage building failed for Fedora 29 amd64 build NetBSD 6 amd64 build: Failed (click for details)Make failed for NetBSD 6 amd64 build:
NetBSD 6 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10096/artifact/CI007BUILD/config.status/config.status CentOS 7 amd64 build: Failed (click for details)Package building failed for CentOS 7 amd64 build Debian 10 amd64 build: Failed (click for details)Debian 10 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10096/artifact/DEB10BUILD/config.status/config.statusPackage building failed for Debian 10 amd64 build:
FreeBSD 12 amd64 build: Failed (click for details)FreeBSD 12 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10096/artifact/FBSD12AMD64/config.status/config.statusMake failed for FreeBSD 12 amd64 build:
OpenBSD 6 amd64 build: Failed (click for details)Make failed for OpenBSD 6 amd64 build:
OpenBSD 6 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10096/artifact/CI011BUILD/config.status/config.status NetBSD 8 amd64 build: Failed (click for details)Make failed for NetBSD 8 amd64 build:
NetBSD 8 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10096/artifact/CI012BUILD/config.status/config.status FreeBSD 11 amd64 build: Failed (click for details)Make failed for FreeBSD 11 amd64 build:
FreeBSD 11 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10096/artifact/CI009BUILD/config.status/config.status Ubuntu 18.04 amd64 build: Failed (click for details)Package building failed for Ubuntu 18.04 amd64 build:
Ubuntu 18.04 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10096/artifact/U1804AMD64/config.status/config.status
|
💚 Outdated resultsBasic BGPD CI results: SUCCESS, 0 tests failed
For details, please contact louberger |
Continuous Integration Result: FAILEDContinuous Integration Result: FAILEDSee below for issues. This is a comment from an automated CI system. Get source / Pull Request: SuccessfulBuilding Stage: FailedDebian 10 amd64 build: Failed (click for details)Debian 10 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10100/artifact/DEB10BUILD/config.status/config.statusPackage building failed for Debian 10 amd64 build:
Fedora 29 amd64 build: Failed (click for details)Fedora 29 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10100/artifact/F29BUILD/config.status/config.statusPackage building failed for Fedora 29 amd64 build Ubuntu 18.04 amd64 build: Failed (click for details)Package building failed for Ubuntu 18.04 amd64 build:
Ubuntu 18.04 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10100/artifact/U1804AMD64/config.status/config.status FreeBSD 12 amd64 build: Failed (click for details)FreeBSD 12 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10100/artifact/FBSD12AMD64/config.status/config.statusMake failed for FreeBSD 12 amd64 build:
Ubuntu 16.04 i386 build: Failed (click for details)Make failed for Ubuntu 16.04 i386 build:
Ubuntu 16.04 i386 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10100/artifact/U1604I386/config.status/config.status FreeBSD 11 amd64 build: Failed (click for details)Make failed for FreeBSD 11 amd64 build:
FreeBSD 11 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10100/artifact/CI009BUILD/config.status/config.status Ubuntu 18.04 ppc64le build: Failed (click for details)Ubuntu 18.04 ppc64le build: config.log output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10100/artifact/U1804PPC64LEBUILD/config.log/ Ubuntu 18.04 ppc64le build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10100/artifact/U1804PPC64LEBUILD/config.status/config.statusPackage building failed for Ubuntu 18.04 ppc64le build:
OpenBSD 6 amd64 build: Failed (click for details)Make failed for OpenBSD 6 amd64 build:
OpenBSD 6 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10100/artifact/CI011BUILD/config.status/config.status NetBSD 8 amd64 build: Failed (click for details)Make failed for NetBSD 8 amd64 build:
NetBSD 8 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10100/artifact/CI012BUILD/config.status/config.status NetBSD 6 amd64 build: Failed (click for details)Make failed for NetBSD 6 amd64 build:
NetBSD 6 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10100/artifact/CI007BUILD/config.status/config.status CentOS 7 amd64 build: Failed (click for details)Package building failed for CentOS 7 amd64 build Successful on other platforms
Warnings Generated during build:Checkout code: Successful with additional warningsDebian 10 amd64 build: Failed (click for details)Debian 10 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10100/artifact/DEB10BUILD/config.status/config.statusPackage building failed for Debian 10 amd64 build:
Fedora 29 amd64 build: Failed (click for details)Fedora 29 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10100/artifact/F29BUILD/config.status/config.statusPackage building failed for Fedora 29 amd64 build Ubuntu 18.04 amd64 build: Failed (click for details)Package building failed for Ubuntu 18.04 amd64 build:
Ubuntu 18.04 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10100/artifact/U1804AMD64/config.status/config.status FreeBSD 12 amd64 build: Failed (click for details)FreeBSD 12 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10100/artifact/FBSD12AMD64/config.status/config.statusMake failed for FreeBSD 12 amd64 build:
Ubuntu 16.04 i386 build: Failed (click for details)Make failed for Ubuntu 16.04 i386 build:
Ubuntu 16.04 i386 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10100/artifact/U1604I386/config.status/config.status FreeBSD 11 amd64 build: Failed (click for details)Make failed for FreeBSD 11 amd64 build:
FreeBSD 11 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10100/artifact/CI009BUILD/config.status/config.status Ubuntu 18.04 ppc64le build: Failed (click for details)Ubuntu 18.04 ppc64le build: config.log output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10100/artifact/U1804PPC64LEBUILD/config.log/ Ubuntu 18.04 ppc64le build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10100/artifact/U1804PPC64LEBUILD/config.status/config.statusPackage building failed for Ubuntu 18.04 ppc64le build:
OpenBSD 6 amd64 build: Failed (click for details)Make failed for OpenBSD 6 amd64 build:
OpenBSD 6 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10100/artifact/CI011BUILD/config.status/config.status NetBSD 8 amd64 build: Failed (click for details)Make failed for NetBSD 8 amd64 build:
NetBSD 8 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10100/artifact/CI012BUILD/config.status/config.status NetBSD 6 amd64 build: Failed (click for details)Make failed for NetBSD 6 amd64 build:
NetBSD 6 amd64 build: config.status output from configure script can be found at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-10100/artifact/CI007BUILD/config.status/config.status CentOS 7 amd64 build: Failed (click for details)Package building failed for CentOS 7 amd64 build
|
💚 Outdated resultsBasic BGPD CI results: SUCCESS, 0 tests failed
For details, please contact louberger |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi Rafael,
Thanks for posting this PR. It's a good idea to integrate FPM calls with dplane module.
I have a question regarding reusing encoding functions for kernel netlink messages for FPM netlink message.
In the EVPN case, there are extra custom attributes to the FPM netlink message which are not used in the kernel netlink message.
Function: netlink_route_info_add_nh
if (re && CHECK_FLAG(re->flags, ZEBRA_FLAG_EVPN_ROUTE)) {
nhi.encap_info.encap_type = FPM_NH_ENCAP_VXLAN;
zl3vni = zl3vni_from_vrf(nexthop->vrf_id);
if (zl3vni && is_l3vni_oper_up(zl3vni)) {
/* Add VNI to VxLAN encap info */
nhi.encap_info.vxlan_encap.vni = zl3vni->vni;
}
}
Function: netlink_route_info_encode
encap = nhi->encap_info.encap_type;
if (encap > FPM_NH_ENCAP_NONE) {
addattr_l(&req->n, in_buf_len, RTA_ENCAP_TYPE, &encap,
sizeof(uint16_t));
switch (encap) {
case FPM_NH_ENCAP_NONE:
break;
case FPM_NH_ENCAP_VXLAN:
vxlan = &nhi->encap_info.vxlan_encap;
nest = addattr_nest(&req->n, in_buf_len,
RTA_ENCAP);
addattr32(&req->n, in_buf_len, VXLAN_VNI,
vxlan->vni);
addattr_nest_end(&req->n, nest);
break;
case FPM_NH_ENCAP_MAX:
break;
}
}
How do we add these attributes with this change?
@adharkar Good catch, I think the more important question here is: why does the FPM interface have this extra attribute while the normal kernel communication doesn't? The simplest thing to do here is to implement this difference for the zebra kernel netlink function, unless there is something strange here. |
Generalize the netlink route message building function so it can be used in the future by the netlink Forwarding Plane Manager (FPM) interface. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
* Use `inet_ntop` instead of `inet_ntoa` * Replace function name with `__func__` * Inline functions Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
* Remove variable `family`; * Remove always false `RTM_DELROUTE` checks; Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
Instead of retuning always `0`, lets return the amount of used bytes for the message. This will be used by the new FPM interface to know how many bytes we must reserve for the output buffer. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
Add a public reset api, so a context can be reset and reused; add apis to init a context for a route or mac update. Signed-off-by: Mark Stapp <mjs@voltanet.io>
Initial import of the new zebra data plane plugin for FPM netlink. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
Implement the code that walks the RIB to send routes that are already inside the RIB. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
Those functions are going to be used by the new data plane plugin for netlink FPM. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
Implement the code that walks the RMAC to send routes that are already inside installed in the OS. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
Add commands to enable/disable and configure FPM. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
Add commands to show and reset FPM counters. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
Remove code duplication and document hardcoded values. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
Add counters to debug the output buffer usage and pull down its data when the remote receiver is slow (so we get more space for writes). Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
Enqueue all contexts inside FPM to avoid losing updates and to move all processing to the FPM thread. This helps in situations with huge amount of routes (e.g. BGP peer flapping with a million routes). Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
FPM has a thread to encode and enqueue output buffer that might compete with zebra RIB/RMAC walk on startup, so lets use atomic operations to make sure we are not getting statistic/counters wrong. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
Changes: * Let the package builder scripts know that we have a new module that needs to be taken care of. * Include the frr atomic header to avoid undeclared atomic operations. * Disable build on *BSDs because the code is using some zebra netlink functions only available for Linux. * Move data plane FPM module outside old FPM automake definition. * Fix atomic usage for Ubuntu 14.04 (always use explicit). Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
* Use 32bit atomic instead of 64bit. * Don't use semicolon at the end of macros. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
Store VNI information in the data plane context so we can use it to build the FPM netlink update with that information later. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
* Break lines longer than 80 columns. * Remove space after '('. * Use '%pIX' instead of 'inet_ntop'. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
To enable them just configure `debug zebra fpm`. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
Implement the fix made in `master` to the remain pieces of code in the data plane FPM module. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
062eaca
to
9d5c326
Compare
Rebased again to fix recent conflicts. |
Continuous Integration Result: SUCCESSFULCongratulations, this patch passed basic tests Tested-by: NetDEF / OpenSourceRouting.org CI System CI System Testrun URL: https://ci1.netdef.org/browse/FRR-FRRPULLREQ-11852/ This is a comment from an automated CI system. Warnings Generated during build:Debian 10 amd64 build: Successful with additional warningsDebian Package lintian failed for Debian 10 amd64 build:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for pushing this through - looks good now
💚 Basic BGPD CI results: SUCCESS, 0 tests failedResults table
For details, please contact louberger |
Merging this now - if issues come up with further use, we can address those with fix PRs |
Summary
In order to reduce code duplication and simplify the FPM code, I've implemented a new FPM netlink interface using the new zebra data plane framework: it reuses all zebra netlink code for encoding messages and does not access zebra data structures directly (no more pointers to data we don't have).
This is a work-in-progress and I'll update it as I finish the todo list below and receive feedback.
I would like to receive reports from FPM netlink users that use RMAC since I don't have the appropriate tools to test it.
Related
Fixes #5369 .
TODO