[pve-devel] pve-manager : add hd resize feature

[Alexandre DERUMIER]

gui patches just implement api call to "qm resize ...."
so it should work since some week with qm command.




sub volume_resize {
    my ($class, $scfg, $storeid, $volname, $size, $running) = @_;

     >> if running, we return 1 to qemu_block_resize (so it'll use vm_mon_cmd qm resize)
    return 1 if $running;  


    >> if not running, we resize offline with rbd command

    my $cmd = &$rbd_cmd($scfg, $storeid, 'resize', '--size', ($size/1024/1024), $volname);
    run_command($cmd, errmsg => "rbd resize $volname' error", errfunc => sub {});
    return undef;
}


do you use virtio or virtio-scsi ?

It's work fine for me with virtio.


----- Mail original ----- 

De: "Stefan Priebe - Profihost AG" <s.priebe at profihost.ag> 
À: "Alexandre DERUMIER" <aderumier at odiso.com> 
Cc: pve-devel at pve.proxmox.com 
Envoyé: Vendredi 14 Décembre 2012 15:55:31 
Objet: Re: [pve-devel] pve-manager : add hd resize feature 

Hi Alexandre, 

strange. It does not work to me. I've only applied these two patches - 
do i need more patches? 

I've lost the overview of all your patches. 

Stefan 

Am 14.12.2012 15:53, schrieb Alexandre DERUMIER: 
> I have also retested with rbd device on virtio disk (kernel 3.6), and it's working fine too ... 
> 
> ----- Mail original ----- 
> 
> De: "Stefan Priebe" <s.priebe at profihost.ag> 
> À: "Alexandre Derumier" <aderumier at odiso.com> 
> Cc: pve-devel at pve.proxmox.com 
> Envoyé: Jeudi 13 Décembre 2012 22:30:07 
> Objet: Re: [pve-devel] pve-manager : add hd resize feature 
> 
> Hi Alexandre, 
> 
> this works fine but the new size isn't exposed / send to guest. The new 
> size should be viewable in guest when kernel supports this. 
> 
> So shouldn't we add this: 
> my $input = "block_resize drive-scsi1 ${size}G"; 
> PVE::QemuServer::vm_human_monitor_command ( $vmid, $input ); 
> 
> So the guest already know about new size if guest kernel supports it. 
> 
> Greets, 
> Stefan 
> 
> Am 13.12.2012 15:41, schrieb Alexandre Derumier: 
>> Please review, but I think it's clean. 
>> 
>> _______________________________________________ 
>> pve-devel mailing list 
>> pve-devel at pve.proxmox.com 
>> http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel 
>>