[PATCH storage v3 0/2] fix #7598: qemu-img: increase timeout for image resize

[Jakob Klocker <j.klocker@proxmox.com>]

Image resizing currently has a timeout of 10 seconds, which can fail
on slow disks and leave the VM in a state where the disk is updated,
but the VM config isn't. 

Since there is no deterministic way to check, with less overhead, if
this is the case, increasing the timeout is the most pragmatic
solution. The resize is only called in a worker context anyway,
therefore this patch series mirrors the ZFS timeout: it increases
the timeout in a worker context to 1 hour; as a fallback for a 
non-worker context the timeout stays 10 seconds.

Additionally, a minor refactoring: the timeout argument was dropped at
the two call sites of qemu_img_resize, since both passed the default
value of 10 seconds.

Changes since v2
----------------
- restore timeout parameter since the function can be used by
third-party plugins (Thanks @Fiona & @Elias)

Changes since v1
-----------------
- drop the timeout parameter in qemu_img_resize
- increase the timeout instead of querying the disk information
(see v1 for more info. Thanks @Fabian)

v1: https://lore.proxmox.com/pve-devel/20260603082557.25359-1-j.klocker@proxmox.com/t/#u

Link: https://bugzilla.proxmox.com/show_bug.cgi?id=7598


pve-storage:

Jakob Klocker (2):
  qemu-img: drop redundant timeout argument at resize call
  fix #7598: qemu-img: increase timeout for image resizing in worker
    context

 src/PVE/Storage/Common.pm    | 6 ++++--
 src/PVE/Storage/LVMPlugin.pm | 2 +-
 src/PVE/Storage/Plugin.pm    | 2 +-
 3 files changed, 6 insertions(+), 4 deletions(-)


Summary over all repositories:
  3 files changed, 6 insertions(+), 4 deletions(-)

-- 
Generated by murpp 0.12.0