AMBARI-19061. Ambari-server restart command not working. if any parameters are changed and reset to default. (aonishuk)
Project: http://git-wip-us.apache.org/repos/asf/ambari/repo Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/2de7ee8d Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/2de7ee8d Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/2de7ee8d Branch: refs/heads/branch-feature-AMBARI-18456 Commit: 2de7ee8d01ce850bbbf9844ad8789615b6570168 Parents: c2dc753 Author: Andrew Onishuk <[email protected]> Authored: Fri Dec 2 11:45:57 2016 +0200 Committer: Andrew Onishuk <[email protected]> Committed: Fri Dec 2 11:45:57 2016 +0200 ---------------------------------------------------------------------- .../src/main/python/ambari_server/utils.py | 23 ++++++++++---------- 1 file changed, 12 insertions(+), 11 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/ambari/blob/2de7ee8d/ambari-server/src/main/python/ambari_server/utils.py ---------------------------------------------------------------------- diff --git a/ambari-server/src/main/python/ambari_server/utils.py b/ambari-server/src/main/python/ambari_server/utils.py index 26e59ae..fc631f5 100644 --- a/ambari-server/src/main/python/ambari_server/utils.py +++ b/ambari-server/src/main/python/ambari_server/utils.py @@ -127,17 +127,18 @@ def save_main_pid_ex(pids, pidfile, exclude_list=[], skip_daemonize=False): """ pid_saved = False try: - pfile = open(pidfile, "w") - for item in pids: - if pid_exists(item["pid"]) and (item["exe"] not in exclude_list): - pfile.write("%s\n" % item["pid"]) - pid_saved = True - logger.info("Ambari server started with PID " + str(item["pid"])) - if pid_exists(item["pid"]) and (item["exe"] in exclude_list) and not skip_daemonize: - try: - os.kill(int(item["pid"]), signal.SIGKILL) - except: - pass + if pids: + pfile = open(pidfile, "w") + for item in pids: + if pid_exists(item["pid"]) and (item["exe"] not in exclude_list): + pfile.write("%s\n" % item["pid"]) + pid_saved = True + logger.info("Ambari server started with PID " + str(item["pid"])) + if pid_exists(item["pid"]) and (item["exe"] in exclude_list) and not skip_daemonize: + try: + os.kill(int(item["pid"]), signal.SIGKILL) + except: + pass except IOError as e: logger.error("Failed to write PID to " + pidfile + " due to " + str(e)) pass
