Bug 800424

Summary: [webadmin] User's selection of vm disk allocation type during import vm is not passed to CopyImageVDSCommand
Product: [Retired] oVirt Reporter: Rami Vaknin <rvaknin>
Component: ovirt-engine-webadminAssignee: Gilad Chaplik <gchaplik>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: medium Docs Contact:
Priority: unspecified    
Version: unspecifiedCC: acathrow, dfediuck, ecohen, iheim, mgoldboi, yeylon
Target Milestone: ---   
Target Release: 3.2   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: storage
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-02-15 06:47:54 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Storage RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On:    
Bug Blocks: 840628    
Attachments:
Description Flags
engine.log none

Description Rami Vaknin 2012-03-06 13:32:07 UTC
Created attachment 567953 [details]
engine.log

Version:
Devel env, last commit's hash: 17debc7caca685e4e9055d32e896ee3b349974eb

There is an option to choose the allocation type of imported vm's disk by choosing "Collapse All Snapshots" in the import vm dialog and the destination allocation type of each disk.
The destination disk allocation type remains as it was before import, without relation to the user's selection in webadmin, it looks like the user selection allocation parameter was not passed to the backend's CopyImageVDSCommand.

From engine.log - vm's disk that I selected to be pre-allocatted:

2012-03-06 14:21:29,082 INFO  [org.ovirt.engine.core.vdsbroker.irsbroker.CopyImageVDSCommand] (pool-5-thread-400) [7a6307aa] START, CopyImageVDSCommand(storagePoolId = bebb2832-9abb-4916-a845-faebbb380437, ignoreFailoverLimit = false, compatabilityVersion = 3.0, storageDomainId = dc63baf6-63f1-42f3-a934-4f79ed0d7e4f, imageGroupId = d9fe281b-755c-4f02-9da5-dc3c0ff9b828, imageId = 828c239d-b56d-48dd-a47c-4bad534fc559, dstImageGroupId = d9fe281b-755c-4f02-9da5-dc3c0ff9b828, vmId = d5b30893-014f-4f58-ad69-bc217f699289, dstImageId = 828c239d-b56d-48dd-a47c-4bad534fc559, imageDescription = Active VM, dstStorageDomainId = 3ded2058-2a4c-491f-9c7f-e5c79f7f1ec6, copyVolumeType = LeafVol, volumeFormat = COW, preallocate = Sparse, postZero = false, force = true), log id: 41ce5771

Comment 1 Gilad Chaplik 2012-08-19 09:25:38 UTC
pushed upstream (http://gerrit.ovirt.org/#/c/7313/)

Comment 3 Itamar Heim 2013-01-16 16:10:30 UTC
3.2 beta built, moving to ON_QA status to allow testing