oxidecomputer / oxidecomputer/omicron

Convert remaining UUIDs in Omicron to `TypedUuid`

Open
#5,088 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Rust
Stars
572
Forks
97
Avg merge
2d 12h
Merged PRs (30d)
96

Description

In #4934 we landed initial support for converting Omicron's UUIDs to TypedUuid, along with a couple of simpler cases. This issue tracks covering all our remaining UUID instances to TypedUuid.

This will need to be a collaborative effort, so any help would be fantastic!

  • Rack ID
  • Sled ID
  • Silo ID
  • Dataset ID
  • Blueprint ID
  • Collection ID
  • ... (please edit this issue and add more kinds of IDs above this line)

Contributor guide

Open the contributing guide

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 reviewing the initial TypedUuid conversion in #4934 and locating Omicron's remaining UUID instances for Rack, Sled, Silo, Dataset, Blueprint, and Collection IDs. Done means the remaining UUID kinds have been converted to TypedUuid and the checklist reflects any additional kinds discovered.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
api, backend
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.