From d3c2b8b2aa28a09f4c2c30169ba061fae70268c7 Mon Sep 17 00:00:00 2001 From: Alexander Kukushkin Date: Fri, 11 Mar 2016 15:18:58 +0100 Subject: [PATCH] replace unused variables with _ --- patroni/api.py | 4 ++-- patroni/ctl.py | 2 +- patroni/ha.py | 4 ++-- patroni/scripts/wale_restore.py | 2 +- 4 files changed, 6 insertions(+), 6 deletions(-) diff --git a/patroni/api.py b/patroni/api.py index 1186321f..f2877b05 100644 --- a/patroni/api.py +++ b/patroni/api.py @@ -143,7 +143,7 @@ class RestApiHandler(BaseHTTPRequestHandler): self.wfile.write(data) def poll_failover_result(self, leader, member): - for a in range(0, 15): + for _ in range(0, 15): time.sleep(1) try: cluster = self.server.patroni.dcs.get_cluster() @@ -166,7 +166,7 @@ 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, tags in self.server.patroni.ha.fetch_nodes_statuses(members): + for member, reachable, _, xlog_location, tags in self.server.patroni.ha.fetch_nodes_statuses(members): if reachable and not tags.get('nofailover', False): return None return b'failover is not possible: no good candidates have been found' diff --git a/patroni/ctl.py b/patroni/ctl.py index c4cc9fe0..9d07267b 100644 --- a/patroni/ctl.py +++ b/patroni/ctl.py @@ -240,7 +240,7 @@ def dsn(cluster_name, config_file, dcs, role, member): if member is None and role is None: role = 'master' - config, dcs, cluster = ctl_load_config(cluster_name, config_file, dcs) + _, dcs, cluster = ctl_load_config(cluster_name, config_file, dcs) m = get_any_member(cluster=cluster, role=role, member=member) if m is None: raise PatroniCtlException('Can not find a suitable member') diff --git a/patroni/ha.py b/patroni/ha.py index 98ba4398..d4697fe0 100644 --- a/patroni/ha.py +++ b/patroni/ha.py @@ -203,7 +203,7 @@ class Ha(object): ret = False members = [m for m in members if m.name != self.state_handler.name and not m.nofailover and m.api_url] if members: - for member, reachable, in_recovery, xlog_location, tags in self.fetch_nodes_statuses(members): + for member, reachable, _, xlog_location, tags in self.fetch_nodes_statuses(members): if reachable and not tags.get('nofailover', False): ret = True # TODO: check xlog_location elif not reachable: @@ -223,7 +223,7 @@ class Ha(object): # find specific node and check that it is healthy members = [m for m in self.cluster.members if m.name == failover.member] if members: - member, reachable, in_recovery, xlog_location, tags = self.fetch_node_status(members[0]) + member, reachable, _, xlog_location, tags = self.fetch_node_status(members[0]) if reachable and not tags.get('nofailover', False): # node is healthy logger.info('manual failover: to %s, i am %s', member.name, self.state_handler.name) return False diff --git a/patroni/scripts/wale_restore.py b/patroni/scripts/wale_restore.py index c80cdfae..f3691907 100755 --- a/patroni/scripts/wale_restore.py +++ b/patroni/scripts/wale_restore.py @@ -154,7 +154,7 @@ def main(): args = parser.parse_args() # retry cloning in a loop - for retry in range(0, args.retries + 1): + for _ in range(0, args.retries + 1): restore = WALERestore(scope=args.scope, datadir=args.datadir, connstring=args.connstring, env_dir=args.envdir, threshold_mb=args.threshold_megabytes, threshold_pct=args.threshold_backup_size_percentage, use_iam=args.use_iam,