From 268b8e42eb2d3cf25f628ba82509fd3ce62b02b1 Mon Sep 17 00:00:00 2001 From: Iustin Pop <iustin@google.com> Date: Thu, 27 Nov 2008 03:13:05 +0000 Subject: [PATCH] Fix a message in LUExportInstance We never verified the node name before, so this is most likely not a non-retrieve but a wrong name case. Reviewed-by: ultrotter --- lib/cmdlib.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/lib/cmdlib.py b/lib/cmdlib.py index 52c08f4a3..ad1d27667 100644 --- a/lib/cmdlib.py +++ b/lib/cmdlib.py @@ -5139,8 +5139,9 @@ class LUExportInstance(LogicalUnit): self.dst_node = self.cfg.GetNodeInfo( self.cfg.ExpandNodeName(self.op.target_node)) - assert self.dst_node is not None, \ - "Cannot retrieve locked node %s" % self.op.target_node + if self.dst_node is None: + # This is wrong node name, not a non-locked node + raise errors.OpPrereqError("Wrong node name %s" % self.op.target_node) # instance disk type verification for disk in self.instance.disks: -- GitLab