From 5c1d42b7f825fa124ff3701b32f9ecc011bece95 Mon Sep 17 00:00:00 2001 From: Thomas Lamprecht Date: Thu, 17 Oct 2019 19:13:01 +0200 Subject: [PATCH] Fix #2171: vm_start: volid based statefiles were not activated So, while we could just make this a special case before the config_to_command call and set the $conf->{vmstate} to the statefile for the case were it's a valid volumeid, the special case handling get's much easier when we do this outside of that method. So it's basically a trade-off, and after looking far to long at all nice revisions Alwin made for me and Fabians request, and even trying out different approaches, it was never perfect. But having slight code duplication over the movement mess I proposed (as I did not had the full picture then, sorry Alwin) felt like the slightly nicer trade off, as all worked I just use this one now, it has very clear semantics, easy to understand and that now three lines are duplicated is IMO irrelevant. Co-developed-by: Alwin Antreich Signed-off-by: Thomas Lamprecht --- PVE/QemuConfig.pm | 3 +-- PVE/QemuServer.pm | 6 +++++- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/PVE/QemuConfig.pm b/PVE/QemuConfig.pm index edbf1a7c..e9796a39 100644 --- a/PVE/QemuConfig.pm +++ b/PVE/QemuConfig.pm @@ -359,8 +359,7 @@ sub __snapshot_rollback_vm_start { my ($class, $vmid, $vmstate, $data) = @_; my $storecfg = PVE::Storage::config(); - my $statefile = PVE::Storage::path($storecfg, $vmstate); - PVE::QemuServer::vm_start($storecfg, $vmid, $statefile, undef, undef, undef, $data->{forcemachine}); + PVE::QemuServer::vm_start($storecfg, $vmid, $vmstate, undef, undef, undef, $data->{forcemachine}); } sub __snapshot_rollback_get_unused { diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm index a3c1e04f..edb49bc7 100644 --- a/PVE/QemuServer.pm +++ b/PVE/QemuServer.pm @@ -5462,8 +5462,12 @@ sub vm_start { push @$cmd, '-incoming', $migrate_uri; push @$cmd, '-S'; - } else { + } elsif (-e $statefile) { push @$cmd, '-loadstate', $statefile; + } else { + my $statepath = PVE::Storage::path($storecfg, $statefile); + push @$vollist, $statepath; + push @$cmd, '-loadstate', $statepath; } } elsif ($paused) { push @$cmd, '-S'; -- 2.39.5