[pve-devel] [PATCH pve-manager v2] Remove swap/memory limit in the Manager, fixes #1799

[Dominik Csapak]

looks mostly ok one comment inline

On 06/07/2018 01:40 PM, René Jochum wrote:
> ---
>   www/manager6/lxc/ResourceEdit.js  | 2 --
>   www/manager6/qemu/HardwareView.js | 4 ++--
>   www/manager6/qemu/MemoryEdit.js   | 2 --
>   3 files changed, 2 insertions(+), 6 deletions(-)
> 
> diff --git a/www/manager6/lxc/ResourceEdit.js b/www/manager6/lxc/ResourceEdit.js
> index ec697cac..395fab7e 100644
> --- a/www/manager6/lxc/ResourceEdit.js
> +++ b/www/manager6/lxc/ResourceEdit.js
> @@ -118,7 +118,6 @@ Ext.define('PVE.lxc.MemoryInputPanel', {
>   		xtype: 'proxmoxintegerfield',
>   		name: 'memory',
>   		minValue: 16,
> -		maxValue: 512*1024,
>   		value: '512',
>   		step: 32,
>   		fieldLabel: gettext('Memory') + ' (MiB)',
> @@ -129,7 +128,6 @@ Ext.define('PVE.lxc.MemoryInputPanel', {
>   		xtype: 'proxmoxintegerfield',
>   		name: 'swap',
>   		minValue: 0,
> -		maxValue: 128*1024,
>   		value: '512',
>   		step: 32,
>   		fieldLabel: gettext('Swap') + ' (MiB)',
> diff --git a/www/manager6/qemu/HardwareView.js b/www/manager6/qemu/HardwareView.js
> index a87a9df1..44153f7a 100644
> --- a/www/manager6/qemu/HardwareView.js
> +++ b/www/manager6/qemu/HardwareView.js
> @@ -54,11 +54,11 @@ Ext.define('PVE.qemu.HardwareView', {
>   		renderer: function(value, metaData, record, ri, ci, store, pending) {
>   		    var res = '';
>   
> -		    var max = me.getObjectValue('memory', 512, pending);
> +		    var value = me.getObjectValue('memory', undefined, pending);
>   		    var balloon =  me.getObjectValue('balloon', undefined, pending);
>   		    var shares = me.getObjectValue('shares', undefined, pending);
>   
> -		    res  = Proxmox.Utils.format_size(max*1024*1024);
> +		    res  = Proxmox.Utils.format_size(value*1024*1024);

this is wrong

the second parameter in getObjectValue is the default not the maximum

else the patch looks good

>   
>   		    if (balloon !== undefined && balloon > 0) {
>   			res = Proxmox.Utils.format_size(balloon*1024*1024) + "/" + res;
> diff --git a/www/manager6/qemu/MemoryEdit.js b/www/manager6/qemu/MemoryEdit.js
> index 384351b0..4c4d1815 100644
> --- a/www/manager6/qemu/MemoryEdit.js
> +++ b/www/manager6/qemu/MemoryEdit.js
> @@ -39,7 +39,6 @@ Ext.define('PVE.qemu.MemoryInputPanel', {
>   		fieldLabel: gettext('Memory') + ' (MiB)',
>   		name: 'memory',
>   		minValue: 1,
> -		maxValue: 512*1024,
>   		step: 32,
>   		hotplug: me.hotplug,
>   		listeners: {
> @@ -61,7 +60,6 @@ Ext.define('PVE.qemu.MemoryInputPanel', {
>   		xtype: 'pveMemoryField',
>   		name: 'balloon',
>   		minValue: 1,
> -		maxValue: 512*1024,
>   		step: 32,
>   		fieldLabel: gettext('Minimum memory') + ' (MiB)',
>   		hotplug: me.hotplug,
>