oxidecomputer / oxidecomputer/crucible
IO can get stuck if a downstairs goes offline during LiveRepair
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 260
- Forks
- 34
- Avg merge
- 2d 1h
- Merged PRs (30d)
- 8
Description
We no longer have a time based dismissal of a downstairs. We now will only dismiss a downstairs if enough IO has accrued that we have decided we don't want to hold any more in memory, and so replay is no longer an option.
This process runs into trouble if we are in the middle of a LiveRepair. Live Repair IOs require all three downstairs to go through the IO at the same time, if a single downstairs is present when the LR starts, but goes missing, we will wait forever for it to return so we can complete the repair IO.
This is tangentially related somewhat to https://github.com/oxidecomputer/crucible/issues/1194, where we should not even try to replay IOs to downstairs if a LiveRepair has happened.
It's easy enough to reproduce this hang.
Start IO to all downstairs.
Take downstairs 0 offline, wait for enough IO that it goes to faulted.
Bring downstairs 0 back, let LR start.
Once LR is started, stop downstairs 1 or 2.
IOs will now stop, as we are now going to wait for an IO from the offline downstairs.
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 LiveRepair flow alongside downstairs faulting, dismissal, replay, and IO coordination; the issue does not name specific files or tests. Reproduce the sequence described here, then verify that stopping a downstairs after LiveRepair starts no longer leaves IO waiting forever.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- distributed-systems
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100