test(forge): add the forge-test VM runner (gate step for #678) #760

Merged

hexajon merged 5 commits from feat/forge-test-runner into main

2026-10-14 16:31:25 +00:00

hexajon commented

2026-10-14 16:31:23 +00:00

Owner

Summary

Adds scripts/forge-vm/run-forge-tests.sh, the gate step that ADR 0052 and #678 call for. It resets the forge-test VM to its base snapshot, pushes the committed tree into the VM with git archive of HEAD, and runs the forge-tagged go test as the non-root test user over incus exec. Pushing rather than mounting means the VM holds no live handle on the host tree, there is no disk-device mount to linger or go resource-busy, and the run is always the frozen committed state. Also hardens provision.sh to remove stray devices and assert the device set (the old abandoned cache mount was breaking a clean start), and documents the runner in the forge-vm README. The run passes green against real throwaway Forgejo and Gitea servers. Refs #678. The remaining #678 work, wiring this into CI and migrating the fake-server tests, stays open.

## Summary Adds scripts/forge-vm/run-forge-tests.sh, the gate step that ADR 0052 and #678 call for. It resets the forge-test VM to its base snapshot, pushes the committed tree into the VM with git archive of HEAD, and runs the forge-tagged go test as the non-root test user over incus exec. Pushing rather than mounting means the VM holds no live handle on the host tree, there is no disk-device mount to linger or go resource-busy, and the run is always the frozen committed state. Also hardens provision.sh to remove stray devices and assert the device set (the old abandoned cache mount was breaking a clean start), and documents the runner in the forge-vm README. The run passes green against real throwaway Forgejo and Gitea servers. Refs #678. The remaining #678 work, wiring this into CI and migrating the fake-server tests, stays open.

hexajon added 5 commits

2026-10-14 16:31:23 +00:00

test(forge): add the forge-test VM runner (gate step for #678 ) c5abb5907a

fix(forge): surface incus start and stop errors, stop swallowing them 7de976a031

fix(forge): provision removes stray VM devices and asserts the device set c18b0e6d5a

fix(forge): reset via snapshot revert, drop the manual stop dance d87ec34dc8

refactor(forge): push the committed tree into the VM instead of mounting it

drift / surface-diff (pull_request) Successful in 41s

Details

drift / forgejo-smoke (pull_request) Has been skipped

Details

ci / mcp-conformance (pull_request) Successful in 1m27s