[pve-devel] [PATCH manager 1/9] Better naming for kv1 comb box, document the two private functions

2017-01-12 Thread Emmanuel Kasper
--- www/manager6/qemu/Clone.js | 17 - 1 file changed, 12 insertions(+), 5 deletions(-) diff --git a/www/manager6/qemu/Clone.js b/www/manager6/qemu/Clone.js index 9d6fdae..3296f5e 100644 --- a/www/manager6/qemu/Clone.js +++ b/www/manager6/qemu/Clone.js @@ -53,10 +53,14 @@ Ext.defi

[pve-devel] [PATCH manager 2/9] Use correct parameter for GuestIDSelector

2017-01-12 Thread Emmanuel Kasper
--- www/manager6/qemu/Clone.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/www/manager6/qemu/Clone.js b/www/manager6/qemu/Clone.js index 3296f5e..2843921 100644 --- a/www/manager6/qemu/Clone.js +++ b/www/manager6/qemu/Clone.js @@ -191,7 +191,7 @@ Ext.define('PVE.window.Clon

[pve-devel] [PATCH manager 5/9] Group the Pool selector combobox to the VM properties

2017-01-12 Thread Emmanuel Kasper
The pool membership relates here to the VM membership, not the storage membership, so it makes sense to move it closer to the VM ID / VM name form fields. --- www/manager6/qemu/Clone.js | 15 +++ 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/www/manager6/qemu/Clone.js b

[pve-devel] [PATCH manager 9/9] Use component references via lookupReference()

2017-01-12 Thread Emmanuel Kasper
This allows to add the components in each column in the order they appear in the Clone Window. References from child components are kept in an object in the parent component so it should be the same performance-wise (contrary to to Ext.ComponentQuery which is doing DOM parsing) --- www/manager6/q

[pve-devel] [PATCH manager 7/9] Do not show a Clone Mode selection Window for non-templates clones

2017-01-12 Thread Emmanuel Kasper
As Non-templates clones are always a full clone. --- www/manager6/qemu/Clone.js | 1 + 1 file changed, 1 insertion(+) diff --git a/www/manager6/qemu/Clone.js b/www/manager6/qemu/Clone.js index 3b3577e..af088b2 100644 --- a/www/manager6/qemu/Clone.js +++ b/www/manager6/qemu/Clone.js @@ -169,6 +169

[pve-devel] [PATCH manager 6/9] Autoselect a storage on VM Clone.

2017-01-12 Thread Emmanuel Kasper
--- www/manager6/qemu/Clone.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/www/manager6/qemu/Clone.js b/www/manager6/qemu/Clone.js index 966bd06..3b3577e 100644 --- a/www/manager6/qemu/Clone.js +++ b/www/manager6/qemu/Clone.js @@ -228,7 +228,7 @@ Ext.define('PVE.window.Clon

[pve-devel] [PATCH manager 4/9] Make the Snapshot selector an optional parameter in the Clone Window

2017-01-12 Thread Emmanuel Kasper
The snapshot selector was known to confuse first time pve users. Some wondered that they had no snapshot called 'current' in their snapshot trees, and other thought that snapshots would be included in the copied image. So before displaying the Clone Window, do an API call to verify if the source V

[pve-devel] [PATCH manager 0/9] Usability Improvements and minor refactoring for the Clone Window

2017-01-12 Thread Emmanuel Kasper
target storage, just like in the Wizards Emmanuel Kasper (9): Better naming for kv1 comb box, document the two private functions Use correct parameter for GuestIDSelector Disable the submit button based on whole form validity Make the Snapshot selector an optional parameter in the Clone

[pve-devel] [PATCH manager v3] Update French translation

2017-01-12 Thread Emmanuel Kasper
n 11 10:58:53 2017\n" -"PO-Revision-Date: 2016-04-07 13:14+0100\n" +"PO-Revision-Date: 2017-01-12 09:40+0100\n" "Last-Translator: Emmanuel Kasper \n" "Language-Team: French\n" "Language: fr\n" @@ -30,7 +30,7 @@ msgstr "/chemin/repertoire

[pve-devel] [PATCH manager v2] Use component references via lookupReference()

2017-01-12 Thread Emmanuel Kasper
This allows to add the components in each column in the order they appear in the Clone Window. References from child components are kept in an object in the parent component so it should be the same performance-wise (contrary to to Ext.ComponentQuery which is doing DOM parsing) --- changes since v

[pve-devel] [PATCH manager v3] Use component references via lookupReference()

2017-01-12 Thread Emmanuel Kasper
This allows to add the components in each column in the order they appear in the Clone Window. References from child components are kept in an object in the parent component so it should be the same performance-wise (contrary to to Ext.ComponentQuery which is doing DOM parsing) --- changes: * cor

[pve-devel] [PATCH manager] Disable autoSelect of target storage

2017-01-12 Thread Emmanuel Kasper
autoSelect: true was surreptitiously commited with the refactoring of 3656c0b9f3742f952b1684f3b6ea9f519d8cd5bd ( Use component references via lookupReference() ) --- www/manager6/qemu/Clone.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/www/manager6/qemu/Clone.js b/www/man

[pve-devel] [PATCH manager 2/3] Do not try to translate domain specific terms

2017-01-18 Thread Emmanuel Kasper
No one is translating DHCP in his/her own language. --- www/manager6/ceph/Config.js | 2 +- www/manager6/ceph/Pool.js | 2 +- www/manager6/ceph/Status.js | 2 +- www/manager6/dc/Health.js | 2 +- www/manager6/lxc/Network.js | 18 +- 5 files changed, 13 insertions(+), 13 del

[pve-devel] [PATCH manager 1/3] Remove duplicate strings

2017-01-18 Thread Emmanuel Kasper
--- www/manager6/Utils.js| 2 +- www/manager6/node/Config.js | 2 +- www/manager6/qemu/Options.js | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/www/manager6/Utils.js b/www/manager6/Utils.js index 180b283..40d2fe1 100644 --- a/www/manager6/Utils.js +++ b/www/manage

[pve-devel] [PATCH manager 3/3] Remove redundant helper for login window

2017-01-18 Thread Emmanuel Kasper
the information is already provided by the label, the helper does not add meaningfull information here. --- www/manager6/ceph/Config.js| 2 +- www/manager6/window/LoginWindow.js | 6 ++ 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/www/manager6/ceph/Config.js b/www/man

[pve-devel] [PATCH manager 0/3] Reduce the number of required translations

2017-01-18 Thread Emmanuel Kasper
A side note: if you use an english technical term in your own language (DHCP, DNS) it probably does not need a translation. Emmanuel Kasper (3): Remove duplicate strings Do not try to translate domain specific terms Remove redundant helper for login window www/manager6/Utils.js

Re: [pve-devel] Question: open virtual appliance

2017-01-18 Thread Emmanuel Kasper
On 01/16/2017 10:14 PM, Gustaf Ankarloo wrote: > Are there any plans of making a import function for OVA/OVF ? > I know it's doable with a lot of manual tinkering I have this somewhere in my todo list although it is not high priority at the momment. I would like to try to following approach: * e

[pve-devel] [PATCH manager] Reduce the number of translations needed

2017-01-18 Thread Emmanuel Kasper
Mostly split out technical term who don't need to be translated outside of the gettext function paramater. --- www/manager6/ceph/Status.js | 4 ++-- www/manager6/dc/Health.js| 2 +- www/manager6/dc/Summary.js | 2 +- www/manager6/grid/FirewallOptions.js | 10 +

[pve-devel] [PATCH manager v2 2/2] Split the Enable + name construct in two separate gettext strings

2017-01-18 Thread Emmanuel Kasper
This reduce the number of strings to translate. --- www/manager6/grid/FirewallOptions.js | 10 +- www/manager6/lxc/Options.js | 6 +++--- www/manager6/lxc/ResourceEdit.js | 2 +- www/manager6/qemu/ProcessorEdit.js | 2 +- 4 files changed, 10 insertions(+), 10 deletions(-)

[pve-devel] [PATCH manager v2 1/2] Remove technical terms from content to be translated

2017-01-18 Thread Emmanuel Kasper
--- www/manager6/ceph/Status.js | 4 ++-- www/manager6/dc/Health.js | 2 +- www/manager6/dc/Summary.js | 2 +- www/manager6/ha/ResourceEdit.js | 3 ++- 4 files changed, 6 insertions(+), 5 deletions(-) diff --git a/www/manager6/ceph/Status.js b/www/manager6/ceph/Status.js index ff71

Re: [pve-devel] Question: open virtual appliance

2017-01-19 Thread Emmanuel Kasper
On 01/19/2017 07:46 AM, Alexandre DERUMIER wrote: > Hi, > > as a first step, I have send patch to import external disk image. > > now for ova import, > I think we need a method to extract files from ova (it's a simple tar file), > > then an xml parser to parse OVF descriptor > > Does somebody

[pve-devel] [PATCH manager 2/2] Use the enabled/disabled toggler to render grid options

2017-01-23 Thread Emmanuel Kasper
Allows getting rid of multiple "Enable foobar" gettext strings in LXC and Firewall option panels. --- www/manager6/grid/FirewallOptions.js | 16 www/manager6/lxc/Options.js | 8 2 files changed, 12 insertions(+), 12 deletions(-) diff --git a/www/manager6/grid/F

[pve-devel] [PATCH manager 1/2] Add new renderer function for toggling enabled/disabled options

2017-01-23 Thread Emmanuel Kasper
--- www/manager6/Utils.js | 6 ++ 1 file changed, 6 insertions(+) diff --git a/www/manager6/Utils.js b/www/manager6/Utils.js index 40d2fe1..c05a137 100644 --- a/www/manager6/Utils.js +++ b/www/manager6/Utils.js @@ -745,6 +745,8 @@ Ext.define('PVE.Utils', { utilities: { yesText: gettext

[pve-devel] [PATCH manager] Fix #1266: Convert Unix timestamps to milliseconds for JS Date() constructor

2017-01-31 Thread Emmanuel Kasper
While we're at it, use Ext.Date.format to localize the displayed dates. We use long day names and long month name on purpose since the date is displayed read only in a large window, and not sortable. Output before the patch: Sun Jan 18 1970 05:37:06 GMT+0100 (CET) Output after: Donnerstag 26 Janu

[pve-devel] [PATCH manager v2 1/2] Add a new renderer for displaying dates in a human readable way

2017-01-31 Thread Emmanuel Kasper
render_timestamp returns a Date as "2017-01-31 16:17:33" which is good for computers and column sorting but unfit for human consumption the new renderer added by this patch renders a Date as "Tuesday 31 January 2017 16:18:08" Day-month-year order is the most used ordering in the world and also co

[pve-devel] [PATCH manager v2 2/2] Fix #1266: use a renderer when displaying Unix timestamps.

2017-01-31 Thread Emmanuel Kasper
The timestamp is displayed read only in a large window, and not sortable, hence we use the human readable renderer. --- www/manager6/qemu/Snapshot.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/www/manager6/qemu/Snapshot.js b/www/manager6/qemu/Snapshot.js index d438d68..3

[pve-devel] [PATCH manager] Use ip from iproute2 package instead of net-tools

2017-02-06 Thread Emmanuel Kasper
iproute2 is a direct depency from pve-qemu-kvm and should be always available use -statistics so we get the RX/TX traffic sum like ifconfig --- PVE/Report.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/PVE/Report.pm b/PVE/Report.pm index 901fcf8..d32b968 100644 --- a/PVE/R

Re: [pve-devel] [PATCH manager] Use ip from iproute2 package instead of net-tools

2017-02-06 Thread Emmanuel Kasper
On 02/06/2017 03:02 PM, Wolfgang Bumiller wrote: > On Mon, Feb 06, 2017 at 02:37:16PM +0100, Emmanuel Kasper wrote: >> iproute2 is a direct depency from pve-qemu-kvm and should >> be always available >> >> use -statistics so we get the RX/TX traffic sum like ifconfig &

[pve-devel] [PATCH manager v2] Use ip from iproute2 package instead of net-tools

2017-02-06 Thread Emmanuel Kasper
iproute2 is a direct depency from pve-qemu-kvm and should be always available use -statistics so we get the RX/TX traffic sum like ifconfig use -details to get extra info like current vlan_protocol --- changes since v1: * add -details as suggested by w.bumiller PVE/Report.pm | 2 +- 1 file chang

Re: [pve-devel] LXC with 2 and more NICs

2017-02-06 Thread Emmanuel Kasper
On 02/02/2017 10:03 PM, Detlef Bracker wrote: > Dear, > > I thing so, thats a bug! > > A ping from outside to the LXC-containers to all NICs works fine! > > A ping from console via the NICs 2- is not possible! So, this can > been a big problem, when a daemon will send from the NICs 2- >

[pve-devel] [PATCH manager 1/3] Fix handling of failed API call verifyFeature()

2017-02-06 Thread Emmanuel Kasper
This call can for instance fails when a source VM has references to non existing storages. --- www/manager6/qemu/Clone.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/www/manager6/qemu/Clone.js b/www/manager6/qemu/Clone.js index ce6e4f3..8a3b34a 100644 --- a/www/manager6/qem

[pve-devel] [PATCH manager 0/3] Add a Clone button to the VM toolbar

2017-02-06 Thread Emmanuel Kasper
Cloning a VM was up to now only accessible via right clicking the resource tree, a path that many novice PVE users did not notice. Adding this is as a general button will make the feature easier to find. Emmanuel Kasper (3): Fix handling of failed API call verifyFeature() Move the snapshot

[pve-devel] [PATCH manager 3/3] Add a clone button in the VM toolbar

2017-02-06 Thread Emmanuel Kasper
Up to now the Migrate function had a first class button in the toolbar, but the Clone function didn't. With this button the toolbar still fit correctly in a smaller 1280 pixel wide display. --- www/manager6/qemu/Config.js | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --gi

[pve-devel] [PATCH manager 2/3] Move the snapshot API test to a static function

2017-02-06 Thread Emmanuel Kasper
This will allow code reuse in a following patch. No functional change. --- www/manager6/qemu/Clone.js | 25 +++-- www/manager6/qemu/CmdMenu.js | 18 +- 2 files changed, 24 insertions(+), 19 deletions(-) diff --git a/www/manager6/qemu/Clone.js b/www/manager6/

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

[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 --- www/ext4/Makefile | 1 + 1 file changed, 1 insertion

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

[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 --- defines.mk | 1 + www/Makefile

[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

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

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

2015-04-02 Thread Emmanuel Kasper
Signed-off-by: Emmanuel Kasper --- 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

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

2015-04-02 Thread Emmanuel Kasper
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 --- PVE/ExtJSIndex.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --

[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 --- PVE/Makefile | 1 + bin/pveproxy | 19 --- 2 files changed, 13

[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 --- 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..ae6b8f9 --- /dev

[pve-devel] [PATCH] Add missing ${DESTDIR} to include the jslint script in the debian package.

2015-04-09 Thread Emmanuel Kasper
Our Makefile was missing the DESTDIR path variable, so /usr/bin/jslint was copied directly from our source dir, instead of going through the debian packaging step. Signed-off-by: Emmanuel Kasper --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b

[pve-devel] ExtJS 5 migration path

2015-04-14 Thread Emmanuel Kasper
Hello ! I am involved at the moment in the migration of our Javascript code base from Ext Js 4 to Ext JS 5, and I would like to start it doing with the following approach: * create a manager5/ directory * copy incrementally in manager5/ the javascript files forming our current pvemanagerli

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

2015-04-14 Thread Emmanuel Kasper
is is only meant for Ext 5 development Signed-off-by: Emmanuel Kasper --- 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, '/

[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 --- PVE/ExtJSIndex5.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/PVE/ExtJSIndex

[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 --- www/manager5/Utils.js

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 th

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

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

[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 + + + + _EOD -- 2.1.4 ___

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, > wh

[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 + + + + _EOD -- 2.1.4 ___

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

[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 --- a/www/manager5/grid/Resource

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

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

2015-05-18 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 b/www/manager5/W

[pve-devel] [PATCH 2/3] ext5migrate: fix display of PVE version underneath product name

2015-05-18 Thread Emmanuel Kasper
With the alignment set to 'middle' in the north region, the 'versioninfo' component was wrongly pushed downwards, hiding the version string. Besides that we need to manually set a 'height' argument to 'versioninfo' equal to the size of the north region, to prevent extjs to generate a problematic pa

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

2015-05-18 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 @@ Ext.define('PVE.StdWorkspa

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

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

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

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

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] https://bugs.debian.org/cgi-bin/bugreport.c

[pve-devel] [PATCH 3/4] Temporary disable loading the tabs of the main Config panel

2015-05-28 Thread Emmanuel Kasper
--- www/manager5/dc/Config.js | 2 ++ 1 file changed, 2 insertions(+) diff --git a/www/manager5/dc/Config.js b/www/manager5/dc/Config.js index 6c19805..aa9c0f8 100644 --- a/www/manager5/dc/Config.js +++ b/www/manager5/dc/Config.js @@ -100,6 +100,8 @@ Ext.define('PVE.dc.Config', {

[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 insertions(+)

[pve-devel] [PATCH 1/4] Remove unused misspelled parameter

2015-05-28 Thread Emmanuel Kasper
--- www/manager/data/ResourceStore.js | 1 - www/manager/form/NodeSelector.js | 1 - www/manager/form/RealmComboBox.js | 1 - www/manager/form/ViewSelector.js | 1 - www/manager/node/TimeEdit.js | 2 +- www/manager5/data/ResourceStore.js | 1 - www/manager5/form/RealmComboBox.js | 1 -

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

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 (jo

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

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 th

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

[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 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(-) diff --git a/w

Re: [pve-devel] qemu 2.3 : format=raw need to be explicitly defined for some storages

2015-06-09 Thread Emmanuel Kasper
On 06/09/2015 02:49 PM, Alexandre DERUMIER wrote: > Hi, > > I was testing qemu 2.3 with librbd, and I notice a warning > > WARNING: Image format was not specified for > 'rbd:pooltest/vm-100-disk-1:mon_host=10.5.0.16:auth_supported=none' and > probing guessed raw. > Automatically detect

Re: [pve-devel] feature request : pve-manager tasks grid : keep running tasks on top ?

2015-06-10 Thread Emmanuel Kasper
Hello Stefan On 06/09/2015 09:07 PM, Stefan Priebe wrote: > > Am 09.06.2015 um 20:15 schrieb Dietmar Maurer: >>> Sometimes some long running tasks (move disk for example), can take >>> hours, >>> and are lost inside the other logs, need to scroll to find them,... >> >> I guess this is a bug then.

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 impr

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

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

[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 @@ +/* Extends

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

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

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

[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

[pve-devel] [PATCH] Force Internet Explorer to interpret pages in IE10 compatibility modus

2015-06-18 Thread Emmanuel Kasper
IE11 is not supported with ExtJS 4, so we have to force the IE rendering engine to parse the page as an older version. This fixes the problem of the component tree icons not appearing. IE11 is supported in ExtJS so this header is not needed in manager5/ (tested OK) Closes https://bugzilla.proxmox.

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

[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 +++ b/www/manager5/gr

[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 b/PVE/ExtJSInde

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

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 man

<    2   3   4   5   6   7   8   9   10   >