Vinzenz Feenstra has uploaded a new change for review.

Change subject: virt: typo: unlink is in os not in os.path
......................................................................

virt: typo: unlink is in os not in os.path

When this code path was reached it caused an exception and left the VM
during recovery in 'PAUSED' state and all communication with the guest
was no longer available.

Change-Id: Id588d272e142fc25e6cd0cd2ab5b7385caf45815
Bug-Url: https://bugzilla.redhat.com/1136982
Signed-off-by: Vinzenz Feenstra <[email protected]>
---
M vdsm/virt/vm.py
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/81/33881/1

diff --git a/vdsm/virt/vm.py b/vdsm/virt/vm.py
index 1eebb69..ff078ae 100644
--- a/vdsm/virt/vm.py
+++ b/vdsm/virt/vm.py
@@ -2771,7 +2771,7 @@
                 # to an unexpected stop of VDSM therefore We're going to clean
                 # it up now
                 if os.path.islink(uuidPath):
-                    os.path.unlink(uuidPath)
+                    os.unlink(uuidPath)
 
                 # We don't want an exception to be thrown when the path already
                 # exists


-- 
To view, visit http://gerrit.ovirt.org/33881
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id588d272e142fc25e6cd0cd2ab5b7385caf45815
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Vinzenz Feenstra <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to