oxidecomputer / oxidecomputer/oxide.rs

Export and Import columns may become hard to read in `oxide system networking switch-port-settings show` if there are many prefixes allowed

Open
#819 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Rust
Stars
75
Forks
21
Avg merge
2d 2h
Merged PRs (30d)
19

Description

I am running a4x2 with:

  • omicron 1059fe109fadbccf559e4adad134eaffcb67b85a
  • Oxide CLI version compiled from main with this commit:
$ oxide version
Oxide CLI 0.6.1+20240710.0
Built from commit: 46635f7834ee3dd4372065eae464ba23c0581bee 
Oxide API: 20240821.0

I added filtering requirements with 3 prefixes allowed for export (I could have chosen import).

$ oxide system networking bgp filter --rack 6622a0c7-093b-4407-ab34-7ce3e3ef4f22 --switch switch1 --port qsfp0 --peer 169.254.40.1 --direction export --allowed 4.5.6.0/24 --allowed 4.5.7.0/24 --allowed 4.5.8.0/24

I note that the Export (or Import) column may become hard to read if I have many prefixes allowed.

$ oxide system networking switch-port-settings show
switch0/qsfp0
=============
Autoneg  Fec   Speed
false    None  Speed100G

Address          Lot            VLAN
169.254.10.2/30  initial-infra  None
169.254.30.2/30  initial-infra  None

BGP Peer      Config   Export          Import          Communities  Connect Retry  Delay Open  Enforce First AS  Hold Time  Idle Hold Time  Keepalive  Local Pref  Md5 Auth  Min TTL  MED   Remote ASN  VLAN
169.254.10.1  as65547  [no filtering]  [no filtering]  []           3              3           false             6          0               2          None        None      None     None  None        None
169.254.30.1  as65547  [no filtering]  [no filtering]  []           3              3           false             6          0               2          None        None      None     None  None        None

Destination  Nexthop  Vlan  Preference

switch1/qsfp0
=============
Autoneg  Fec   Speed
false    None  Speed100G

Address          Lot            VLAN
169.254.20.2/30  initial-infra  None
169.254.40.2/30  initial-infra  None

BGP Peer      Config   Export                              Import          Communities  Connect Retry  Delay Open  Enforce First AS  Hold Time  Idle Hold Time  Keepalive  Local Pref  Md5 Auth  Min TTL  MED   Remote ASN  VLAN
169.254.20.1  as65547  [no filtering]                      [no filtering]  []           3              3           false             6          3               2          None        None      None     None  None        None
169.254.40.1  as65547  [4.5.6.0/24 4.5.7.0/24 4.5.8.0/24]  [no filtering]  []           3              3           false             6          3               2          None        None      None     None  None        None

Destination  Nexthop  Vlan  Preference

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reproducing the output with oxide system networking switch-port-settings show after configuring several allowed export or import prefixes. Trace the CLI's formatting for the Export and Import columns and make the resulting table remain readable with long prefix lists; verify the command output with multiple prefixes.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
cli
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.