From: Lukas Wagner <l.wagner@proxmox.com>
Date: Fri, 25 Aug 2023 09:45:12 +0000 (+0200)
Subject: vzdump: fix notifications for backing up VMs with 2+ disks to PBS
X-Git-Url: https://git.proxmox.com/?a=commitdiff_plain;h=461c5ee29081f92e87d3641d8f0336996a7a07ff;p=pve-manager.git

vzdump: fix notifications for backing up VMs with 2+ disks to PBS

In some situations, such as backing up VMs with 2 or more disks to
PBS, we get passed the backup archive size as a string instead of
as an integer. This led to errors rendering the notification template
down the line.

This commit explicitly casts the data from the task table to an int.
It would be a good idea to actually hunt down the places that produced
the string instead of an integer, but as a quick fix and as a
safeguard against similar lurking errors this approach is fine, IMO.

Signed-off-by: Lukas Wagner <l.wagner@proxmox.com>
---

diff --git a/PVE/VZDump.pm b/PVE/VZDump.pm
index 2671e3b1..454ab494 100644
--- a/PVE/VZDump.pm
+++ b/PVE/VZDump.pm
@@ -387,11 +387,11 @@ sub build_guest_table {
 	my $size = $successful ? $task->{size} : 0;
 	my $filename = $successful ? $task->{target} : undef;
 	push @{$table->{data}}, {
-	    "vmid" => $task->{vmid},
+	    "vmid" => int($task->{vmid}),
 	    "name" => $task->{hostname},
 	    "status" => $task->{state},
-	    "time" => $task->{backuptime},
-	    "size" => $size,
+	    "time" => int($task->{backuptime}),
+	    "size" => int($size),
 	    "filename" => $filename,
 	};
     }