On Thu, Nov 22, 2018 at 04:52:08PM +0100, Claudio Jeker wrote:
> Since a while announcements added by bgpctl will overwrite the ones from
> the config and no longer live next to each other. Because of this step 13
> in the decision process is no longer needed.
>
> OK?
Ping
--
:wq Claudio
Index: bgpd.8
===================================================================
RCS file: /cvs/src/usr.sbin/bgpd/bgpd.8,v
retrieving revision 1.59
diff -u -p -r1.59 bgpd.8
--- bgpd.8 4 Nov 2018 14:34:00 -0000 1.59
+++ bgpd.8 22 Nov 2018 14:50:47 -0000
@@ -112,10 +112,6 @@ If it is not present then a length of 0
.It
The path coming from the peer with the lowest IP address is selected.
IPv4 sessions will be preferred over IPv6 ones.
-.It
-In case of locally announced prefixes
-.Nm
-will prefer statically set prefixes over dynamically inserted ones.
.El
.Pp
Attributes set by filters can be used to tip the decision process to prefer
Index: rde_decide.c
===================================================================
RCS file: /cvs/src/usr.sbin/bgpd/rde_decide.c,v
retrieving revision 1.72
diff -u -p -r1.72 rde_decide.c
--- rde_decide.c 27 Sep 2018 15:53:14 -0000 1.72
+++ rde_decide.c 22 Nov 2018 14:46:42 -0000
@@ -229,16 +229,7 @@ prefix_cmp(struct prefix *p1, struct pre
return (-memcmp(&peer1->remote_addr, &peer2->remote_addr,
sizeof(peer1->remote_addr)));
- /* 13. for announced prefixes prefer dynamic routes */
- if ((asp1->flags & F_ANN_DYNAMIC) != (asp2->flags & F_ANN_DYNAMIC)) {
- if (asp1->flags & F_ANN_DYNAMIC)
- return (1);
- else
- return (-1);
- }
-
fatalx("Uh, oh a politician in the decision process");
- /* NOTREACHED */
}
/*
