oxidecomputer / oxidecomputer/omicron
single sled config needs random allocation strategy for region repairs
Open
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 572
- Forks
- 97
- Avg merge
- 2d 12h
- Merged PRs (30d)
- 96
Description
On a single sled omicron (dev) setup, the region_replacement and region_snapshot_replacement should
use the nexus config for determining what allocation_strategy to use for replacement, instead
of defaulting to RegionAllocationStrategy::RandomWithDistinctSleds { seed: None },.
Contributor guide
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 by tracing the region_replacement and region_snapshot_replacement paths and how they currently choose RegionAllocationStrategy. Check where the nexus config exposes allocation_strategy for replacement. Done means single-sled omicron dev repairs use the configured strategy instead of defaulting to RandomWithDistinctSleds.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- backend
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100