From: Claudio Jeker <cjeker@diehard.n-r-g.com>
Subject: Re: bgpd.conf: document `announce EVPN`
To: Denis Fondras <denis@openbsd.org>
Cc: tech@openbsd.org
Date: Mon, 24 Feb 2025 13:56:11 +0100

On Sun, Feb 23, 2025 at 02:38:30PM +0100, Denis Fondras wrote:
> On Sun, Feb 23, 2025 at 07:28:48AM +0100, Claudio Jeker wrote:
> > On Sat, Feb 22, 2025 at 01:15:05PM +0100, Denis Fondras wrote:
> > > Just mention the existence of `announce EVPN` option.
> > > 
> > > Denis
> > > 
> > > Index: bgpd.conf.5
> > > ===================================================================
> > > RCS file: /cvs/src/usr.sbin/bgpd/bgpd.conf.5,v
> > > diff -u -p -r1.249 bgpd.conf.5
> > > --- bgpd.conf.5	27 Jan 2025 15:22:11 -0000	1.249
> > > +++ bgpd.conf.5	22 Feb 2025 12:12:50 -0000
> > > @@ -979,7 +979,7 @@ The neighbor properties are as follows:
> > >  .Bl -tag -width Ds -compact
> > >  .It Xo
> > >  .Ic announce
> > > -.Pq Ic IPv4 Ns | Ns Ic IPv6
> > > +.Pq Ic IPv4 Ns | Ns Ic IPv6 | Ns Ic EVPN
> > >  .Pq Ic none Ns | Ns Ic unicast Ns | Ns Ic vpn Ns | Ns Ic flowspec
> > >  .Op Ic enforce
> > >  .Xc
> > 
> > Would it make sense to add a second line here:
> > 
> > .It Xo
> > .Ic announce
> > .Ic EVPN
> > .Op Ic enforce
> > .Xc
> > 
> > To make it clear that EVPN is alone?
> > 
> 
> Sure ! Something like that ?

I was more thinking like this.
-- 
:wq Claudio

Index: bgpd.conf.5
===================================================================
RCS file: /cvs/src/usr.sbin/bgpd/bgpd.conf.5,v
diff -u -p -r1.249 bgpd.conf.5
--- bgpd.conf.5	27 Jan 2025 15:22:11 -0000	1.249
+++ bgpd.conf.5	24 Feb 2025 12:55:20 -0000
@@ -983,11 +983,18 @@ The neighbor properties are as follows:
 .Pq Ic none Ns | Ns Ic unicast Ns | Ns Ic vpn Ns | Ns Ic flowspec
 .Op Ic enforce
 .Xc
+.It Xo
+.Ic announce
+.Ic EVPN
+.Op Ic enforce
+.Xc
 For the given address family, control which
 .Em subsequent address families
 are announced during the capabilities negotiation.
 Only routes for that address family and subsequent address families will be
 announced and processed.
+.Ic EVPN
+does not require a subsequent address family.
 .Pp
 At the moment, only
> 
> Index: bgpd.conf.5
> ===================================================================
> RCS file: /cvs/src/usr.sbin/bgpd/bgpd.conf.5,v
> diff -u -p -r1.249 bgpd.conf.5
> --- bgpd.conf.5	27 Jan 2025 15:22:11 -0000	1.249
> +++ bgpd.conf.5	23 Feb 2025 13:37:55 -0000
> @@ -979,6 +979,13 @@ The neighbor properties are as follows:
>  .Bl -tag -width Ds -compact
>  .It Xo
>  .Ic announce
> +.Ic EVPN
> +.Op Ic enforce
> +.Xc
> +Enable processing of EVPN NLRI.
> +.Pp
> +.It Xo
> +.Ic announce
>  .Pq Ic IPv4 Ns | Ns Ic IPv6
>  .Pq Ic none Ns | Ns Ic unicast Ns | Ns Ic vpn Ns | Ns Ic flowspec
>  .Op Ic enforce
> 
 .Ic none ,