This fixes the bug: https://bugzilla.proxmox.com/show_bug.cgi?id=677
Remarks:
* allowBlank is already set in the class definition
* The value of the comboBox can be either '' after the component initial
load, or 'null' if the user selected a macro and cleans the field afterwards.
This is the reason of the if (!value) test which will catch empty strings and
null.
We're replacing the use of 'find' in the backup code for
various reasons, so we cannot use find-specific options
anymore. Besides, it's up to the plugin to create the
archive and there's no guarantee using 'find' even makes
sense. Even in the current code it's not being used when
hitting the rsync case anyway.
Now using simple glob patterns as that is more widely
supported.
Wolfgang Link [Mon, 7 Sep 2015 10:48:31 +0000 (12:48 +0200)]
fix output percent_used in rbd pools
This fix is essential of cause json field changes in the ceph api.
They change the field size in pool to max_avail
and the field total_size in stats to total_max_avail
These added dependencies let you execute `make install` repeatedly,
whitout using `make clean` in between.
Without it make autodeleted the %.1.pod and %.8.pod files as the
install target doesn't declare a dependency although it needs them.
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Thomas Lamprecht [Thu, 27 Aug 2015 10:13:33 +0000 (12:13 +0200)]
set size in the rootfs parameter on CT create
As the size parameter is now deprecated and was replaced with the
new rootfs parameter, use that one when creating an container.
So we don't get an 'parameter verification failed' error.
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Emmanuel Kasper [Tue, 25 Aug 2015 08:21:49 +0000 (10:21 +0200)]
Pass the list of of combobox items as a property
The original fix using a config Object was a bit overkill, this works well too , requires less code in the child classes, and is more consistent with rest of the code we have.
Dietmar Maurer [Tue, 25 Aug 2015 05:57:32 +0000 (07:57 +0200)]
fix bug #690: workaround - list all storages
It is currently unclear howto restrict storages for container usage.
The 'rootdir' property does not work anymore, because we can now create
containers on normal VM image files.
Emmanuel Kasper [Tue, 18 Aug 2015 12:43:33 +0000 (14:43 +0200)]
ext5migrate: move setting the Panel title into the class properties
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.
Emmanuel Kasper [Tue, 18 Aug 2015 08:36:57 +0000 (10:36 +0200)]
ext5migrate: do not set a custome idProperty for the KeyValue model
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 change should be safe.
bridgevlanport: replace sed pipe with substitution
The previous code stripped *all* combinations of zero or
more letters and digits followed by a dot of an interface
name to retrieve the VLANID. At the same time the supposedly
opposite code for IF_VLAN_RAW_DEVICE only actually extracts
the part up to the *first* dot. Thus an interface named
'a.b.0' would have a VLANID of 0 and a raw device name of
just 'a'.
I also don't see a reason to limit the removed portions to
alphanumeric characters. I.e. an interface named tap-test.0
would result in a VLANID of 'tap-0' and an
IF_VLAN_RAW_DEVICE of 'test'.
A simple shell substitution seems to do a much better job
and is more efficient than forking out two processes with a
pipe.
Thomas Lamprecht [Mon, 27 Jul 2015 11:14:30 +0000 (13:14 +0200)]
Use JSONSchema to parse vzdump config
Instead of a lot of hardcoded if's use JSONSchema::parse_config to
parse and validate vzdump.conf. To do that $confdesc was extended
to match a valid schema.
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
ext5migrate: set buttons property directly instead of calling applyIf
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.
bridge_vids is optional, and allow on the specified vlans.(current take 1 value or range, need to be improve with list)
If not specified, the allowed vlan are 2-4094.
vlan 1 is the default pvid. (all untagged traffic is going to this vlan).
projects / pve-manager.git / log