From 396b57338951144137e286851d8f8952ca55ed88 Mon Sep 17 00:00:00 2001 From: Iustin Pop <iustin@google.com> Date: Tue, 23 Jun 2009 17:22:50 +0200 Subject: [PATCH] backend.StartMaster: fix variable name MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit As per comments for patch βConvert node_start_master to new style resultβ, the βpayloadβ variable is renamed to βerr_msgsβ. Signed-off-by: Iustin Pop <iustin@google.com> Reviewed-by: Guido Trotter <ultrotter@google.com> --- lib/backend.py | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/lib/backend.py b/lib/backend.py index c8d5c85f3..b0e7e1c80 100644 --- a/lib/backend.py +++ b/lib/backend.py @@ -193,7 +193,7 @@ def StartMaster(start_daemons): # GetMasterInfo will raise an exception if not able to return data master_netdev, master_ip, _ = GetMasterInfo() - payload = [] + err_msgs = [] if utils.TcpPing(master_ip, constants.DEFAULT_NODED_PORT): if utils.OwnIpAddress(master_ip): # we already have the ip: @@ -201,7 +201,7 @@ def StartMaster(start_daemons): else: msg = "Someone else has the master ip, not activating" logging.error(msg) - payload.append(msg) + err_msgs.append(msg) else: result = utils.RunCmd(["ip", "address", "add", "%s/32" % master_ip, "dev", master_netdev, "label", @@ -209,7 +209,7 @@ def StartMaster(start_daemons): if result.failed: msg = "Can't activate master IP: %s" % result.output logging.error(msg) - payload.append(msg) + err_msgs.append(msg) result = utils.RunCmd(["arping", "-q", "-U", "-c 3", "-I", master_netdev, "-s", master_ip, master_ip]) @@ -222,10 +222,10 @@ def StartMaster(start_daemons): if result.failed: msg = "Can't start daemon %s: %s" % (daemon, result.output) logging.error(msg) - payload.append(msg) + err_msgs.append(msg) - if payload: - _Fail("; ".join(payload)) + if err_msgs: + _Fail("; ".join(err_msgs)) def StopMaster(stop_daemons): -- GitLab