]> git.proxmox.com Git - qemu-server.git/commitdiff
destroy_vm: use write_config from our Config module to set an "empty" config
authorThomas Lamprecht <t.lamprecht@proxmox.com>
Fri, 18 Oct 2019 09:21:58 +0000 (11:21 +0200)
committerThomas Lamprecht <t.lamprecht@proxmox.com>
Fri, 18 Oct 2019 09:22:07 +0000 (11:22 +0200)
brings us more in line with what we do in pve-container, also it's
good to not use file_set_contents directly if we have all those nice
wrapper interface methods to do things in a safe and guaranteed way.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
PVE/QemuServer.pm

index 9fc81ae47fe18b2c01fc75d4ae908097f62ef052..d69cd969a96ecd22cce0d4f2e4a47d614ce19932 100644 (file)
@@ -2619,8 +2619,6 @@ sub touch_config {
 sub destroy_vm {
     my ($storecfg, $vmid, $keep_empty_config, $skiplock) = @_;
 
-    my $conffile = PVE::QemuConfig->config_file($vmid);
-
     my $conf = PVE::QemuConfig->load_config($vmid);
 
     PVE::QemuConfig->check_lock($conf) if !$skiplock;
@@ -2663,7 +2661,7 @@ sub destroy_vm {
     });
 
     if ($keep_empty_config) {
-       PVE::Tools::file_set_contents($conffile, "memory: 128\n");
+       PVE::QemuConfig->write_config($vmid, "memory: 128\n");
     } else {
        PVE::QemuConfig->destroy_config($vmid);
     }