Federico Simoncelli has posted comments on this change.

Change subject: image: shrink the new volume When merging block cow volumes
......................................................................


Patch Set 4: I would prefer that you didn't submit this

(1 inline comment)

We'll need a new qemu-kvm requirement (blocked on bz888008). You could add a 
comment/placeholder in the spec file so that we(/I) won't forget that. 
Temporarily marking as -1.

....................................................
Commit Message
Line 10: sizes of the original volumes that were merged. With qcow
Line 11: volumes on block devices we don't use the whole size of the new volume,
Line 12: so we want to allocate only the actual size needed by the new volume
Line 13: with some additions to match the vdsm conventions.
Line 14: 
Bug-Url?
Line 15: Change-Id: I8ace4c48d278cb84ce871bc402643131265c3198


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8ace4c48d278cb84ce871bc402643131265c3198
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yeela Kaplan <[email protected]>
Gerrit-Reviewer: Ayal Baron <[email protected]>
Gerrit-Reviewer: Eduardo <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Shu Ming <[email protected]>
Gerrit-Reviewer: Yeela Kaplan <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to