Index | Thread | Search

From:
Kirill A. Korinsky <kirill@korins.ky>
Subject:
Re: relayctl.8: mark name|id argument as required for host/redirect/table
To:
Rafael Sadowski <rafael@sizeofvoid.org>
Cc:
tech@openbsd.org
Date:
Thu, 30 Jul 2026 12:43:40 +0200

Download raw body.

Thread
On Thu, 30 Jul 2026 10:21:02 +0200,
Rafael Sadowski <rafael@sizeofvoid.org> wrote:
> 
> name|id is required, not optional. OK?
> 

OK kirill@

> diff --git a/relayctl.8 b/relayctl.8
> index c329136..a60b507 100644
> --- a/relayctl.8
> +++ b/relayctl.8
> @@ -46,10 +46,10 @@ to communicate with
>  .Pp
>  The following commands are available:
>  .Bl -tag -width Ds
> -.It Cm host disable Op Ar name | id
> +.It Cm host disable Ar name | id
>  Disable a host.
>  Treat it as though it were always down.
> -.It Cm host enable Op Ar name | id
> +.It Cm host enable Ar name | id
>  Enable the host.
>  Start checking its health again.
>  .It Cm load Ar filename
> @@ -64,14 +64,14 @@ Continuously report any changes in the host checking engine and the
>  engine.
>  .It Cm poll
>  Schedule an immediate check of all hosts.
> -.It Cm redirect disable Op Ar name | id
> +.It Cm redirect disable Ar name | id
>  Disable a redirection.
>  If it has
>  .Xr pf 4
>  redirection rules installed, remove them.
>  Mark the redirection's main table and \(en
>  if applicable \(en disable the backup table as well.
> -.It Cm redirect enable Op Ar name | id
> +.It Cm redirect enable Ar name | id
>  Enable a redirection.
>  Mark the redirection's main table and \(en if applicable \(en enable
>  the backup table as well.
> @@ -102,14 +102,14 @@ routes.
>  Dump the complete list of running relay sessions.
>  .It Cm show summary
>  Display a list of all relays, redirections, routers, tables, and hosts.
> -.It Cm table disable Op Ar name | id
> +.It Cm table disable Ar name | id
>  Disable a table.
>  Consider all hosts disabled.
>  If it is a main table of a redirection which has a non-empty backup table,
>  swap the contents of the
>  .Xr pf 4
>  table with those of the backup table.
> -.It Cm table enable Op Ar name | id
> +.It Cm table enable Ar name | id
>  Enable a table.
>  Start doing checks for all hosts that aren't individually disabled
>  again.
> 

-- 
wbr, Kirill