search

autotest / tp-libvirt

Test Provider for Libvirt and related virtualization backends
Other
33 stars 169 forks source link

Resolve local variable 'res' referenced before assignment #5907

Closed hholoubk closed 4 weeks ago

hholoubk commented 1 month ago

Test was failing in case of coldplug + nondisk combination due missing property definiton.

For not hotplug will now start the VM in case of disk only if the disk attachment was successful. in case of other test_devices the VM should start

hholoubk commented 1 month ago

@smitterl could you please review the issue?

hholoubk commented 1 month ago

Oh .. great .. I am very new in this :D how to check if a solution already exists? so I didn't need to try to fix it

chloerh commented 1 month ago

Oh .. great .. I am very new in this :D how to check if a solution already exists? so I didn't need to try to fix it

That's my fault. I should've logged work to the issue. You could check "worklog" to see if there's solution or not. But that's still not 100%...(depends on people)

hholoubk commented 4 weeks ago

Removing the PR as the issue was resolved in other PR