[pve-devel] applied: [PATCH manager 2/2] bulk migration: only pass 'with-local-disks' along if defined

Thomas Lamprecht t.lamprecht at proxmox.com
Thu Dec 5 13:56:48 CET 2019


To make it backward compaitble. NBo real harm without this, but lots
of ugly undefiend $val warnings...

Signed-off-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
---
 PVE/API2/Nodes.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/PVE/API2/Nodes.pm b/PVE/API2/Nodes.pm
index 66832537..58497b2b 100644
--- a/PVE/API2/Nodes.pm
+++ b/PVE/API2/Nodes.pm
@@ -1953,8 +1953,8 @@ my $create_migrate_worker = sub {
 	    vmid => $vmid,
 	    target => $target,
 	    online => $online,
-	    'with-local-disks' => $with_local_disks
 	};
+	$params->{'with-local-disks'} = $with_local_disks if defined($with_local_disks);
 
 	$upid = PVE::API2::Qemu->migrate_vm($params);
     } else {
-- 
2.20.1





More information about the pve-devel mailing list