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 2671e3b1ef0a23bc933901848934d2ba381b0107..454ab4944d91c3b5f1884bd28cb00145db6228e3 100644 (file)
--- 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,
        };
     }