Commit 1b67be9b authored by Guido Trotter's avatar Guido Trotter
Browse files

TLMigrateInstance: pass lu to _Check*

The various _Check* helper functions expect an lu to be passed in, but
the TL is passed instead. This works... sometimes! :)
Signed-off-by: default avatarGuido Trotter <>
Reviewed-by: default avatarIustin Pop <>
parent 6df291d4
......@@ -5461,15 +5461,15 @@ class TLMigrateInstance(Tasklet):
target_node = secondary_nodes[0]
# check memory requirements on the secondary node
_CheckNodeFreeMemory(self, target_node, "migrating instance %s" %
_CheckNodeFreeMemory(, target_node, "migrating instance %s" %, i_be[constants.BE_MEMORY],
# check bridge existance
_CheckInstanceBridgesExist(self, instance, node=target_node)
_CheckInstanceBridgesExist(, instance, node=target_node)
if not self.cleanup:
_CheckNodeNotDrained(self, target_node)
_CheckNodeNotDrained(, target_node)
result = self.rpc.call_instance_migratable(instance.primary_node,
result.Raise("Can't migrate, please use failover",
......@@ -5658,7 +5658,7 @@ class TLMigrateInstance(Tasklet):
self.feedback_fn("* checking disk consistency between source and target")
for dev in instance.disks:
if not _CheckDiskConsistency(self, dev, target_node, False):
if not _CheckDiskConsistency(, dev, target_node, False):
raise errors.OpExecError("Disk %s is degraded or not fully"
" synchronized on target node,"
" aborting migrate." % dev.iv_name)
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment