On Mon, Mar 13, 2023 at 05:26:35PM +0100, Claudio Jeker wrote:
> This diff extends the show rib command to only show invalid (ineligible)
> or leaked (via OTC leak detection) routes.
> 
> While there rename the OTC_LOOP defines to OTC_LEAK since this is not a
> route loop but a route leak.

Apart from a manpage nit this is ok

> -- 
> :wq Claudio
> 
> Index: usr.sbin/bgpctl/bgpctl.8
> ===================================================================
> RCS file: /cvs/src/usr.sbin/bgpctl/bgpctl.8,v
> retrieving revision 1.105
> diff -u -p -r1.105 bgpctl.8
> --- usr.sbin/bgpctl/bgpctl.8  2 Mar 2023 17:09:52 -0000       1.105
> +++ usr.sbin/bgpctl/bgpctl.8  13 Mar 2023 16:03:01 -0000
> @@ -387,8 +387,20 @@ are defined:
>  .It Cm best
>  Alias for
>  .Ic selected .
> +.It Cm detail
> +Show more detailed output for matching routes.
>  .It Cm error
>  Show only prefixes which are marked invalid and were treated as withdrawn.
> +.It Ar family
> +Limit the output to the given address family.
> +.It Cm in
> +Show routes from the unfiltered Adj-RIB-In.
> +.It Cm invalid
> +Show only routes which are not eligible.
> +.It Cm leaked
> +Show only routes where a route leak was detected.
> +.It Cm out
> +Show the filtered routes sent to a neighbor.
>  .It Cm selected
>  Show only selected routes.
>  .It Cm ssv
> @@ -396,17 +408,9 @@ Show each RIB entry as a single line, wi
>  Only works if
>  .Cm detail
>  is specified.
> -.It Cm detail
> -Show more detailed output for matching routes.
> -.It Ar family
> -Limit the output to the given address family.
> -.It Cm in
> -Show routes from the unfiltered Adj-RIB-In.
>  The
>  .Cm neighbor
>  needs to be specified.

This should probably move up with "in"

> -.It Cm out
> -Show the filtered routes sent to a neighbor.
>  The
>  .Cm neighbor
>  needs to be specified.

and this with "out"

Reply via email to