Make the LUInstanceCreate return node names, not UUIDs
authorHrvoje Ribicic <riba@google.com>
Mon, 3 Feb 2014 10:43:53 +0000 (10:43 +0000)
committerHrvoje Ribicic <riba@google.com>
Mon, 3 Feb 2014 15:45:31 +0000 (16:45 +0100)
The LUInstanceCreate returned names instead of UUIDs in 2.6. Along the
way, the names were internally replaced with UUIDs, and the abstraction
leaked. This patch fixes the issue.

Signed-off-by: Hrvoje Ribicic <riba@google.com>
Reviewed-by: Thomas Thrainer <thomasth@google.com>

lib/cmdlib/instance.py

index cccfe1c..0c84328 100644 (file)
@@ -1,7 +1,7 @@
 #
 #
 
-# Copyright (C) 2006, 2007, 2008, 2009, 2010, 2011, 2012, 2013 Google Inc.
+# Copyright (C) 2006, 2007, 2008, 2009, 2010, 2011, 2012, 2013, 2014 Google Inc.
 #
 # This program is free software; you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
@@ -1462,7 +1462,7 @@ class LUInstanceCreate(LogicalUnit):
                                             False, self.op.reason)
       result.Raise("Could not start instance")
 
-    return list(iobj.all_nodes)
+    return self.cfg.GetNodeNames(list(iobj.all_nodes))
 
 
 class LUInstanceRename(LogicalUnit):