Skip to content

Commit

Permalink
Merge pull request #354 from breitenj/fix-ha-vm-deletion
Browse files Browse the repository at this point in the history
Fix HA VM deletion
  • Loading branch information
ggongaware authored May 18, 2021
2 parents 01f54a7 + abefe2a commit 0d6e7d7
Showing 1 changed file with 14 additions and 2 deletions.
16 changes: 14 additions & 2 deletions proxmox/resource_vm_qemu.go
Original file line number Diff line number Diff line change
Expand Up @@ -1375,8 +1375,20 @@ func resourceVmQemuDelete(d *schema.ResourceData, meta interface{}) error {
if err != nil {
return err
}
// give sometime to proxmox to catchup
time.Sleep(2 * time.Second)

// Wait until vm is stopped. Otherwise, deletion will fail.
waited := 0
for waited < 300 {
vmState, err := client.GetVmState(vmr)
if err == nil && vmState["status"] == "stopped" {
break
} else if err != nil {
return err
}

time.Sleep(1 * time.Second)
}

_, err = client.DeleteVm(vmr)
return err
}
Expand Down

0 comments on commit 0d6e7d7

Please sign in to comment.