]> git.proxmox.com Git - qemu-server.git/commit
fix #2816: restore: remove timeout when allocating disks
authorFiona Ebner <f.ebner@proxmox.com>
Tue, 12 Sep 2023 09:16:15 +0000 (11:16 +0200)
committerThomas Lamprecht <t.lamprecht@proxmox.com>
Fri, 15 Sep 2023 14:27:44 +0000 (16:27 +0200)
commit853757ccec20d5e84d6a1cc656a66beaf3d3e94c
tree0a85a21dc7e272f9aff9dd60e20a22f5a27a78cf
parent6d1ac42b951529499c380a4f4950c13145edbc34
fix #2816: restore: remove timeout when allocating disks

10 minutes is not long enough when disks are large and/or network
storages are used when preallocation is not disabled. The default is
metadata preallocation for qcow2, so there are still reports of the
issue [0][1]. If allocation really does not finish like the comment
describing the timeout feared, just let the user cancel it.

Also note that when restoring a PBS backup, there is no timeout for
disk allocation, and there don't seem to be any user complaints yet.

The 5 second timeout for receiving the config from vma is kept,
because certain corruptions in the VMA header can lead to the
operation hanging there.

There is no need for the $tmp variable before setting back the old
timeout, because that is at least one second, so we'll always be able
to set the $oldtimeout variable to undef in time in practice.
Currently, there shouldn't even be an outer timeout in the first
place, because the only call path leading to here is via the create
API (also used by qmrestore), both of which don't set a timeout.

[0]: https://forum.proxmox.com/threads/126825/
[1]: https://forum.proxmox.com/threads/128093/

Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
PVE/QemuServer.pm