Federico Simoncelli has posted comments on this change.

Change subject: sparsify: integrating virt-sparsify into vdsm
......................................................................


Patch Set 10:

(1 comment)

http://gerrit.ovirt.org/#/c/28328/10/vdsm/storage/image.py
File vdsm/storage/image.py:

Line 573:         finally:
Line 574:             srcVolume.teardown(sdUUID=srcSdUUID, volUUID=srcVolUUID)
Line 575:             dstVolume.teardown(sdUUID=dstSdUUID, volUUID=dstVolUUID)
Line 576: 
Line 577:         # Shrink to optimal size
Add also srcVolume.shrinkToOptimalSize()
(To save some space until engine cleans up the volume)
Line 578:         dstVolume.shrinkToOptimalSize()
Line 579: 
Line 580:     def cloneStructure(self, sdUUID, imgUUID, dstSdUUID):
Line 581:         self._createTargetImage(sdCache.produce(dstSdUUID), sdUUID, 
imgUUID)


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Id7bd2b4b6d45781fa27a128dd68d14b7561d0901
Gerrit-PatchSet: 10
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Utkarsh Singh <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: Utkarsh Singh <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to