Dan Kenigsberg has posted comments on this change.
Change subject: vm: remove unused reqsize and needExtend
......................................................................
Patch Set 6: Looks good to me, approved
(1 inline comment)
....................................................
File vdsm/vm.py
Line 778: truesize = int(res['truesize'])
Line 779:
Line 780: # This is for backward compatibility if the volume
extension
Line 781: # request came in before a vdsm update and the newSize
value
Line 782: # was stored in the drive property rather than in the
task.
task->attribute
Line 783: newSize = drive.get('newSize', d.reqsize)
Line 784:
Line 785: self.log.debug('_afterLvExtend apparentsize %s req size
%s',
Line 786: apparentsize / constants.MEGAB, newSize)
# MiB
--
To view, visit http://gerrit.ovirt.org/8381
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I2387963eebb1b97be0d32396f14817ab7219732f
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Ayal Baron <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Daniel Paikov <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Igor Lvovsky <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
