Commit de8c7666 authored by Guido Trotter's avatar Guido Trotter
Browse files

Fix LUReplaceDisks with iallocator

self._RunAllocator() sets self.op.remote_node, but doesn't return the
new remote node. If we set it to the return value of the function we
basically reset it to None, and iallocator is never run.

Reviewed-by: imsnah
parent 506cff12
...@@ -3573,7 +3573,7 @@ class LUReplaceDisks(LogicalUnit): ...@@ -3573,7 +3573,7 @@ class LUReplaceDisks(LogicalUnit):
if self.op.remote_node is not None: if self.op.remote_node is not None:
raise errors.OpPrereqError("Give either the iallocator or the new" raise errors.OpPrereqError("Give either the iallocator or the new"
" secondary, not both") " secondary, not both")
self.op.remote_node = self._RunAllocator() self._RunAllocator()
remote_node = self.op.remote_node remote_node = self.op.remote_node
if remote_node is not None: if remote_node is not None:
......
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