test-stress(disk): box rootfs is a bounded, isolated disk that survives a fill#623
Draft
G4614 wants to merge 3 commits into
Draft
test-stress(disk): box rootfs is a bounded, isolated disk that survives a fill#623G4614 wants to merge 3 commits into
G4614 wants to merge 3 commits into
Conversation
A box must not see or exhaust the host filesystem. This integration test starts an alpine box and checks that its `/` is its own small ext4 (a few hundred MB, sized from the image — not the host's tens of millions of 1K-blocks), then fills it with dd and asserts the write hits ENOSPC rather than wandering onto the host disk, and that the VM stays Running and serving exec afterward. Covers the box-internal disk quadrant (the per-box blast radius), complementing the host-disk admission guard. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
G4614
changed the title
…lf-bounded The existing test proved one box's rootfs is bounded and survives a fill, but not that boxes don't share a disk pool. Add a two-box test: fill the victim's rootfs to ENOSPC and assert the bystander keeps its free space, still accepts writes, and both VMs stay alive — the per-box disk boundary the title claims. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Spawn 6 dd writers racing to fill the same rootfs and assert every one sees "No space left on device" (no hangs, no silent partial success, RC!=0 for all), and the guest agent still accepts exec afterward. Closes the gap that the single-writer fill test leaves open: a regression where one ENOSPC could wedge the rootfs for the others (stuck journal commit, EXT4 lock pile-up, agent dying on the I/O storm) would have passed the single-writer test silently. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
add tests to confirm the stability of box volume(guest)
Test plan
make test:integration:rust FILTER=box_rootfs_is_bounded_isolated_and_survives_fill→ passes./is its own small ext4, fills it to ENOSPC, and confirms the VM stays up./total (1K-blocks)/withddRunning, freshexecstill works