launchboxio / cluster-api-provider-proxmox

CAPI Provider for Proxmox
MIT License
3 stars 1 forks source link

Volume cleanup #10

Closed robwittman closed 9 months ago

robwittman commented 9 months ago

When VMs are deleted, TrueNAS (used for ZFS over ISCSI) fails to remove the created vm-XXX-cloudinit storage volume, due to dataset is busy. We had added a time.Sleep(time.Second *10) to try and wait it out, but they are still being orphaned. This then causes issues when launching new VMs that re-use an ID.

We should ensure that when ProxmoxMachines are cleaned up, the cloud-init volumes are also removed from the given storage