Change in vdsm[ovirt-3.4]: blockVolume: prepare volumes for shrinkToOptimalSize

ybronhei at redhat.com ybronhei at redhat.com
Sun Jun 8 08:26:41 UTC 2014


Yaniv Bronhaim has submitted this change and it was merged.

Change subject: blockVolume: prepare volumes for shrinkToOptimalSize
......................................................................


blockVolume: prepare volumes for shrinkToOptimalSize

The qemu-img check command needs the entire volume chain in order to
be able to open the image.
Up until now shrinkToOptimalSize worked only for base volumes, in the
other cases it was failing with:

 QImgError: ecode=1, stdout=[], stderr=[
   "Could not open '(image path)': No such file or directory"]

Change-Id: Ic9fceee28932da6895829c7e7efadb735c1ec060
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1105848
Signed-off-by: Federico Simoncelli <fsimonce at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/28009
Reviewed-by: Kevin Wolf <kwolf%redhat.com at gtempaccount.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/28382
Tested-by: Daniel Erez <derez at redhat.com>
Reviewed-by: Allon Mureinik <amureini at redhat.com>
Reviewed-by: Yaniv Bronhaim <ybronhei at redhat.com>
---
M vdsm/storage/blockVolume.py
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Yaniv Bronhaim: Looks good to me, approved
  Daniel Erez: Verified
  Allon Mureinik: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic9fceee28932da6895829c7e7efadb735c1ec060
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.4
Gerrit-Owner: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Kevin Wolf <kwolf%redhat.com at gtempaccount.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list