The current validation routine just says "failed", without specifying
the node name. This is very confusing, and we should log the node name
too.
---
 lib/cmdlib.py |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/cmdlib.py b/lib/cmdlib.py
index 0e2e184..a39e85e 100644
--- a/lib/cmdlib.py
+++ b/lib/cmdlib.py
@@ -4234,8 +4234,8 @@ def _CheckHVParams(lu, nodenames, hvname, hvparams):
       continue
     msg = info.RemoteFailMsg()
     if msg:
-      raise errors.OpPrereqError("Hypervisor parameter validation failed:"
-                                 " %s" % msg)
+      raise errors.OpPrereqError("Hypervisor parameter validation"
+                                 " failed on node %s:" " %s" % (node, msg))
 
 
 class LUCreateInstance(LogicalUnit):
-- 
1.6.2.4

Reply via email to