one comment inline
On 3/4/20 11:46 AM, Dietmar Maurer wrote:
Signed-off-by: Dietmar Maurer <diet...@proxmox.com>
---
PVE/Storage/Plugin.pm | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/PVE/Storage/Plugin.pm b/PVE/Storage/Plugin.pm
index eab73f5..bd4bb8c 100644
--- a/PVE/Storage/Plugin.pm
+++ b/PVE/Storage/Plugin.pm
@@ -6,6 +6,7 @@ use warnings;
use File::chdir;
use File::Path;
use File::Basename;
+use Time::Local qw(timelocal);
use PVE::Tools qw(run_command);
use PVE::JSONSchema qw(get_standard_option);
@@ -924,6 +925,11 @@ my $get_subdir_files = sub {
my $format = $2;
$info = { volid => "$sid:backup/$1", format => $format };
+ if ($fn =~ m!^vzdump\-(?:lxc|qemu)\-(?:[1-9][0-9]{2,8})\-(\d{4})_(\d{2})_(\d{2})\-(\d{2})_(\d{2})_(\d{2})\.${format}$!) {
+ my $epoch = timelocal($6, $5, $4, $3, $2-1, $1 - 1900);
+ $info->{ctime} = $epoch;
+ }
would it not be 'better' if we use the ctime from patch 4/6?
i guess its probably because of consistency of filename/time,
but after 4/6 we already have the ctime and do not need to parse anything
also would be more correct since the api documentation (3/6) explicity
says 'creation time'
+
if (defined($vmid) || $fn =~
m!\-([1-9][0-9]{2,8})\-[^/]+\.${format}$!) {
$info->{vmid} = $vmid // $1;
}
_______________________________________________
pve-devel mailing list
pve-devel@pve.proxmox.com
https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel