Change in vdsm[ovirt-3.2]: Deactivate libvirtVM.Drives()

fsimonce at redhat.com fsimonce at redhat.com
Thu May 30 09:13:55 UTC 2013


Federico Simoncelli has submitted this change and it was merged.

Change subject: Deactivate libvirtVM.Drives()
......................................................................


Deactivate libvirtVM.Drives()

Change I8d310f216987b7 has introduced teardownVolumePath() calls with a
Drive object instead of a drive dictionary from self.conf.
In its previous form, teardownVolumePath() has silently ignored such
input, keeping the volume active.

This patch makes teardownVolumePath() handle such inputs properly. This
is one baby step in the process of getting rid of self.conf: we would
like to use the objects everywhere within vdsm.

Note that this patch introduces an ugly log line when pointlessly trying
to tear down a cdrom device. This would be fixed in a subsequent patch.

Change-Id: I8ba965f0987037fa5dafc6b3ea6eeb6f01a82436
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=962549
Signed-off-by: Eduardo <ewarszaw at redhat.com>
Signed-off-by: Dan Kenigsberg <danken at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/14869
---
M vdsm/clientIF.py
M vdsm/libvirtvm.py
2 files changed, 18 insertions(+), 7 deletions(-)

Approvals:
  Federico Simoncelli: Verified
  Dan Kenigsberg: Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I8ba965f0987037fa5dafc6b3ea6eeb6f01a82436
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.2
Gerrit-Owner: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Eduardo <ewarszaw at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Yeela Kaplan <ykaplan at redhat.com>


More information about the vdsm-patches mailing list