[pve-devel] applied: [PATCH qemu-server v4 1/2] cfg2cmd: push vmstate to volid list to ensure it gets also deactivated

Thomas Lamprecht t.lamprecht at proxmox.com
Thu Oct 17 19:23:21 CEST 2019


the volume id list is only used to activate before real start and
deactivate later, so use it for the vmstate file too.

This not only makes config_to_command have less side effects, it also
ensures that the vmstate is deactivated again

Co-developed-by: Alwin Antreich <a.antreich at proxmox.com>
Signed-off-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
---
 PVE/QemuServer.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm
index 3a9aa1e..a3c1e04 100644
--- a/PVE/QemuServer.pm
+++ b/PVE/QemuServer.pm
@@ -4183,7 +4183,7 @@ sub config_to_command {
 
     if (my $vmstate = $conf->{vmstate}) {
 	my $statepath = PVE::Storage::path($storecfg, $vmstate);
-	PVE::Storage::activate_volumes($storecfg, [$vmstate]);
+	push @$vollist, $statepath;
 	push @$cmd, '-loadstate', $statepath;
     }
 
-- 
2.20.1





More information about the pve-devel mailing list