[pve-devel] [PATCH v2 2/3] ext5migrate: do not set a custome idProperty for the KeyValue model

2015-08-18 Thread Emmanuel Kasper
ExtJS refuses with version to have an id set to an empty string, hence we can't use our key property as a custom idProperty (an empty string for key is ok for us, and is used to set back a PVE property to its default value) We always access the KeyValues entities via their Key property, so this

[pve-devel] [PATCH v2 3/3] Declare data object in one statement when we want to delete a parameter

2015-08-18 Thread Emmanuel Kasper
Also add missing formatting --- www/manager5/form/KVComboBox.js | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/www/manager5/form/KVComboBox.js b/www/manager5/form/KVComboBox.js index 77bf8f4..8b66dfc 100644 --- a/www/manager5/form/KVComboBox.js +++

[pve-devel] v2 Fix PVE Combobox to work with ExtJS5

2015-08-18 Thread Emmanuel Kasper
Add missing commit message summary line to first patch. ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] [PATCH v2 1/3] ext5migrate: pass the items we want in the ComboBox as a config property

2015-08-18 Thread Emmanuel Kasper
Since ExtJS5, when doing such a declaration, Ext.applyIf(me, { items: { xtype: 'pveKVComboBox', comboItems: data, } }); the comboItems property is passed in the parent class as a config object, instead of being directly

[pve-devel] [PATCH] mountpoint: add support for host /dev/xxx block device passthrough

2015-08-18 Thread Alexandre Derumier
example: mp0: volume=/dev/sda,mp=/home/ Fix this request: http://forum.proxmox.com/threads/23256-LXC-Cannot-assign-a-block-device-to-container Signed-off-by: Alexandre Derumier aderum...@odiso.com --- src/PVE/LXC.pm| 2 +- src/lxc-pve-mount-hook| 5 +

[pve-devel] [PATCH 3/3] Declare data object in one statement when we want to delete a parameter

2015-08-18 Thread Emmanuel Kasper
Also add missing formatting --- www/manager5/form/KVComboBox.js | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/www/manager5/form/KVComboBox.js b/www/manager5/form/KVComboBox.js index 77bf8f4..8b66dfc 100644 --- a/www/manager5/form/KVComboBox.js +++

[pve-devel] [PATCH 2/3] ext5migrate: do not set a custome idProperty for the KeyValue model

2015-08-18 Thread Emmanuel Kasper
ExtJS refuses with version to have an id set to an empty string, hence we can't use our key property as a custom idProperty (an empty string for key is ok for us, and is used to set back a PVE property to its default value) We always access the KeyValues entities via their Key property, so this

Re: [pve-devel] [PATCH 1/5] vm_start : attach loop devices for non rootfs mountpoint

2015-08-18 Thread Alexandre DERUMIER
I have re-tested with subvol, loop and rbd, seem to works fine. (and more clean) - Mail original - De: dietmar diet...@proxmox.com À: aderumier aderum...@odiso.com Cc: pve-devel pve-devel@pve.proxmox.com Envoyé: Mardi 18 Août 2015 07:33:14 Objet: Re: [pve-devel] [PATCH 1/5] vm_start :

[pve-devel] [PATCH 1/2] ext5migrate: enable striped row, it now works with ExtJS5

2015-08-18 Thread Emmanuel Kasper
--- www/manager5/dc/Log.js | 2 +- www/manager5/dc/Tasks.js | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/www/manager5/dc/Log.js b/www/manager5/dc/Log.js index 75eccb0..385823b 100644 --- a/www/manager5/dc/Log.js +++ b/www/manager5/dc/Log.js @@ -30,7 +30,7 @@

[pve-devel] [PATCH 2/2] ext5migrate: move setting the Panel title into the class properties

2015-08-18 Thread Emmanuel Kasper
Ext.applyIf(me, { title: gettext('Search') }); in initComponent() is not safe anymore with ExtJS5. Since the 'title' property is existing at that stage (even if with a null or undefined value), Ext.applyIf will not update it. --- www/manager5/grid/ResourceGrid.js | 6 ++ 1 file

Re: [pve-devel] memory consumption

2015-08-18 Thread Alexandre DERUMIER
Hi, ) it only happens on machines with ceph osd installed ) which also are using rbd client for virtual hdds I'm never see this, maybe it could be a tcmalloc bug not releasing memory. Is it the kvm process witch use a lot more ram ? - Mail original - De: VELARTIS Philipp Dürhammer

Re: [pve-devel] [PATCH pve-storage] upload API: safer filename handling

2015-08-18 Thread Wolfgang Bumiller
Ah yes, missed that when cleaning up. On August 18, 2015 at 1:36 PM Thomas Lamprecht t.lampre...@proxmox.com wrote: @@ -382,14 +382,15 @@ __PACKAGE__-register_method ({ }; die can't activate storage '$param-{storage}' on node '$node'\n if $@; -

Re: [pve-devel] The network performance future for VMs

2015-08-18 Thread Alexandre DERUMIER
So, i would like to ask about of the future of PVE in network performance terms. dpdk will be implemented in openvswitch through vhost-user, I'm waiting for ovs 2.4 to look at this. - Mail original - De: Cesar Peschiera br...@click.com.py À: pve-devel pve-devel@pve.proxmox.com Envoyé:

Re: [pve-devel] [PATCH pve-storage] upload API: safer filename handling

2015-08-18 Thread Thomas Lamprecht
@@ -382,14 +382,15 @@ __PACKAGE__-register_method ({ }; die can't activate storage '$param-{storage}' on node '$node'\n if $@; - PVE::Tools::run_command([@remcmd, '/bin/mkdir', '-p', $dirname], + my $quoted_dir = PVE::Tools::shell_quote($dirname); +

Re: [pve-devel] memory consumption

2015-08-18 Thread Eneko Lacunza
Hi Philipp, What Ceph packages? We have 3 small clusters using creph/rbd and I'm not seeing this. Biggest VM is 6GB of RAM though, biggest disk is 200GB and a VM has a total of 425GB in 5 disks. Are you using Proxmox Ceph Server? What cache setting in VM harddisk config? Cheers Eneko El

[pve-devel] [PATCH pve-storage] upload API: safer filename handling

2015-08-18 Thread Wolfgang Bumiller
Replace possibly-dangerous characters in uploaded filenames with underscores, this includes spaces, colons, commas, equal signs and any byte = 128. Previously only spaces were turned into underscores. Also shell_quote the destination for scp, and to make life easier - since the destination

[pve-devel] memory consumption

2015-08-18 Thread VELARTIS Philipp Dürhammer
Hi, some of my virtual machines on different servers are consuming a lot more ram then they should! (up to twice) which is a horror if the machines have about 15-20G but consume 30-40GB I was checking: ) all machines have the same pve packages installed ) it only happens on machines with ceph

Re: [pve-devel] qemu-sever=4.0-15 and pve-qemu-kvm2.4-1 are incompatible at the moment

2015-08-18 Thread Wolfgang Bumiller
Ah, I checked the forum thread you mentioned and it looks like the machines they were trying to backup were still running, so they were probably already running before upgrading, iow. still running the incompatible binary, so of course that would be a problem with an API change like that. On

Re: [pve-devel] qemu-sever=4.0-15 and pve-qemu-kvm2.4-1 are incompatible at the moment

2015-08-18 Thread Wolfgang Bumiller
I would propose to increase the dependencies for the qemu-server package The only other alternative would be changing the pve-qemu-kvm patches to whitelist this API call in qemu to allow returning builtin types again and go back to the way it used to work previously. Would mean more

Re: [pve-devel] The network performance future for VMs

2015-08-18 Thread Cesar Peschiera
Oh, ok. In the past, i had problems with DRBD 8.4.5 when OVS is enabled, so i had that change my setup from OVS to the Linux stack, after of it, i had no more problems with DRBD. About of the problem with OVS and DRBD, i did not test in depth the problem (in the season of preproduction

Re: [pve-devel] memory consumption

2015-08-18 Thread VELARTIS Philipp Dürhammer
OK, i was testing around a bit and my conclution is: because of setting the cache size too big the vms used a lot more memory then expected. 1) because oft he bug described by alexandre changing the cache setting didn't change anything 2) my ceph.conf does not show in ceph -n mon.2

Re: [pve-devel] memory consumption

2015-08-18 Thread Alexandre DERUMIER
if cache=none is set this should overwrite the ceph config from rbd cache = true= false but still this machine is consuming 20G instead of 16G Be carefull, they are a bug in current qemu librbd, setting rbd_cache=true|false in ceph.conf, always override qemu cache setting. This is fixed in

Re: [pve-devel] memory consumption

2015-08-18 Thread Alexandre DERUMIER
To be more precise some values like network ip's etc still are shown and seem to work but not the cache setting and also backbill settings The cache setting is client side setting, monitors don't known nothing about it. - Mail original - De: VELARTIS Philipp Dürhammer

Re: [pve-devel] memory consumption

2015-08-18 Thread VELARTIS Philipp Dürhammer
Ok thank you. The strange behaviour with the same result but different cache settings confused me even more... But one thing that i think still needs improvement: Proxmox bare metal now installs ceph client by default but does not symlink /etc/ceph/ceph.conf to /etc/pve/ceph.conf So on

Re: [pve-devel] [PATCH resend pve-container 3/3] add file access methods to LXCSetup::Base

2015-08-18 Thread Dietmar Maurer
this one doe snot apply - please can you rebase? ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Re: [pve-devel] memory consumption

2015-08-18 Thread Dietmar Maurer
Proxmox bare metal now installs ceph client by default but does not symlink /etc/ceph/ceph.conf to /etc/pve/ceph.conf So on all rbd client nodes qemu runns with standard ceph conf because it is not able to find the file Whats wrong with ceph standard conf? The file /etc/pve/ceph.conf is

Re: [pve-devel] The network performance future for VMs

2015-08-18 Thread Alexandre DERUMIER
So now my question is if DPDK can be activated also with the Linux stack?. I need to dig a little more about this. Intel seem to push the ovs-dpdk in all conferenfece I have see. (Seem to be easy with vhost-user virtual network card, and this one can't work with linux bridge, because it's

Re: [pve-devel] [PATCH v2 pve-storage] upload API: safer filename handling

2015-08-18 Thread Dietmar Maurer
applied modified version On 08/18/2015 03:16 PM, Wolfgang Bumiller wrote: Ah, the commit message isn't up to date either now: I forgot to remove this part: , and to make life easier ... (up to the end of the paragraph) That's actually not the case (anymore).

Re: [pve-devel] [PATCH pve-container 1/3] archlinux template support

2015-08-18 Thread Dietmar Maurer
Please can you resend this series - I miss patch 3/3. ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Re: [pve-devel] [PATCH v2 3/3] Declare data object in one statement when we want to delete a parameter

2015-08-18 Thread Dietmar Maurer
On 08/18/2015 10:36 AM, Emmanuel Kasper wrote: Also add missing formatting --- www/manager5/form/KVComboBox.js | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/www/manager5/form/KVComboBox.js b/www/manager5/form/KVComboBox.js index 77bf8f4..8b66dfc 100644 ---

[pve-devel] [PATCH resend pve-container 2/3] Added PVE::LXCSetup::Base::setup_securetty

2015-08-18 Thread Wolfgang Bumiller
--- src/PVE/LXCSetup/ArchLinux.pm | 3 +++ src/PVE/LXCSetup/Base.pm | 14 ++ src/PVE/LXCSetup/Redhat.pm| 10 +- src/PVE/LXCSetup/Ubuntu.pm| 7 +-- 4 files changed, 19 insertions(+), 15 deletions(-) diff --git a/src/PVE/LXCSetup/ArchLinux.pm

[pve-devel] [PATCH resend pve-container 3/3] add file access methods to LXCSetup::Base

2015-08-18 Thread Wolfgang Bumiller
These should be used for all container file access as in the future they'll support user-id mapping. --- src/PVE/LXCSetup/ArchLinux.pm | 12 src/PVE/LXCSetup/Base.pm | 72 +++ src/PVE/LXCSetup/Debian.pm| 16 --

[pve-devel] [PATCH resend pve-container 1/3] archlinux template support

2015-08-18 Thread Wolfgang Bumiller
--- src/PVE/LXC.pm| 2 +- src/PVE/LXCSetup.pm | 10 +--- src/PVE/LXCSetup/ArchLinux.pm | 54 +++ src/PVE/LXCSetup/Base.pm | 53 ++ src/PVE/LXCSetup/Makefile | 2 +- 5 files

Re: [pve-devel] memory consumption

2015-08-18 Thread VELARTIS Philipp Dürhammer
Hi, debugging and reading more if cache=none is set this should overwrite the ceph config from rbd cache = true= false but still this machine is consuming 20G instead of 16G Von: pve-devel [mailto:pve-devel-boun...@pve.proxmox.com] Im Auftrag von VELARTIS Philipp Dürhammer Gesendet: Dienstag,

Re: [pve-devel] [PATCH v2 pve-storage] upload API: safer filename handling

2015-08-18 Thread Wolfgang Bumiller
Ah, the commit message isn't up to date either now: I forgot to remove this part: , and to make life easier ... (up to the end of the paragraph) That's actually not the case (anymore). On August 18, 2015 at 1:54 PM Wolfgang Bumiller w.bumil...@proxmox.com wrote: Replace possibly-dangerous

Re: [pve-devel] memory consumption

2015-08-18 Thread VELARTIS Philipp Dürhammer
Hi Eneko! I use Proxmox Ceph Server. I found out that my config for rbd cache size was too big which could explain me the problem. But when i make: ceph -n mon.2 --show-config it shows me the standard config size from ceph. And also that rbd cache = false. Very strange. Is this not the way to