From 685363c5a8827d4d2a380d88bbdce1b7ea2ded22 Mon Sep 17 00:00:00 2001 From: Oleksii Kliukin Date: Mon, 26 Oct 2015 18:57:05 +0100 Subject: [PATCH] Fix an invocation fo fetch_node_statuses in the API code to account for the tags --- patroni/api.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/patroni/api.py b/patroni/api.py index bb168641..e3cb891b 100644 --- a/patroni/api.py +++ b/patroni/api.py @@ -148,8 +148,8 @@ class RestApiHandler(BaseHTTPRequestHandler): members = [m for m in cluster.members if m.name != cluster.leader.name and m.api_url] if not members: return b'failover is not possible: cluster does not have members except leader' - for member, reachable, in_recovery, xlog_location in self.server.patroni.ha.fetch_nodes_statuses(members): - if reachable: + for member, reachable, in_recovery, xlog_location, tags in self.server.patroni.ha.fetch_nodes_statuses(members): + if reachable and not tags.get('nofailover'): return None return b'failover is not possible: no good candidates have been found'