[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