Skip to content

virtualbox: test VBoxVMI disconnect safety - #308

Merged
Wenzel merged 2 commits into
devfrom
vbox/vboxvmi-disconnect
Oct 12, 2026
Merged

Wenzel merged 2 commits into
devfrom
vbox/vboxvmi-disconnect

Conversation

@Wenzel

@Wenzel Wenzel commented Oct 12, 2026

Copy link
Copy Markdown
Owner

Closes #287.

Adds live VBoxVMI integration coverage for:

  • normal close and SIGKILL of a client-owned pause → guest reaches Running;
  • VBoxManage pre-existing pause → disconnect preserves Paused;
  • client pause → external resume → external pause → disconnect preserves Paused;
  • second concurrent client receives vboxvmi::Error::Busy;
  • eight-iteration external resume/pause vs disconnect race preserves Paused.

The killed-client child is RAII-managed: every unwind path kills and waits, so tests cannot leak a process/socket. Driver docs, ADR and test instructions describe cleanup ownership, Busy refusal and teardown behavior.

Server: Wenzel/virtualbox#9 (merged into vmi/vboxvmi). Full VBoxVMI read/write/pause/vCPU/register regression passed; fmt/clippy/tests green locally.

@Wenzel
Wenzel merged commit ed30335 into dev Oct 12, 2026
22 checks passed
@Wenzel Wenzel mentioned this pull request Oct 12, 2026
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant