builder/virtualbox: only power off the machine if it isrunning

0f376457 · Mitchell Hashimoto · bdd60fe9 · 0f376457
Commit 0f376457 authored Jun 23, 2013 by Mitchell Hashimoto
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 2 deletions

builder/virtualbox/step_run.go builder/virtualbox/step_run.go +5 -2

No files found.
--- a/builder/virtualbox/step_run.go
+++ b/builder/virtualbox/step_run.go
@@ -48,7 +48,10 @@ func (s *stepRun) Cleanup(state map[string]interface{}) {

 	driver := state["driver"].(Driver)
 	ui := state["ui"].(packer.Ui)
-	if err := driver.VBoxManage("controlvm", s.vmName, "poweroff"); err != nil {
-		ui.Error(fmt.Sprintf("Error shutting down VM: %s", err))
+
+	if running, _ := driver.IsRunning(s.vmName); running {
+		if err := driver.VBoxManage("controlvm", s.vmName, "poweroff"); err != nil {
+			ui.Error(fmt.Sprintf("Error shutting down VM: %s", err))
+		}
 	}
 }