From: Dominik Csapak <d.csapak@proxmox.com>
Date: Wed, 28 Nov 2018 09:40:45 +0000 (+0100)
Subject: better cleanup logging for migration
X-Git-Url: https://git.proxmox.com/?a=commitdiff_plain;h=0f56fff2925643527c71714ddb9897158f6a7543;p=qemu-server.git

better cleanup logging for migration

if we migrate a vm we call cleanup but the logging looks like:

Starting cleanup for 101
trying to acquire lock...
 OK
Configuration file 'nodes/pve-ceph-01/qemu-server/101.conf' does not exist

with this patch, we omit any logging in the case we do not have the config,
since we cannot know what to clean up

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
---

diff --git a/PVE/CLI/qm.pm b/PVE/CLI/qm.pm
index eceb9b3..26d4217 100755
--- a/PVE/CLI/qm.pm
+++ b/PVE/CLI/qm.pm
@@ -753,6 +753,9 @@ __PACKAGE__->register_method({
 	my $clean = $param->{'clean-shutdown'};
 	my $guest = $param->{'guest-requested'};
 
+	# return if we do not have the config anymore
+	return if !-f PVE::QemuConfig->config_file($vmid);
+
 	my $storecfg = PVE::Storage::config();
 	warn "Starting cleanup for $vmid\n";