[pve-devel] Touch ExtJS files after unpacking them (speeds up the build)

2015-03-26 Thread Emmanuel Kasper
Hello I have noticed when calling make install in pve-manager/www/ext4 that the make all target was called three times in a row, because the timestamps of files inside a zip archives are older than the timestamp of the zip itself. Adding a a simple touch after unpacking the files fixes this.

[pve-devel] [PATCH] Add touch command after unpacking our ext js files.

2015-03-26 Thread Emmanuel Kasper
Files inside a zip archive have an older modified date than the zip which contains them. Touching the files after their creation will prevent unnecessary execution of the target and will speed up the build. Signed-off-by: Emmanuel Kasper e.kas...@proxmox.com --- www/ext4/Makefile | 1 + 1 file

[pve-devel] Moving to ext js 5, the first steps

2015-03-30 Thread Emmanuel Kasper
Hello I have started some preliminary work to upgrade the ext js framework we use in the pvemanager web ui to version 5.1 This should bring us some slight performance improvements in grid rendering, and a more modern themed ui. Please note that Ext JS 5.1 supports Internet Explorer = 8.0, so

[pve-devel] [PATCH 2/2] Add makefile targets for the ext js framework in version 5.1 (gpl)

2015-03-30 Thread Emmanuel Kasper
This commit only installs the required javascript / css files for ext js 5.1 in /usr/share/pve-manager/ext5, but does not use it by default, as we have to sort out the upgrade issues between 4.0 and 5.1 before. Signed-off-by: Emmanuel Kasper e.kas...@proxmox.com --- defines.mk | 1

[pve-devel] [PATCH] Touch data after unpacking them. This will prevent the make install target to be called twice in a row.

2015-03-30 Thread Emmanuel Kasper
Signed-off-by: Emmanuel Kasper e.kas...@proxmox.com --- www/touch/Makefile | 1 + 1 file changed, 1 insertion(+) diff --git a/www/touch/Makefile b/www/touch/Makefile index 9e41864..5b7efa6 100644 --- a/www/touch/Makefile +++ b/www/touch/Makefile @@ -10,6 +10,7 @@ TOUCHDATA

Re: [pve-devel] [PATCH 2/2] Add makefile targets for the ext js framework in version 5.1 (gpl)

2015-03-31 Thread Emmanuel Kasper
On 03/31/2015 06:28 AM, Dietmar Maurer wrote: Applied, thanks! I found some duplicated files after installation: /usr/share/pve-manager/ext5/locale/ext-locale-de.js /usr/share/pve-manager/ext5/packages/ext-locale/build/ext-locale-de.js Seems locale files are installed twice? And do we really

[pve-devel] [PATCH] Restrict ext js 5 theme installation to ext-theme-classic and its base css classes

2015-03-31 Thread Emmanuel Kasper
Signed-off-by: Emmanuel Kasper e.kas...@proxmox.com --- www/ext5/Makefile | 12 +--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/www/ext5/Makefile b/www/ext5/Makefile index 3c8fa8e..559db73 100644 --- a/www/ext5/Makefile +++ b/www/ext5/Makefile @@ -8,6 +8,12 @@ EXTDATA

[pve-devel] Debiam 8 released planned on 25 April

2015-04-01 Thread Emmanuel Kasper
Hello I have read today in the debian-devel-announce list that they plan the next release for the 25th of April https://lists.debian.org/debian-devel-announce/2015/03/msg00016.html lg Emmanuel ___ pve-devel mailing list pve-devel@pve.proxmox.com

[pve-devel] [PATCH 1/2] Add url routing for serving extjs 5 files

2015-04-02 Thread Emmanuel Kasper
Signed-off-by: Emmanuel Kasper e.kas...@proxmox.com --- bin/pveproxy | 1 + 1 file changed, 1 insertion(+) diff --git a/bin/pveproxy b/bin/pveproxy index c0eecb1..09d1215 100755 --- a/bin/pveproxy +++ b/bin/pveproxy @@ -91,6 +91,7 @@ sub init { add_dirs($dirs, '/pve2/locale/', '/usr/share

[pve-devel] [PATCH 2/2] Add HTML doctype to our web ui, remove IE10-specific standard mode tag.

2015-04-02 Thread Emmanuel Kasper
are anyway not supported by ext5. The HTML doctype will make all browsers switch to their standard compliant mode, hence we don't need anymore to toggle this specifically for IE. Signed-off-by: Emmanuel Kasper e.kas...@proxmox.com --- PVE/ExtJSIndex.pm | 2 +- 1 file changed, 1 insertion(+), 1

[pve-devel] [PATCH 1/2] Add new ExtJSIndex5 package to allow running the pveproxy with two different versions of extjs

2015-04-02 Thread Emmanuel Kasper
Signed-off-by: Emmanuel Kasper e.kas...@proxmox.com --- PVE/ExtJSIndex5.pm | 69 ++ 1 file changed, 69 insertions(+) create mode 100644 PVE/ExtJSIndex5.pm diff --git a/PVE/ExtJSIndex5.pm b/PVE/ExtJSIndex5.pm new file mode 100644 index 000

[pve-devel] [PATCH 2/2] Enhance pveproxy by adding an optional ext5 parameter to load the extjs 5 framework

2015-04-02 Thread Emmanuel Kasper
Calling the pveproxy with https://localhost:8006/?ext5=1 will load the web ui with the extjs 5 js libs. Please note that at the momment the workspace does not load with extjs 5. Signed-off-by: Emmanuel Kasper e.kas...@proxmox.com --- PVE/Makefile | 1 + bin/pveproxy | 19 --- 2

[pve-devel] [PATCH 1/3] Initial import of the pve manager with ext5 code path

2015-04-14 Thread Emmanuel Kasper
Developement of the ext5 version of the pve-manager will take place in the www/management5 directory of the git repo. We start the initial import with Utils.js, who contains variables and static functions used by all other components. Signed-off-by: Emmanuel Kasper e.kas...@proxmox.com --- www

[pve-devel] [PATCH 2/3] Add extra /manager5 url route to serve local javascript files

2015-04-14 Thread Emmanuel Kasper
-off-by: Emmanuel Kasper e.kas...@proxmox.com --- bin/pveproxy | 1 + 1 file changed, 1 insertion(+) diff --git a/bin/pveproxy b/bin/pveproxy index bcec0f9..fe25a80 100755 --- a/bin/pveproxy +++ b/bin/pveproxy @@ -93,6 +93,7 @@ sub init { add_dirs($dirs, '/pve2/touch/', '/usr/share/pve-manager

[pve-devel] [PATCH 3/3] Change source attribute of javascript include to point to our manager5 directory.

2015-04-14 Thread Emmanuel Kasper
For ext 5 developpment, we will include in the html headers every javascript ressources incrementally, so we can bypass the make install step. Signed-off-by: Emmanuel Kasper e.kas...@proxmox.com --- PVE/ExtJSIndex5.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/PVE

[pve-devel] [PATCH 1/2] switch default extjs 5 theme to neptune

2015-04-29 Thread Emmanuel Kasper
--- PVE/ExtJSIndex5.pm | 2 +- www/ext5/Makefile | 7 +++ 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/PVE/ExtJSIndex5.pm b/PVE/ExtJSIndex5.pm index 45931aa..1fd6dbc 100644 --- a/PVE/ExtJSIndex5.pm +++ b/PVE/ExtJSIndex5.pm @@ -13,7 +13,7 @@ sub get_index { meta

[pve-devel] [PATCH 2/2] ext5migrate: replace deprecated call Ext.ModelMgr.create()

2015-04-29 Thread Emmanuel Kasper
Ext.ModelMgr.create() was removed in 5.* The recommended way to create a new instance is to call Ext.create() Ext.create() does not take a id argument, however it sets the id of the instance to the id of the config object given to the Ext.create() call. --- www/manager5/tree/ResourceTree.js | 2

[pve-devel] [PATCH] Add Readme explaining how to use extjs5 for dev

2015-04-30 Thread Emmanuel Kasper
--- www/manager5/Readme.md | 22 ++ 1 file changed, 22 insertions(+) create mode 100644 www/manager5/Readme.md diff --git a/www/manager5/Readme.md b/www/manager5/Readme.md new file mode 100644 index 000..87e00c3 --- /dev/null +++ b/www/manager5/Readme.md @@ -0,0 +1,22 @@

[pve-devel] [PATCH] Add manager5 directory in case of missing development symlink

2015-05-06 Thread Emmanuel Kasper
--- Makefile | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Makefile b/Makefile index 429264b..a09393c 100644 --- a/Makefile +++ b/Makefile @@ -102,6 +102,8 @@ install: country.dat vznet.conf vzdump.conf vzdump-hook-script.pl pve-apt.conf p install -m 0644 copyright ${DOCDIR}

[pve-devel] [PATCH] ext5migrate: remove Ext.grid.feature.Chunking hack

2015-05-06 Thread Emmanuel Kasper
With ExtJS 4, we introduced an override to Ext.grid.feature.Chunking to fix scrollings problems in the grid when having a high number of nodes. Ext.grid.feature.Chunking has been removed from ExtJS in version 5, so we hope either the problem is fixed on ExtJS side, or we will have to find a

[pve-devel] [PATCH] Add required url routing for ext5 migration directory

2015-05-06 Thread Emmanuel Kasper
--- PVE/ExtJSIndex5.pm | 4 1 file changed, 4 insertions(+) diff --git a/PVE/ExtJSIndex5.pm b/PVE/ExtJSIndex5.pm index 1fd6dbc..a3da17b 100644 --- a/PVE/ExtJSIndex5.pm +++ b/PVE/ExtJSIndex5.pm @@ -39,6 +39,10 @@ _EOD script type=text/javascript

[pve-devel] [PATCH 1/2] Load KVComboBox.js earlier as it is needed for the login window

2015-05-13 Thread Emmanuel Kasper
--- PVE/ExtJSIndex5.pm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/PVE/ExtJSIndex5.pm b/PVE/ExtJSIndex5.pm index a3da17b..34952ac 100644 --- a/PVE/ExtJSIndex5.pm +++ b/PVE/ExtJSIndex5.pm @@ -35,9 +35,9 @@ _EOD script type=text/javascript

[pve-devel] [PATCH 2/2] ext5migrate: Replace obsolete call to Ext.ModelMgr.create()

2015-05-13 Thread Emmanuel Kasper
See 2489e2abe35b46d1c7897dd9431eecaedbccd9e5 for reasoning --- www/manager5/grid/ResourceGrid.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/www/manager5/grid/ResourceGrid.js b/www/manager5/grid/ResourceGrid.js index 0449573..3dfbc45 100644 ---

Re: [pve-devel] A strategy if the upgrade of wheezy to jessie isn'tpossible

2015-05-12 Thread Emmanuel Kasper
Hello Cesar On 05/12/2015 01:16 PM, Cesar Peschiera wrote: Alexandre, many thanks for the information. And please, let me to do a questions: 1) Should i change the information of his repositories for his future updates? (in terms of Debian Jessie and PVE no-subscription), as also, what

[pve-devel] [PATCH 1/3] ext5migrate: replace obsolete 'margins' parameter with 'margin'

2015-05-19 Thread Emmanuel Kasper
As mentioned in http://docs.sencha.com/extjs/5.1/whats_new/5.0/extjs_upgrade_guide.html the 'margins' component parameter is now obsolete. --- www/manager5/Workspace.js | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/www/manager5/Workspace.js

[pve-devel] [PATCH 3/3] remove hardcoded span html tag, as it appears to be unused

2015-05-19 Thread Emmanuel Kasper
--- www/manager5/Workspace.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/www/manager5/Workspace.js b/www/manager5/Workspace.js index 4e9e2e0..d4e0156 100644 --- a/www/manager5/Workspace.js +++ b/www/manager5/Workspace.js @@ -260,9 +260,9 @@

[pve-devel] [PATCH 2/2] ext5migrate: remove Ext.grid.feature.Chunking hack

2015-05-12 Thread Emmanuel Kasper
With ExtJS 4, we introduced an override to Ext.grid.feature.Chunking to fix scrollings problems in the grid when having a high number of nodes. Ext.grid.feature.Chunking has been removed from ExtJS in version 5, so we hope either the problem is fixed on ExtJS side, or we will have to find a

[pve-devel] [PATCH 1/2] Add required url routing for ext5 compatible main grid

2015-05-12 Thread Emmanuel Kasper
--- PVE/ExtJSIndex5.pm | 4 1 file changed, 4 insertions(+) diff --git a/PVE/ExtJSIndex5.pm b/PVE/ExtJSIndex5.pm index 1fd6dbc..a3da17b 100644 --- a/PVE/ExtJSIndex5.pm +++ b/PVE/ExtJSIndex5.pm @@ -39,6 +39,10 @@ _EOD script type=text/javascript

Re: [pve-devel] [PATCH 1/3] Initial import of the pve manager with ext5 code path

2015-04-15 Thread Emmanuel Kasper
On 04/15/2015 08:02 AM, Dietmar Maurer wrote: This approach is not optimal, because the changelog does not mention that this is just a copy of the original file. We need to make that more clear. I see two way to do that: A.) Use special commit message 'copy file to manager5' (something like

Re: [pve-devel] Grub problems and ZFS

2015-05-21 Thread Emmanuel Kasper
Hi all, I might have found the cause for the sometimes failing grub install. As I updated the grub packages today I discovered very extended install time for the grub package which was related to the post-install script. The script ran for several minuttes and while running consuming a full

[pve-devel] [PATCH 2/2] Remove the queryForm parameter as it blocks the combobox list loading.

2015-05-21 Thread Emmanuel Kasper
For unknown reasons, setting queryForm: here prevents loading the option list in the combobox. However: the queryForm parameter only would only make sense if we also set 'editable' to 'true', since queryForm specifies how the drop down list should react to user input in the text field of the

[pve-devel] [PATCH 1/2] Disable setting 'triggerAction' locally as we use the default anyway.

2015-05-21 Thread Emmanuel Kasper
Ref: http://docs.sencha.com/extjs/5.1/5.1.0-apidocs/#!/api/Ext.form.field.ComboBox-cfg-triggerAction --- www/manager5/form/ViewSelector.js | 1 - 1 file changed, 1 deletion(-) diff --git a/www/manager5/form/ViewSelector.js b/www/manager5/form/ViewSelector.js index 5306c72..19f75f6 100644 ---

[pve-devel] Fix background reloading of log entries

2015-06-08 Thread Emmanuel Kasper
After adding the HTML5 doctype, the call to setStyle was not setting the css properties anymore. The alternative and more flexible appyStyles() do work though, so we now use that. A quick recursive grep shows that we were using setStyle() only in LogView.js

[pve-devel] [PATCH 2/2] Add descriptive header

2015-06-08 Thread Emmanuel Kasper
--- www/manager/panel/LogView.js | 5 + 1 file changed, 5 insertions(+) diff --git a/www/manager/panel/LogView.js b/www/manager/panel/LogView.js index 6e00a97..be63ca3 100644 --- a/www/manager/panel/LogView.js +++ b/www/manager/panel/LogView.js @@ -1,3 +1,8 @@ +/* + * Display log entries in

[pve-devel] [PATCH 1/2] Fix a bug where the autoloading of syslog entries was not working.

2015-06-08 Thread Emmanuel Kasper
It looks that after adding the !DOCTYPE html HTML5 doctype, setStyle() is not working properly with ExtJS4, although is is the recommended doctype. Hence we switch to applyStyles() instead of setStyle(). --- www/manager/panel/LogView.js | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-)

Re: [pve-devel] pve-manager: add IvyBridge cpu model

2015-06-03 Thread Emmanuel Kasper
Hello Alexandre On 06/03/2015 02:21 PM, Alexandre DerHumier wrote: It's already added to qemuserver for qemu 2.3 ___ Just to get the big picture, what does happen if I set my guest to this cpu model, and my host is an for instance Opteron ( so

Re: [pve-devel] Preview of the GUI with ExtJS5

2015-06-22 Thread Emmanuel Kasper
Hi As for the remove-button issue you mentioned a while ago, I had an idea which is probably a bit of a PITA to implement, but just to get it out in the open: what about moving the item related action buttons into the list's rows so that there's a remove/edit-button per element? (The row

[pve-devel] Preview of the GUI with ExtJS5

2015-06-22 Thread Emmanuel Kasper
Hi You might have noticed some patches moving around the list related to ExtJS5. Not so many components are working yet, but there are enough stuff which at least display so I can share a screenshot: http://i.imgur.com/y8uMXkx.png Things to notice: * fonts are a bit bigger, 13px instead of 12px.

Re: [pve-devel] RFC: qemu-server : add cloudinit support

2015-06-15 Thread Emmanuel Kasper
-) We don't really like the whole rebooting after configuration option. It could be an optional flag, but ideally the user only needs to boot once, and since the IP options are always available in the GUI it also wouldn't be harmful to always include the cloud-init drive. This also improves

[pve-devel] [PATCH 5/5] Add documentation header

2015-06-15 Thread Emmanuel Kasper
--- www/manager5/data/DiffStore.js | 16 +++- 1 file changed, 15 insertions(+), 1 deletion(-) diff --git a/www/manager5/data/DiffStore.js b/www/manager5/data/DiffStore.js index b118c59..66bcfed 100644 --- a/www/manager5/data/DiffStore.js +++ b/www/manager5/data/DiffStore.js @@ -1,5

[pve-devel] [PATCH 1/5] Add documentation header

2015-06-15 Thread Emmanuel Kasper
--- www/manager5/data/UpdateStore.js | 6 ++ 1 file changed, 6 insertions(+) diff --git a/www/manager5/data/UpdateStore.js b/www/manager5/data/UpdateStore.js index b927930..bf48611 100644 --- a/www/manager5/data/UpdateStore.js +++ b/www/manager5/data/UpdateStore.js @@ -1,3 +1,9 @@ +/*

[pve-devel] [PATCH 4/5] ext5migrate: bump date-time fields to 150px width

2015-06-15 Thread Emmanuel Kasper
ExtJS5 Neptune default theme use 13px font instead of 12px in ExtJS4 so we need to make room for it. --- www/manager5/dc/Log.js | 4 ++-- www/manager5/dc/Tasks.js | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/www/manager5/dc/Log.js b/www/manager5/dc/Log.js index

[pve-devel] Adapt Tasks and Cluster log tabs for ExtJS

2015-06-15 Thread Emmanuel Kasper
Hello This patch serie allows to load Tasks and Cluser log tabs with ExtJS. ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] [PATCH 2/5] ext5migrate: replace calls to me.model.prototype.fields.each() and Ext.ModelMgr.create

2015-06-15 Thread Emmanuel Kasper
on ExtJS4, me.model.prototype.fields is a config object containing all the fields of the model, this object has properties like length, keys on ExtJS 5, me.model.prototype.fields is an array of config object that we have to iterate to get the field name Ext.ModelMgr.create() is non-working in

[pve-devel] [PATCH 3/5] ext5migrate: use 'refresh' event to reload components using DiffStore

2015-06-15 Thread Emmanuel Kasper
'datachanged' event was not reloading the store with ExtJS5, but 'refresh' does. According to the API description 'refresh' seems to be what we need: http://docs.sencha.com/extjs/5.1/5.1.0-apidocs/#!/api/Ext.data.AbstractStore-event-refresh --- www/manager5/data/DiffStore.js | 2 +- 1 file

[pve-devel] [PATCH] Fix qemu create wizard when iothread is set

2015-06-17 Thread Emmanuel Kasper
The variable to use to check the disk type is confid, as set in line 16 with: var confid = me.confid || (values.controller + values.deviceid); --- www/manager/qemu/HDEdit.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/www/manager/qemu/HDEdit.js b/www/manager/qemu/HDEdit.js

Re: [pve-devel] ceph packages for jessie

2015-06-16 Thread Emmanuel Kasper
Also,currently, debian jessie packages are firefly 0.80.7-2. But ceph repo (wheezy packages) are already at 0.80.9. BTW, I see that jessie-backports have 0.80.9-2~bpo8+1 rmadison ceph | grep jessie-backports ceph | 0.80.9-2~bpo8+1 | jessie-backports | source, amd64, arm64, armel, armhf,

Re: [pve-devel] Grub problems and ZFS

2015-05-27 Thread Emmanuel Kasper
On 05/23/2015 09:19 AM, Dietmar Maurer wrote: As a side note os-prober is completely useless on a bare-metal installer and I think installing proxmox on top of Debian should remove this package too? I remove those 'recommend' line from our grub packages, but someone needs to report that to

Re: [pve-devel] should we use systemd journal for logging

2015-06-01 Thread Emmanuel Kasper
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=717388 So the question is if we can/should remove syslog and enable persistent systemd journal instead? Any opinions on that? My two cents: I had a look at how Centos 7 handles the issue, and they treat it the same way as Debian

Re: [pve-devel] Grub problems and ZFS

2015-05-28 Thread Emmanuel Kasper
On 05/27/2015 12:15 PM, Dietmar Maurer wrote: IMHO this way of bypassing os-prober is cleaner than adding a 'Conflict' in our zfs-grub package, since it minimizes the packages conflicts when adding our proxmox repo on top of debian's. [1]

[pve-devel] [PATCH 4/4] Add comment header for important classes, idea taken from Workspace.js

2015-05-28 Thread Emmanuel Kasper
--- www/manager5/dc/Config.js | 4 www/manager5/dc/Log.js| 4 www/manager5/dc/Tasks.js | 4 www/manager5/form/ViewSelector.js | 3 +++ www/manager5/panel/StatusPanel.js | 3 +++ www/manager5/tree/ResourceTree.js | 3 +++ 6 files changed, 21

[pve-devel] (no subject)

2015-05-28 Thread Emmanuel Kasper
This patch serie allows to load the bottom Statuspanel, and add some comment headers to the classes already added in manager5/ ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] [PATCH 2/4] Add necessary javascript includes to load Status Panel

2015-05-28 Thread Emmanuel Kasper
--- PVE/ExtJSIndex5.pm| 3 +++ www/manager5/Workspace.js | 2 +- 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/PVE/ExtJSIndex5.pm b/PVE/ExtJSIndex5.pm index 34952ac..0f1eb89 100644 --- a/PVE/ExtJSIndex5.pm +++ b/PVE/ExtJSIndex5.pm @@ -31,6 +31,7 @@ _EOD script

Re: [pve-devel] proxmox 4.0 : pve-manager : extjs5 or extjs4 ?

2015-06-02 Thread Emmanuel Kasper
On 06/02/2015 11:26 AM, Alexandre DERUMIER wrote: Does proxmox 4.0 target extjs5 for pve-manager ? I would like to add some features in pve-manager, but I don't known if can send patch for extjs4 or wait for extjs5 migration. Hello Alexandre You can send your patches as usual based on ext

[pve-devel] [PATCH 1/3] Include all the javascript files to HTML page header

2015-07-07 Thread Emmanuel Kasper
Based on the JSSRC variable of the www/manager Makefile. Files not yet commited to www/manager5 are commented out. --- PVE/ExtJSIndex5.pm | 198 ++--- 1 file changed, 187 insertions(+), 11 deletions(-) diff --git a/PVE/ExtJSIndex5.pm

[pve-devel] [PATCH 2/3] Add EXT5DEBUG header, comment function to allow the dc Config panel to load

2015-07-07 Thread Emmanuel Kasper
--- www/manager5/grid/SelectFeature.js | 23 --- 1 file changed, 12 insertions(+), 11 deletions(-) diff --git a/www/manager5/grid/SelectFeature.js b/www/manager5/grid/SelectFeature.js index f0d2acd..eb5e719 100644 --- a/www/manager5/grid/SelectFeature.js +++

[pve-devel] [PATCH 3/3] Enable loading of most of the datacenter config panel

2015-07-07 Thread Emmanuel Kasper
This patch does the following: * remove the EXT5DEBUG header we added previously * build the items[] array as a one-dimensional array, as PVE.panel.Config with ExtJS5 is not able to find the elements otherwise * rename the problematic panel tabs with TODO suffix, to be dealed later on ---

Re: [pve-devel] [PATCH] ext5migrate: handle the case when a component's data is in the config object

2015-08-03 Thread Emmanuel Kasper
On 08/03/2015 04:22 PM, Dietmar Maurer wrote: the data property is not available via me.data, but via me.config.data. So we need to handle this in the parent class. So why do we need both cases? + data : me.data || me.config.data Isn't it enough to do: + data :

[pve-devel] (no subject)

2015-08-03 Thread Emmanuel Kasper
So I tried to enhance this patch based on the mailing list suggestions. It might be a small thing, but if 1 user asks publicly for the signification of this field, probably there are 10 others behind who don't dare to ask ... ___ pve-devel mailing

[pve-devel] [PATCH v2] Usability: enhance password label in LXC wizzard

2015-08-03 Thread Emmanuel Kasper
Do not hardcode the name of the admin user, instead move it to a stub function which we can enhance when our templates provide the necessary meta information. --- www/manager/lxc/CreateWizard.js | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git

[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 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

[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

[pve-devel] [PATCH] Typo in create_rootfs() subroutine

2015-08-24 Thread Emmanuel Kasper
--- src/PVE/LXC/Create.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/PVE/LXC/Create.pm b/src/PVE/LXC/Create.pm index 9cb52e3..fa59fcb 100644 --- a/src/PVE/LXC/Create.pm +++ b/src/PVE/LXC/Create.pm @@ -184,7 +184,7 @@ sub create_rootfs { my $old_conf =

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

2015-08-19 Thread Emmanuel Kasper
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

[pve-devel] Improving the consistency of color schemes in PVE GUI

2015-07-28 Thread Emmanuel Kasper
Hello For the current 4.0 we have three different color codes to indicate that a KVM / LXC / physical node is nonrunning vs running: -- CURRENT STATUS Non running / Running LXCGrey / Red

[pve-devel] Regression with latest qemu and iothreads option ?

2015-07-28 Thread Emmanuel Kasper
Hi I am using kvm --version QEMU emulator version 2.3.92 I have noticed if you select the iothread option in the disk options of your KVM, KVM hangs in a very nasty way: the kvm processes starts, the qm manager reports the machine as running but actually it is not: it is not possible to attach

[pve-devel] References to the Java Applet console

2015-08-04 Thread Emmanuel Kasper
Hi I see we still have references to the java applet in Utils.js and VNConsole.js Should we keep them or do we want to remove them ? Emmanuel ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] [PATCH] remove debugging traces

2015-07-28 Thread Emmanuel Kasper
--- www/manager/dc/Summary.js | 3 --- www/manager5/dc/Summary.js | 3 --- 2 files changed, 6 deletions(-) diff --git a/www/manager/dc/Summary.js b/www/manager/dc/Summary.js index 9a210a3..b0f8b32 100644 --- a/www/manager/dc/Summary.js +++ b/www/manager/dc/Summary.js @@ -81,9 +81,6 @@

[pve-devel] [PATCH] Move subscription tab to the outmost right of the node tab panel

2015-07-29 Thread Emmanuel Kasper
This brings consistency with the datacenter tab panel. --- www/manager/node/Config.js | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/www/manager/node/Config.js b/www/manager/node/Config.js index ff6d225..61bb4b2 100644 --- a/www/manager/node/Config.js +++

[pve-devel] [PATCH] ext5migrate: pass configuration to KVCombBox via config object

2015-08-05 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

Re: [pve-devel] Regression with latest qemu and iothreads option ?

2015-08-03 Thread Emmanuel Kasper
Hi On 07/30/2015 09:03 AM, Alexandre DERUMIER wrote: This is fixed in qemu 2.4rc3 I confirm can our pve-qemu-kvm package in version 2.4-1 has this fixed. Thanks for the update ! Emmanuel ___ pve-devel mailing list pve-devel@pve.proxmox.com

[pve-devel] [PATCH] ext5migrate: handle the case when a component's data is in the config object

2015-08-03 Thread Emmanuel Kasper
In ExtJS5 when we use the following call (taken from OptionView.js) Ext.applyIf(me, { items: { xtype: 'pveKVComboBox', data: data, } }); the data property is not available via me.data, but via me.config.data. So we need to

[pve-devel] [PATCH] Usability: enhance password label in LXC wizzard

2015-07-30 Thread Emmanuel Kasper
It seems according to http://pve.proxmox.com/pipermail/pve-user/2015-July/008954.html some users get confused about this field. --- www/manager/lxc/CreateWizard.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/www/manager/lxc/CreateWizard.js

[pve-devel] [PATCH] ext5migrate: set buttons property directly instead of calling applyIf

2015-07-29 Thread Emmanuel Kasper
It seems that in ExtJS5 the prototype of the Window component already sets some default values to me.buttons and me.layout, hence calling applyIf on these properties will silently fail. me.layout is already set to 'auto' by the framework, we don't need to set that anymore. ---

Re: [pve-devel] updated PVE 4.0 packages on pvetest

2015-08-04 Thread Emmanuel Kasper
On 08/04/2015 04:35 PM, Alexandre DERUMIER wrote: Maybe qemu-2.4 has introduced something which makes it incompatible with previous versions of qemu? Yes, maybe. do you have tried qemu 2.4 on top of 2.4 ? I tried nested KVM virtualization with qemu 2.4, and it works here(TM) host

[pve-devel] [PATCH 1/2] Add documentation header

2015-07-22 Thread Emmanuel Kasper
--- www/manager5/data/ObjectStore.js | 5 + 1 file changed, 5 insertions(+) diff --git a/www/manager5/data/ObjectStore.js b/www/manager5/data/ObjectStore.js index 03d0c7a..935f922 100644 --- a/www/manager5/data/ObjectStore.js +++ b/www/manager5/data/ObjectStore.js @@ -1,3 +1,8 @@ +/* This

[pve-devel] [PATCH 2/2] ext5migrate: replace call to private ExtJS function getRoot()

2015-07-22 Thread Emmanuel Kasper
getRoot() semantics changed with ExtJS5. To avoid further annoyances when ExtJS6 is around the corner, replace the call to this private method, set rootProperty in the ObjectStore call, and use the value of rootProperty to filter out the content of the server answer. ---

Re: [pve-devel] backup : add pigz compressor

2015-07-13 Thread Emmanuel Kasper
07/11/2015 09:25 AM, Cesar Peschiera wrote: Yes, that makes sense to me. Or maybe the PVE GUI also can has a third option for use pgzip, and the user also can select the amount of cores to used, so in this case, also maybe will be better add a message of caution that say the use of many

[pve-devel] [PATCH] Add new pvereport command

2015-10-22 Thread Emmanuel Kasper
--- bin/Makefile | 10 +-- bin/pvereport | 89 +++ 2 files changed, 97 insertions(+), 2 deletions(-) create mode 100755 bin/pvereport diff --git a/bin/Makefile b/bin/Makefile index 9cfeafe..827e30d 100644 --- a/bin/Makefile +++

[pve-devel] [PATCH trivial qemu-server] Don't create unused empty /etc/qemu-server directory

2015-10-27 Thread Emmanuel Kasper
--- Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/Makefile b/Makefile index 21324f1..e47f5fb 100644 --- a/Makefile +++ b/Makefile @@ -79,7 +79,6 @@ PKGSOURCES=qm qm.1.gz qm.1.pod qmrestore qmrestore.1.pod qmrestore.1.gz qmextrac .PHONY: install install: ${PKGSOURCES}

[pve-devel] empty /etc/qemu-server/ directory

2015-10-27 Thread Emmanuel Kasper
Hi I noticed while poking aroung /etc, that the qemu-server package creates an empty /etc/qemu-server directory but never uses it, since we always store stuff in /etc/pve/qemu-server I assume it is safe to remove it ? ___ pve-devel mailing list

Re: [pve-devel] [PATCH] Add new pvereport command

2015-10-27 Thread Emmanuel Kasper
On 10/27/2015 07:43 AM, Dietmar Maurer wrote: > applied, comment inline: > >> --- /dev/null >> +++ b/bin/pvereport >> @@ -0,0 +1,89 @@ >> +#!/usr/bin/perl >> + >> +use strict; >> +use warnings; >> +use PVE::pvecfg; >> + >> +($> == 0 ) || die "please run as root\n"; >> + >> +my @general =

[pve-devel] [PATCH v3 pve-manager] Pretty format the output of multiple files in a directory

2015-10-29 Thread Emmanuel Kasper
Also, don't "cat" non existing files --- bin/pvereport | 42 ++ 1 file changed, 30 insertions(+), 12 deletions(-) diff --git a/bin/pvereport b/bin/pvereport index 514cc9a..0e2fb3d 100755 --- a/bin/pvereport +++ b/bin/pvereport @@ -3,19 +3,31 @@ use

[pve-devel] enhance pvereport display

2015-10-29 Thread Emmanuel Kasper
Changed from V1: use PVE::Tools::dir_glob_foreach Changed from V2: add regexp for openvz vm.conf files ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] [PATCH v2 pve-manager] Pretty format the output of multiple files in a directory

2015-10-29 Thread Emmanuel Kasper
Also, don't "cat" non existing files --- bin/pvereport | 42 ++ 1 file changed, 30 insertions(+), 12 deletions(-) diff --git a/bin/pvereport b/bin/pvereport index 514cc9a..42e87cc 100755 --- a/bin/pvereport +++ b/bin/pvereport @@ -3,19 +3,31 @@ use

[pve-devel] [PATCH pve-manager] Pretty format the output of multiple files in a directory

2015-10-28 Thread Emmanuel Kasper
Also, don't "cat" non existing files --- bin/pvereport | 50 ++ 1 file changed, 38 insertions(+), 12 deletions(-) diff --git a/bin/pvereport b/bin/pvereport index 514cc9a..91fdb36 100755 --- a/bin/pvereport +++ b/bin/pvereport @@ -3,19 +3,39 @@

[pve-devel] Initial work for ExtJS6

2015-11-12 Thread Emmanuel Kasper
This patch serie changes allows the GUI to at least load with Ext6 ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] [PATCH pve-manager 1/3] Add mime type for woff2 fonts, also update mime type for ttf fonts

2015-11-12 Thread Emmanuel Kasper
application/font-woff2 is still in discussion but works in main three browsers This is needed for ExtJS6, which includes some woff2 fonts ttf font mime type is taken from the official IANA assignment, and works as well in main three browsers --- PVE/HTTPServer.pm | 5 - 1 file changed, 4

[pve-devel] [PATCH pve-manager 3/3] ext6migrate: Replace deprecated function which is missing in Ext6

2015-11-12 Thread Emmanuel Kasper
See http://docs.sencha.com/extjs/5.1/5.1.0-apidocs/#!/api/Ext.container.Container-method-doLayout This allows the pve manager to load --- www/manager5/Workspace.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/www/manager5/Workspace.js b/www/manager5/Workspace.js

[pve-devel] [PATCH pve-manager 2/3] Switch to triton theme as default theme in ExtJS6

2015-11-12 Thread Emmanuel Kasper
--- PVE/ExtJSIndex6.pm | 2 +- www/ext6/Makefile | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/PVE/ExtJSIndex6.pm b/PVE/ExtJSIndex6.pm index f7449c8..0604fc2 100644 --- a/PVE/ExtJSIndex6.pm +++ b/PVE/ExtJSIndex6.pm @@ -12,7 +12,7 @@ sub get_index { Proxmox

[pve-devel] [PATCH pve-manager 3/3] ext6migrate: move class static properties out of initComponent()

2015-11-16 Thread Emmanuel Kasper
This fixes the Auth realm selection while login with ExtJS6 --- www/manager6/form/RealmComboBox.js | 55 +- 1 file changed, 25 insertions(+), 30 deletions(-) diff --git a/www/manager6/form/RealmComboBox.js b/www/manager6/form/RealmComboBox.js index

[pve-devel] [PATCH pve-manager 1/3] ext6migrate: force an empty array as default value for filters

2015-11-16 Thread Emmanuel Kasper
Without this, filters.add() in line 73 file fails --- www/manager6/grid/ObjectGrid.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/www/manager6/grid/ObjectGrid.js b/www/manager6/grid/ObjectGrid.js index cd135a1..8513709 100644 --- a/www/manager6/grid/ObjectGrid.js +++

[pve-devel] [PATCH pve-manager 2/3] ext6migrate: move static class properties out of initComponent()

2015-11-16 Thread Emmanuel Kasper
Besides fitting more with the declarative style of ExtJS, this has the interesting side effect of allowing comboboxes to work with ExtJS6 --- www/manager6/form/KVComboBox.js | 10 -- 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/www/manager6/form/KVComboBox.js

Re: [pve-devel] [Feature request] add disk for lxc from the API and WebUI

2015-11-12 Thread Emmanuel Kasper
On 11/12/2015 04:01 PM, lyt_yudi wrote: > hi,all > > current, add disk for the lxc from cli. like this > > # cat /etc/pve/lxc/100.conf > arch: amd64 > cpulimit: 2 > cpuunits: 1024 > hostname: test > memory: 2048 > net0: >

  1   2   3   4   5   6   7   8   9   >