Increase timeout of RPC adding/removing keys
authorHelga Velroyen <helgav@google.com>
Thu, 7 Jan 2016 13:27:29 +0000 (14:27 +0100)
committerHelga Velroyen <helgav@google.com>
Thu, 7 Jan 2016 14:10:05 +0000 (15:10 +0100)
This patch increases the timeout for the RPC calls that
add and remove SSH keys to the cluster. This is necessary,
because in big clusters the distribution/removal of a
key takes too long as Ganeti has to contact every node in
the cluster.

This patch increases the timeout from URGENT to FAST
(the next higher option).

The alternatives to this include splitting up the
RPC call to several calls, which will add addiional
overall runtime and RPC overhead as well as security
implications. Since the higher timeout was tested
in a big cluster, we go with this for now.

Signed-off-by: Helga Velroyen <helgav@google.com>
Reviewed-by: Hrvoje Ribicic <riba@google.com>

lib/rpc_defs.py

index 09b2fa8..74b74d1 100644 (file)
@@ -537,7 +537,7 @@ _NODE_CALLS = [
     ("daemon", None, "Daemon name"),
     ("run", None, "Whether the daemon should be running or stopped"),
     ], None, None, "Ensure daemon is running on the node."),
-  ("node_ssh_key_add", MULTI, None, constants.RPC_TMO_URGENT, [
+  ("node_ssh_key_add", MULTI, None, constants.RPC_TMO_FAST, [
     ("node_uuid", None, "UUID of the node whose key is distributed"),
     ("node_name", None, "Name of the node whose key is distributed"),
     ("potential_master_candidates", None, "Potential master candidates"),
@@ -548,7 +548,7 @@ _NODE_CALLS = [
     ("get_public_keys", None, "Whether the node should get the other nodes'"
      " public keys")],
     None, None, "Distribute a new node's public SSH key on the cluster."),
-  ("node_ssh_key_remove", MULTI, None, constants.RPC_TMO_URGENT, [
+  ("node_ssh_key_remove", MULTI, None, constants.RPC_TMO_FAST, [
     ("node_uuid", None, "UUID of the node whose key is removed"),
     ("node_name", None, "Name of the node whose key is removed"),
     ("master_candidate_uuids", None, "List of UUIDs of master candidates."),