oxidecomputer / omicron

Omicron: Oxide control plane
Mozilla Public License 2.0
240 stars 36 forks source link

Extra space in help output for `oxide system networking bgp announce-set list` #6468

Closed elaine-oxide closed 1 day ago

elaine-oxide commented 2 weeks ago

I am running a4x2 with:

diff --git a/cli/src/cmd_net.rs b/cli/src/cmd_net.rs index 0193c45..2a0510e 100644 --- a/cli/src/cmd_net.rs +++ b/cli/src/cmd_net.rs @@ -259,7 +259,7 @@ pub struct CmdBgpAnnounce { impl AuthenticatedCmd for CmdBgpAnnounce { async fn run(&self, client: &Client) -> Result<()> { let mut current: Vec = client

I ran the following:

$ oxide system networking bgp announce-set list --help
List BGP announce sets

Usage: oxide system networking bgp announce-set list [OPTIONS]

Options:
      --limit <limit>            Maximum number of items returned by a single call
      --name-or-id <name-or-id>  A name or id to use when s electing BGP port settings
      --profile <PROFILE>        Configuration profile to use for commands
      --page-token <page-token>  Token returned by previous call to retrieve the subsequent page
      --sort-by <sort-by>        [possible values: name_ascending, name_descending, id_ascending]
  -h, --help                     Print help

I note there is an extra space in "s electing".

elaine-oxide commented 1 day ago

Fixed/changed in https://github.com/oxidecomputer/omicron/pull/6498