[pve-devel] [RFC pve-storage] allow subvolume selection in ZPool storage

2015-08-27 Thread Wolfgang Bumiller
--- PVE/Storage.pm | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/PVE/Storage.pm b/PVE/Storage.pm index 23780d0..9a313b0 100755 --- a/PVE/Storage.pm +++ b/PVE/Storage.pm @@ -986,17 +986,17 @@ sub scan_nfs { sub scan_zfs { -my $cmd = ['zpool', 'list', '-H',

[pve-devel] [PATCH] fix 'uninitialized value in concat' due to unnamed VM

2015-08-27 Thread Thomas Lamprecht
This is an minor fix which let's you start the spice console on an unnamed VM without getting the 'Use of uninitialized value in concatenation' error. Also changes the trailing comma from the $conf definition lineto an semicolon. Signed-off-by: Thomas Lamprecht t.lampre...@proxmox.com ---

[pve-devel] [PATCH v2 qemu-server] fix 'uninitialized value in concat' due to unnamed VM

2015-08-27 Thread Thomas Lamprecht
This is an minor fix which let's you start the spice console on an unnamed VM without getting the 'Use of uninitialized value in concatenation' error. Also changes the trailing comma from the $conf definition lineto an semicolon. Signed-off-by: Thomas Lamprecht t.lampre...@proxmox.com ---

[pve-devel] [PATCH] run spiceterm and return spiceproxy config only if CT is running

2015-08-27 Thread Thomas Lamprecht
This mirrors the behavior of qemu VMs, if the container is not running, return an error when trying to launch the spiceproxy API call. Signed-off-by: Thomas Lamprecht t.lampre...@proxmox.com --- src/PVE/API2/LXC.pm | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/PVE/API2/LXC.pm

Re: [pve-devel] ZFSPlugin snapshot question

2015-08-27 Thread Michael Rasmussen
On Thu, 27 Aug 2015 19:31:49 +0200 (CEST) Dietmar Maurer diet...@proxmox.com wrote: Do you have time to implement that? We would need it to implement snapshot backup for LXC containers. Yes, I will look in to this this weekend. Forgot to mention. Since snapshots by nature is read-only you

Re: [pve-devel] ZFSPlugin snapshot question

2015-08-27 Thread Michael Rasmussen
On Thu, 27 Aug 2015 17:46:45 +0200 (CEST) Dietmar Maurer diet...@proxmox.com wrote: But how can we implement this with the ZFSPlugin?`Any idea if/how it is possible to access a snapshotted device? There are two preconditions which must be fulfilled: 1) LUN must be pointing to the snapshot

[pve-devel] ZFSPlugin snapshot question

2015-08-27 Thread Dietmar Maurer
Hi all, the PVE::Storage::path method has the following signature: PVE::Storage::path($cfg, $volid, $snapname); As you can see, It has that $snapname parameter, and I tried to correctly implement this here:

[pve-devel] [PATCH pve-manager] set size in the rootfs parameter on CT create

2015-08-27 Thread Thomas Lamprecht
As the size parameter is now deprecated and was replaced with the new rootfs parameter, use that one when creating an container. So we don't get an 'parameter verification failed' error. Signed-off-by: Thomas Lamprecht t.lampre...@proxmox.com --- www/manager/lxc/CreateWizard.js | 2 +- 1 file

[pve-devel] [PATCH] fix: start kvm with os type other

2015-08-27 Thread Wolfgang Link
this check is necessary, because we do not set ostype in qemu config when type is other --- PVE/QemuServer.pm | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm index 7115007..53fbcc9 100644 --- a/PVE/QemuServer.pm +++

Re: [pve-devel] [PATCH] fix: start kvm with os type other

2015-08-27 Thread Alexandre DERUMIER
Hi, Seem that x2apic is now enabled by default when kvm is used since this commit http://git.qemu.org/?p=qemu.git;a=commit;h=ef02ef5f4536dba090b12360a6c862ef0e57e3bc So, I think we just need to disable it for solaris + push @$cpuFlags , '-x2apic' if !$nokvm $conf-{ostype} eq 'solaris';

Re: [pve-devel] Unable to restore container backup with vzdump command

2015-08-27 Thread Emmanuel Kasper
Answering to self: we use now pct restore pct restore 100 /var/lib/vz/dump/vzdump-lxc-104-2015_08_27-11_45_22.tar.lzo On 08/27/2015 12:31 PM, Emmanuel Kasper wrote: Hi I am trying to restore a vzdump backup with the following command: vzrestore

[pve-devel] Unable to restore container backup with vzdump command

2015-08-27 Thread Emmanuel Kasper
Hi I am trying to restore a vzdump backup with the following command: vzrestore /var/lib/vz/dump/vzdump-lxc-104-2015_08_27-11_45_22.tar.lzo 100 I am always getting the error: vzrestore /var/lib/vz/dump/vzdump-lxc-104-2015_08_27-11_45_22.tar.lzo 100 can't lock container 100 even though there is

[pve-devel] [PATCH] novnc : bump to a369a80c2480af685b7bddacddae9cca3aea95f9 commit (20/08/2015) V2

2015-08-27 Thread Alexandre Derumier
They are performances improvements and fullscreen button support changelog: only display fullscreen button is resize=scale is defined Signed-off-by: Alexandre Derumier aderum...@odiso.com --- debian/install| 4 +- debian/patches/fix-base-css.patch | 2 +-

Re: [pve-devel] [PATCH pve-container] Add handling for arch and ostype options in update_pct_config()

2015-08-27 Thread Dietmar Maurer
OK I will try to look at that (do not call update_pct_config with those options) , but maybe takes some time, this will be my first real dive in Perl ! I guess this is already fixed with: https://git.proxmox.com/?p=pve-container.git;a=commitdiff;h=ed680718291843c7987181eeff2a8e038b920185

Re: [pve-devel] [PATCH pve-container] Add handling for arch and ostype options in update_pct_config()

2015-08-27 Thread Emmanuel Kasper
On 08/27/2015 09:02 AM, Dietmar Maurer wrote: OK I will try to look at that (do not call update_pct_config with those options) , but maybe takes some time, this will be my first real dive in Perl ! I guess this is already fixed with:

Re: [pve-devel] [PATCH 5/5] make create_rootfs sub generic

2015-08-27 Thread Alexandre DERUMIER
I committed everything now, so you can continue ;-) Ok. (and thanks for the cleanup !) - Mail original - De: dietmar diet...@proxmox.com À: aderumier aderum...@odiso.com Cc: pve-devel pve-devel@pve.proxmox.com Envoyé: Jeudi 27 Août 2015 09:08:41 Objet: Re: [pve-devel] [PATCH 5/5] make

Re: [pve-devel] [PATCH 5/5] make create_rootfs sub generic

2015-08-27 Thread Dietmar Maurer
I wonder if that unmount everything (mountpoints below /)? Currently not, because anyway, I'm still only manage rootfs. (I'm waiting to have this patch serie commited before continue) I committed everything now, so you can continue ;-) ___