Change in vdsm[master]: hsm: Add initialSize argument to createVolume

danken at redhat.com danken at redhat.com
Fri Oct 16 21:12:08 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: hsm: Add initialSize argument to createVolume
......................................................................


hsm: Add initialSize argument to createVolume

Add the possibility to specify the volume initial size,
for block disk with thin provisioning.

This argument is needed when importing a VM from an external
provider along with its disks. If one of the VM existing disk
is a block disk with thin provisioning, the new volume needs
a starting size as the original disk size.

If the initial size is not specified, the default initial
volume size for thin provisioning is the configured
volume_utilization_chunk_mb (1GiB by default)

Change-Id: Iaf5f142541bf0b311a77a0544a1c4ffb689a9fde
Bug-Url: https://bugzilla.redhat.com/1221603
Signed-off-by: Fred Rolland <frolland at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/46417
Reviewed-by: Nir Soffer <nsoffer at redhat.com>
Tested-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-by: Adam Litke <alitke at redhat.com>
Continuous-Integration: Jenkins CI
---
M client/vdsClient.py
M tests/Makefile.am
A tests/blockVolumeTests.py
M tests/miscTests.py
M vdsm/API.py
M vdsm/rpc/bindingxmlrpc.py
M vdsm/rpc/vdsmapi-schema.json
M vdsm/storage/blockVolume.py
M vdsm/storage/fileVolume.py
M vdsm/storage/hsm.py
M vdsm/storage/misc.py
M vdsm/storage/sd.py
M vdsm/storage/sp.py
M vdsm/storage/storageConstants.py
M vdsm/storage/volume.py
15 files changed, 195 insertions(+), 37 deletions(-)

Approvals:
  Adam Litke: Looks good to me, approved
  Nir Soffer: Verified; Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/46417
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Iaf5f142541bf0b311a77a0544a1c4ffb689a9fde
Gerrit-PatchSet: 13
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Fred Rolland <frolland at redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadgett at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list