From f33fbb527c0fa29e3dee57c384d00e1db2fa1198 Mon Sep 17 00:00:00 2001 From: Alex Vandiver Date: Mon, 24 Jan 2022 23:32:19 +0000 Subject: [PATCH] upgrade: Make calling shutdown_server twice, only try once. --- scripts/lib/upgrade-zulip-stage-2 | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/scripts/lib/upgrade-zulip-stage-2 b/scripts/lib/upgrade-zulip-stage-2 index 3063cd63e9..4c11212a1b 100755 --- a/scripts/lib/upgrade-zulip-stage-2 +++ b/scripts/lib/upgrade-zulip-stage-2 @@ -104,9 +104,10 @@ IS_SERVER_UP = True def shutdown_server() -> None: global IS_SERVER_UP - logging.info("Stopping Zulip...") - subprocess.check_call(["./scripts/stop-server"], preexec_fn=su_to_zulip) - IS_SERVER_UP = False + if IS_SERVER_UP: + logging.info("Stopping Zulip...") + subprocess.check_call(["./scripts/stop-server"], preexec_fn=su_to_zulip) + IS_SERVER_UP = False # postgresql.version is required for database servers, but wasn't