vm: add explanation for image reset approach for VirtualBox

kuzeyardabulut · kuzeyardabulut · commit f68e03b235ad · 2025-11-03T17:20:19.000+02:00
diff --git a/vm/virtualbox/virtualbox.go b/vm/virtualbox/virtualbox.go
@@ -96,6 +96,7 @@ func (pool *Pool) Create(_ context.Context, workdir string, index int) (vmimpl.I
 	return inst, nil
 }
 
+// We avoid immutable disks and reset the image manually, as this proved more reliable when VMs fail to restart cleanly.
 func (inst *instance) clone() error {
 	if inst.debug {
 		log.Logf(0, "cloning VM %q to %q", inst.baseVM, inst.vmName)

Original file line number	Diff line number	Diff line change
`@@ -96,6 +96,7 @@ func (pool *Pool) Create(_ context.Context, workdir string, index int) (vmimpl.I`
`96`	`96`	`return inst, nil`
`97`	`97`	`}`
`98`	`98`
	`99`	`+// We avoid immutable disks and reset the image manually, as this proved more reliable when VMs fail to restart cleanly.`
`99`	`100`	`func (inst *instance) clone() error {`
`100`	`101`	`if inst.debug {`
`101`	`102`	`log.Logf(0, "cloning VM %q to %q", inst.baseVM, inst.vmName)`