Francesco Romani 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(a)redhat.com>
Reviewed-on:
https://gerrit.ovirt.org/46417
Reviewed-by: Nir Soffer <nsoffer(a)redhat.com>
Tested-by: Nir Soffer <nsoffer(a)redhat.com>
Reviewed-by: Adam Litke <alitke(a)redhat.com>
Continuous-Integration: Jenkins CI
Reviewed-on:
https://gerrit.ovirt.org/47419
Reviewed-by: Allon Mureinik <amureini(a)redhat.com>
Reviewed-by: Francesco Romani <fromani(a)redhat.com>
---
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:
Nir Soffer: Looks good to me, but someone else must approve
Jenkins CI: Passed CI tests
Allon Mureinik: Looks good to me, but someone else must approve
Francesco Romani: Looks good to me, approved
Freddy Rolland: Verified
--
To view, visit
https://gerrit.ovirt.org/47419
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iaf5f142541bf0b311a77a0544a1c4ffb689a9fde
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.6
Gerrit-Owner: Freddy Rolland <frolland(a)redhat.com>
Gerrit-Reviewer: Adam Litke <alitke(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org