test_snapshot_ab: start VM inside configure_vm

roypat · roypat · commit d43e43d71ff2 · 2025-03-04T12:57:40.000Z
No need to defer the starting.

Signed-off-by: Patrick Roy &lt;roypat@amazon.co.uk&gt;
diff --git a/tests/integration_tests/performance/test_snapshot_ab.py b/tests/integration_tests/performance/test_snapshot_ab.py
@@ -79,6 +79,7 @@ def configure_vm(self, microvm_factory, guest_kernel, rootfs, metrics) -> Microv
                 **vm.dimensions,
             }
         )
+        vm.start()
 
         return vm
 
@@ -131,7 +132,6 @@ def test_restore_latency(
     vm = test_setup.configure_vm(
         microvm_factory, guest_kernel_linux_5_10, rootfs, metrics
     )
-    vm.start()
 
     snapshot = vm.snapshot_full()
     vm.kill()

Original file line number	Diff line number	Diff line change
`@@ -79,6 +79,7 @@ def configure_vm(self, microvm_factory, guest_kernel, rootfs, metrics) -> Microv`
`79`	`79`	`**vm.dimensions,`
`80`	`80`	`}`
`81`	`81`	`)`
	`82`	`+ vm.start()`
`82`	`83`
`83`	`84`	`return vm`
`84`	`85`
`@@ -131,7 +132,6 @@ def test_restore_latency(`
`131`	`132`	`vm = test_setup.configure_vm(`
`132`	`133`	`microvm_factory, guest_kernel_linux_5_10, rootfs, metrics`
`133`	`134`	`)`
`134`		`- vm.start()`
`135`	`135`
`136`	`136`	`snapshot = vm.snapshot_full()`
`137`	`137`	`vm.kill()`