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 <fsimonce(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Daniel Paikov <paikov(a)gmail.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky(a)redhat.com>