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.csa...@proxmox.com>
---
 PVE/CLI/qm.pm | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/PVE/CLI/qm.pm b/PVE/CLI/qm.pm
index eceb9b3..8d4fb9b 100755
--- a/PVE/CLI/qm.pm
+++ b/PVE/CLI/qm.pm
@@ -753,6 +753,12 @@ __PACKAGE__->register_method({
        my $clean = $param->{'clean-shutdown'};
        my $guest = $param->{'guest-requested'};
 
+       my $conf = eval { PVE::QemuConfig->load_config ($vmid) };
+       if (!$conf) {
+           # silently return if vm is migrated
+           return;
+       }
+
        my $storecfg = PVE::Storage::config();
        warn "Starting cleanup for $vmid\n";
 
-- 
2.11.0


_______________________________________________
pve-devel mailing list
pve-devel@pve.proxmox.com
https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to