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 <[email protected]> --- changes from v2: * use !-f config_file to check for the config * better comment PVE/CLI/qm.pm | 3 +++ 1 file changed, 3 insertions(+) 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"; -- 2.11.0 _______________________________________________ pve-devel mailing list [email protected] https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
