smartxworks / virtink

Lightweight Virtualization Add-on for Kubernetes
Apache License 2.0
492 stars 41 forks source link

DataVolume GC cause VM deployment failure #62

Closed lobshunter closed 1 year ago

lobshunter commented 1 year ago

Recent version of CDI will garbage collect completed DataVolume. For VM uses DataVolume, virtink waits its DataVolume import to be completed before starting the VM. But with default CDI deployment, DataVolume will be GCed once completed. Hence virtink fails deploy VM with build VM Pod: DataVolume.cdi.kubevirt.io "cdi-volume" not found error.

Perhaps it's worth mentioning this in CDI VM example. Tuning .spec.config.dataVolumeTTLSeconds of cdi.kubevirt.io/v1beta1 works.(or adapting virtink to CDI's GC action?)

scuzhanglei commented 1 year ago

Thanks for your report, we have notice this issue and https://github.com/smartxworks/virtink/pull/61 will fix it.