Re: [pve-devel] [PATCH] implement vdisk_clone and vdisk_create_base

2013-01-30 Thread Alexandre DERUMIER
>> Example volume names: >>> >>> local:6000/base-6000-disk-9.raw >>> local:6000/base-6000-disk-9.raw/7000/vm-7000-disk-9.qcow2 >>> local:6000/base-6000-disk-9.raw/7000/base-7000-disk-10.qcow2 >>That way it is easy to check if a volume is a clone or a template. Do you >>think >>this can be i

Re: [pve-devel] allow ldap group access

2013-01-30 Thread Dietmar Maurer
> i would like to allow a group of ldap users access to pve. But it seems this > is > not possible. I have to specifiy the access level for each user. > Is this correct? Yes. I you really need that you can write some import script. Maybe we can add an LDAP configuration option like 'import

[pve-devel] allow ldap group access

2013-01-30 Thread Stefan Priebe
Hello, i would like to allow a group of ldap users access to pve. But it seems this is not possible. I have to specifiy the access level for each user. Is this correct? Greets, Stefan ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.p

Re: [pve-devel] Snapshots with RAM don't work

2013-01-30 Thread Stefan Priebe - Profihost AG
Hi, Am 30.01.2013 14:27, schrieb Dietmar Maurer: > we need to use the new command. done >> Would you like to have a patchset to make pve-qemu-kvm compatible with >> 1.4? > > Yes, but I will start that in 2 weeks (not earlier). I will let you know when > I need that patches. OK just send a mail

Re: [pve-devel] Snapshots with RAM don't work

2013-01-30 Thread Dietmar Maurer
> I've a question regarding the virtio ballooning patches. qemu 1.4 supports > the new statistics out of the box but they've changed the command to: guest- > stats-polling-interval . we need to use the new command. > > Should i patch qemu to support the old stats-polling-interval or patch pve? n

Re: [pve-devel] Snapshots with RAM don't work

2013-01-30 Thread Stefan Priebe - Profihost AG
Hi, sorry got it working. So this patch is now also working fine with qemu 1.4. I've a question regarding the virtio ballooning patches. qemu 1.4 supports the new statistics out of the box but they've changed the command to: guest-stats-polling-interval . Should i patch qemu to support the old s

Re: [pve-devel] Snapshots with RAM don't work

2013-01-30 Thread Stefan Priebe - Profihost AG
Hi, OK saving / snapshotting works fine now with 1.4. I've not ported it to threads like the migration code but i adpoted the changes to the while loop. But restoring does still not work. Anjy ideas where to look here? Am 30.01.2013 06:02, schrieb Dietmar Maurer: >> Everything works fine except

Re: [pve-devel] [PATCH] implement vdisk_clone and vdisk_create_base

2013-01-30 Thread Dietmar Maurer
> Example volume names: > > local:6000/base-6000-disk-9.raw > local:6000/base-6000-disk-9.raw/7000/vm-7000-disk-9.qcow2 > local:6000/base-6000-disk-9.raw/7000/base-7000-disk-10.qcow2 That way it is easy to check if a volume is a clone or a template. Do you think this can be implemented for all st

[pve-devel] [PATCH] implement vdisk_clone and vdisk_create_base

2013-01-30 Thread Dietmar Maurer
This is an iplementation for file base storage types. changes compared to patches from Alexandre: * use correct locking * private find_free_diskname() with bug fixes * changed names of new methods * always refer to base volumes in volume names Example volume names: local:6000/base-6000-disk-9.r

Re: [pve-devel] Snapshots with RAM don't work

2013-01-30 Thread Stefan Priebe - Profihost AG
Hi, Am 30.01.2013 06:02, schrieb Dietmar Maurer: >> Everything works fine except the savevm-async - it compiles fine but the VM >> hangs. Do you have already an idea what needs to be changes there? So >> maybe i can start working on that. > > Compare with the code in migration.c (migrate_fd_put_re

Re: [pve-devel] qemu-server : templates - clone - copy V1

2013-01-30 Thread Alexandre DERUMIER
>>The series start with patches about volume_protect().  >>But this should not be required at all with the new system?  Yes, indeed, we can't start the template, but Isn't it always better to have a protection on the storage ? (user try to move file manually with ssh, or something else...) Only

Re: [pve-devel] [PATCH 02/48] Plugin : add volume_protect

2013-01-30 Thread Michael Rasmussen
On Wed, 30 Jan 2013 06:12:56 + Dietmar Maurer wrote: > > What is the advantage of 'chattr +i'? I usually use 'chmod -w' to > set a file read only. > chattr +i also prevents creating hard-links to the file which a read-only file does not guaranty against. A hard-link would effectively preven