Fix parameters to RPC "os_validate"

All other RPC wrappers take the node name(s) as the first parameter.

Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com>

diff --git a/lib/cmdlib.py b/lib/cmdlib.py
index 23ee2e9..b667019 100644 (file)
--- a/lib/cmdlib.py
+++ b/lib/cmdlib.py
@@ -8244,7 +8244,7 @@ def _CheckOSParams(lu, required, nodenames, osname, osparams):
 
   """
   nodenames = _FilterVmNodes(lu, nodenames)
-  result = lu.rpc.call_os_validate(required, nodenames, osname,
+  result = lu.rpc.call_os_validate(nodenames, required, osname,
                                    [constants.OS_VALIDATE_PARAMETERS],
                                    osparams)
   for node, nres in result.items():

diff --git a/lib/rpc.py b/lib/rpc.py
index 7d75a58..61dbe88 100644 (file)
--- a/lib/rpc.py
+++ b/lib/rpc.py
@@ -1240,7 +1240,7 @@ class RpcRunner(object):
     return result
 
   @_RpcTimeout(_TMO_FAST)
-  def call_os_validate(self, required, nodes, name, checks, params):
+  def call_os_validate(self, nodes, required, name, checks, params):
     """Run a validation routine for a given OS.
 
     This is a multi-node call.