Don't restart the service during initial syncing (#994)

Author: dragomirp

src/cluster.py

@@ -54,7 +54,8 @@
 
 PG_BASE_CONF_PATH = f"{POSTGRESQL_CONF_PATH}/postgresql.conf"
 
-RUNNING_STATES = ["running", "streaming"]
+STARTED_STATES = ["running", "streaming"]
+RUNNING_STATES = [*STARTED_STATES, "starting"]
 
 PATRONI_TIMEOUT = 10
 
@@ -345,7 +346,7 @@ def get_standby_leader(
         if cluster_status:
             for member in cluster_status:
                 if member["role"] == "standby_leader":
-                    if check_whether_is_running and member["state"] not in RUNNING_STATES:
+                    if check_whether_is_running and member["state"] not in STARTED_STATES:
                         logger.warning(f"standby leader {member['name']} is not running")
                         continue
                     standby_leader = member["name"]
@@ -415,7 +416,7 @@ def are_all_members_ready(self) -> bool:
         # a standby leader, because sometimes there may exist (for some period of time)
         # only replicas after a failed switchover.
         return all(
-            member["state"] in RUNNING_STATES for member in cluster_status.json()["members"]
+            member["state"] in STARTED_STATES for member in cluster_status.json()["members"]
         ) and any(
             member["role"] in ["leader", "standby_leader"]
             for member in cluster_status.json()["members"]
@@ -573,7 +574,7 @@ def online_cluster_members(self) -> list[ClusterMember]:
         if not cluster_status:
             return []
 
-        return [member for member in cluster_status if member["state"] in RUNNING_STATES]
+        return [member for member in cluster_status if member["state"] in STARTED_STATES]
 
     def are_replicas_up(self) -> dict[str, bool] | None:
         """Check if cluster members are running or streaming."""