From 4589b67a44d7d13df3385abdce34ef0b511cf5d8 Mon Sep 17 00:00:00 2001 From: Michael Scherer Date: Wed, 24 Feb 2016 12:17:44 +0100 Subject: [PATCH] Remove unused variables Session_id is unused in update_session, changed is always specifically set in all exit_json call, and consul_client.session.destroy return True or False, and is unused later (nor checked) --- lib/ansible/modules/extras/clustering/consul_session.py | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/lib/ansible/modules/extras/clustering/consul_session.py b/lib/ansible/modules/extras/clustering/consul_session.py index bfc30232a1..663456daf8 100644 --- a/lib/ansible/modules/extras/clustering/consul_session.py +++ b/lib/ansible/modules/extras/clustering/consul_session.py @@ -172,14 +172,12 @@ def lookup_sessions(module): def update_session(module): name = module.params.get('name') - session_id = module.params.get('id') delay = module.params.get('delay') checks = module.params.get('checks') datacenter = module.params.get('datacenter') node = module.params.get('node') consul_client = get_consul_api(module) - changed = True try: @@ -208,10 +206,9 @@ def remove_session(module): remove a session.""") consul_client = get_consul_api(module) - changed = False try: - session = consul_client.session.destroy(session_id) + consul_client.session.destroy(session_id) module.exit_json(changed=True, session_id=session_id)