[pve-devel] applied: [PATCH manager] vzdump: fix notifications for backing up VMs with 2+ disks to PBS
Thomas Lamprecht
t.lamprecht at proxmox.com
Fri Aug 25 15:32:11 CEST 2023
Am 25/08/2023 um 11:45 schrieb Lukas Wagner:
> 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 at proxmox.com>
> ---
> Popped up in:
> https://forum.proxmox.com/threads/keine-mails-mehr-nach-backup.132675/
>
> PVE/VZDump.pm | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
>
applied, thanks!
More information about the pve-devel
mailing list