Merge pull request #15 from vultr/shutdown_fix

manually shutdown instead of halt api

vultr/step_shutdown.go

@@ -19,28 +19,29 @@ type stepShutdown struct {
 }
 
 func (s *stepShutdown) Run(ctx context.Context, state multistep.StateBag) multistep.StepAction {
-	c := state.Get("config").(*Config)
 	ui := state.Get("ui").(packer.Ui)
-
 	ui.Say("Performing graceful shutdown...")
 	time.Sleep(ShutdownDelaySec * time.Second)
-	id := state.Get("server_id").(string)
-	ui.Say("Performing graceful shutdown...")
 
-	err := s.client.Server.Halt(context.Background(), id)
+	comm := state.Get("communicator").(packer.Communicator)
 
-	if err != nil {
-		state.Put("error", err)
-		ui.Error(err.Error())
-		return multistep.ActionHalt
+	cmd := &packer.RemoteCmd{
+		Command: "shutdown -P now",
 	}
 
-	err = waitForServerState("active", "stopped", id, s.client, c.stateTimeout)
-	if err != nil {
+	if err := comm.Start(ctx, cmd); err != nil {
 		state.Put("error", err)
 		ui.Error(err.Error())
 		return multistep.ActionHalt
 	}
+	cmd.Wait()
+
+	if cmd.ExitStatus() == packer.CmdDisconnect {
+		ui.Say("Server successfully shutdown")
+		time.Sleep(ShutdownDelaySec * time.Second)
+	} else {
+		ui.Say("Sleeping to ensure that server is shut down...")
+	}
 
 	return multistep.ActionContinue
 }