Signed-off-by: Fabian Ebner
---
PVE/Storage/CIFSPlugin.pm | 1 +
PVE/Storage/CephFSPlugin.pm| 1 +
PVE/Storage/DirPlugin.pm | 5 ++--
PVE/Storage/GlusterfsPlugin.pm | 5 ++--
PVE/Storage/NFSPlugin.pm | 5 ++--
PVE/Storage/PBSPlugin.pm | 1 +
PVE/Storage/Plugin.pm
Implement it for generic storages supporting backups (i.e.
directory-based storages) and add a wrapper for PBS.
Signed-off-by: Fabian Ebner
---
PVE/Storage.pm | 27 -
PVE/Storage/PBSPlugin.pm | 50
PVE/Storage/Plugin.pm | 128
test/prune_bac
Signed-off-by: Fabian Ebner
---
Not sure if this is the best place for the new API endpoints.
I decided to opt for two distinct calls rather than just using a
--dry-run option and use a worker for actually pruning, because
removing many backups over the network might take a while.
PVE/API2/Sto
What's still missing is to use this setting when executing backups
as we do with 'maxfiles' at the moment, and also GUI integration.
archive_info has been extended to include more information, mainly
for being able to tell when a backup doesn't use a standard name.
But the information might be use
where 'is_std_name' shows whether the backup name uses the standard naming
schema and most likely was created by our tools.
Also adds a '^' to the existing filename matching regex, which
should be fine since basename() is used beforehand.
Signed-off-by: Fabian Ebner
---
PVE/Storage.pm
Signed-off-by: Fabian Ebner
---
PVE/API2/Storage/Status.pm | 65 +++---
1 file changed, 32 insertions(+), 33 deletions(-)
diff --git a/PVE/API2/Storage/Status.pm b/PVE/API2/Storage/Status.pm
index 14f5930..d9d9b36 100644
--- a/PVE/API2/Storage/Status.pm
+++ b/PVE/
Signed-off-by: Fabian Ebner
---
PVE/Storage/PBSPlugin.pm | 1 +
1 file changed, 1 insertion(+)
diff --git a/PVE/Storage/PBSPlugin.pm b/PVE/Storage/PBSPlugin.pm
index 3c0879c..65696f4 100644
--- a/PVE/Storage/PBSPlugin.pm
+++ b/PVE/Storage/PBSPlugin.pm
@@ -291,6 +291,7 @@ sub list_volumes {
Any feedback for these patches?
On 5/4/20 10:50 AM, Fabian Ebner wrote:
The size of VM state files and the size of unused disks not
referenced by any snapshot is not saved in the VM configuration,
so it's not available here either.
Signed-off-by: Fabian Ebner
---
Changes from v1:
* use v
On 4/17/20 7:47 AM, Alexandre Derumier wrote:
> This add a new "mtu" param to vm nic,
> and force the mtu in the guest for virtio nic only.
>
> Special value: 1 = set the same value than the bridge
> ---
> PVE/QemuServer.pm | 19 +++
> 1 file changed, 19 insertions(+)
>
>
applie
On 6/4/20 11:08 AM, Fabian Ebner wrote:
Implement it for generic storages supporting backups (i.e.
directory-based storages) and add a wrapper for PBS.
Signed-off-by: Fabian Ebner
---
PVE/Storage.pm | 27 -
PVE/Storage/PBSPlugin.pm | 50
PVE/Storage/Plugin.pm
to get the selection model of the parent, we use by default the xtype
'grid', but sometimes we want to use something else (e.g. 'treepanel')
to be flexible we make this configurable
Signed-off-by: Dominik Csapak
---
we could of course fall back to 'treepanel' directly in the code if thats
prefer
On 6/4/20 12:32 PM, Dominik Csapak wrote:
> to get the selection model of the parent, we use by default the xtype
> 'grid', but sometimes we want to use something else (e.g. 'treepanel')
>
> to be flexible we make this configurable
>
> Signed-off-by: Dominik Csapak
> ---
> we could of course fal
https://github.com/CumulusNetworks/ifupdown2/pull/166
Openvswitch already manage mtu if ovs-mtu is defined.
(Ovs manage mtu in userland, and sync mtu for some interfaces in kernel).
If mtu is changed by address module, before the ovs userland mtu,
this give packets drop.
Signed-off-by: Alexandre
Signed-off-by: Stoiko Ivanov
---
www/manager6/node/Disks.js | 8
1 file changed, 8 insertions(+)
diff --git a/www/manager6/node/Disks.js b/www/manager6/node/Disks.js
index 555956f6..7a1358ff 100644
--- a/www/manager6/node/Disks.js
+++ b/www/manager6/node/Disks.js
@@ -97,6 +97,14 @@ Ext.
hide the Serial column (the by-id link contains the same information) and
add flex to all columns
Signed-off-by: Stoiko Ivanov
---
www/manager6/node/ZFS.js | 12 +---
1 file changed, 9 insertions(+), 3 deletions(-)
diff --git a/www/manager6/node/ZFS.js b/www/manager6/node/ZFS.js
index b
This patchset fixes #2777.
Creating zpools should be done using the more stable device paths instead
of /dev/sdX. The case where this was reported was:
* hotplugging 2 disks in a server
* creating the zpool
* rebooting (the disks got new assignments)
so a not too unusual use-case.
The logic for
when compiling the disk list add a property with a stable
/dev/disk/by-id/ path for a block device when available.
This is needed to create zpools with the stable by-id links
The /dev/disk/by-id/ directory can contain multiple links to the same device
(e.g. when it's used as a LVM PV, or one for
If a device has a stable /dev/disk/by-id link use that instead of /dev/sdX
Signed-off-by: Stoiko Ivanov
---
www/manager6/node/ZFS.js | 10 +-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/www/manager6/node/ZFS.js b/www/manager6/node/ZFS.js
index c978f742..b8978559 100644
-
Why do we handle this at the GUI side?
I would prefer to do that on the host instead...
___
pve-devel mailing list
pve-devel@pve.proxmox.com
https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
On 6/4/20 1:33 PM, Alexandre Derumier wrote:
> https://github.com/CumulusNetworks/ifupdown2/pull/166
>
> Openvswitch already manage mtu if ovs-mtu is defined.
> (Ovs manage mtu in userland, and sync mtu for some interfaces in kernel).
>
> If mtu is changed by address module, before the ovs userla
On 6/4/20 5:38 PM, Dietmar Maurer wrote:
> Why do we handle this at the GUI side?
>
> I would prefer to do that on the host instead...
>
yeah, please. In general I'd like that the disk manage stuff adheres more to
what
we do in the installer - as it solved those "issues" already long ago.
I.e.
On 5/4/20 4:08 PM, Aaron Lauterer wrote:
> Signed-off-by: Aaron Lauterer
> ---
> v1 -> v2: adapt handling of return values, closer to what is used in
> production code.
>
in general OK, two suggestions below.
> test/Makefile | 5 +-
> test/vzdump_guest_included_test.pl |
22 matches
Mail list logo
