oxidecomputer / oxidecomputer/falcon
Add CLI/API feature for modifying link state.
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 20
- Forks
- 5
- Avg merge
- 11d 13h
- Merged PRs (30d)
- 2
Description
The capability to modify simnet link-state was added to illumos here
This should be added as a first-class feature to the Falcon CLI and API. This is a little bit tricky as multiple links can exist between the same two nodes. So something like
<topology> link down node1 node2
would not be sufficient. It'd need to be something a bit more verbose like
<topology> link down node1 node2 1
to indicate the index of the link between the nodes, or we could introduce something like link tags and then do something like
<topology> link down node1 node2 primary
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the linked illumos issue 14122, then inspect Falcon's existing CLI and API handling for topology links. The issue names no files or tests and leaves link indexing versus tags unresolved; done requires an agreed way to select one of multiple links and support for changing its state through both interfaces.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- api, cli
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100