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

danken at redhat.com danken at redhat.com
Wed Oct 8 08:07:28 UTC 2014


Dan Kenigsberg has submitted this change and it was merged.

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>
Reviewed-on: http://gerrit.ovirt.org/33881
Reviewed-by: Francesco Romani <fromani at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M vdsm/virt/vm.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Vinzenz Feenstra: Verified
  Dan Kenigsberg: Looks good to me, approved
  Francesco Romani: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Id588d272e142fc25e6cd0cd2ab5b7385caf45815
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list