Change in vdsm[master]: virt: typo: unlink is in os not in os.path

vfeenstr at redhat.com vfeenstr at redhat.com
Tue Oct 7 08:23:23 UTC 2014


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 <vfeenstr at redhat.com>
---
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 <vfeenstr at redhat.com>


More information about the vdsm-patches mailing list