Bug 767111 - [ovirt] [vdsm] extend of thin provisioned disk fails
Summary: [ovirt] [vdsm] extend of thin provisioned disk fails
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: oVirt
Classification: Retired
Component: vdsm
Version: unspecified
Hardware: x86_64
OS: Linux
unspecified
high
Target Milestone: ---
: ---
Assignee: Federico Simoncelli
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-12-13 09:50 UTC by Haim
Modified: 2014-01-13 00:50 UTC (History)
8 users (show)

Fixed In Version: v4.9.3
Clone Of:
Environment:
Last Closed: 2012-08-09 08:01:00 UTC
oVirt Team: ---
Embargoed:


Attachments (Terms of Use)
vdsm log (240.17 KB, application/x-gzip)
2011-12-13 09:56 UTC, Haim
no flags Details

Description Haim 2011-12-13 09:50:45 UTC
Description of problem:

case:

- run vm 
- perform snapshot 
- run 'dd' so disk will get bigger

at certain point, when water mark reach its designated threshold, extend message is sent to spm mailbox, but fails claiming that domain size (i.e, vg) is not bug enough, whereas, in-fact, vg free size is standing on 80g, and request size is 1024m. 

request is correct but then the command is wrong:

Thread-52962::DEBUG::2011-12-13 11:34:14,044::vm::557::vm.Vm::(_lvExtend) vmId=`2dfd58aa-d689-422b-a720-9ce865e2aaa0`::7ed87246-a042-4517-8613-9db0a8a2a3c2/50d9bc13-1666-49f0-aacb-d1b8c7785763 (vda): apparentsize 1024 req 2048

bb95a755-adc2-46ff-8461-f3688732908a::DEBUG::2011-12-13 11:34:14,570::lvm::385::Storage.Misc.excCmd::(cmd) '/usr/bin/sudo -n /sbin/lvm lvextend --config " devices { preferred_names = [\\"^/dev/mapper/\\"] ignore_suspended_devices=1 write_cache_state=0 disable_after_error_count=3 filter = [ \\"a%1UpstreamCluster-SD31320589|1UpstreamCluster112924773|1UpstreamCluster212924773|1UpstreamCluster312924773%\\",\\"r%.*%\\" ] }  global {  locking_type=1  prioritise_write_locks=1  wait_for_locks=1 }  backup {  retain_min = 50  retain_days = 0 } " --autobackup n --size 2147483648m 7ed87246-a042-4517-8613-9db0a8a2a3c2/50d9bc13-1666-49f0-aacb-d1b8c7785763' (cwd None)

Comment 1 Haim 2011-12-13 09:56:48 UTC
Created attachment 546138 [details]
vdsm log

Comment 2 Federico Simoncelli 2011-12-13 11:39:36 UTC
Proposed fix:

commit 8fc03681cdc88f9bf7adc7418bbdbf7d072e9530
Author: Federico Simoncelli <fsimonce>
Date:   Tue Dec 13 09:37:41 2011 +0000

    BZ#767111 Extend volume size must be in Megabyte

    In commit c24396af (Merged SPM to StoragePool and HSM) the
    spm.public_extendVolume call was replaced by pool.extendVolume.
    The problem is that they use different units for the size
    parameter, the first one uses bytes and the second one uses
    megabytes.

    In this patch:
    * do not convert megabytes to bytes when calling pool.extendVolume
    * improve logging to record also the size received from the mailbox

    Change-Id: I955da5ca52708eb9bfd51df66f2a0e13e8eaf3c8

http://gerrit.ovirt.org/579

Comment 3 Itamar Heim 2012-08-09 08:01:00 UTC
closing ON_QA bugs as oVirt 3.1 was released:
http://www.ovirt.org/get-ovirt/


Note You need to log in before you can comment on or make changes to this bug.