From ef36b4fffde3946397726099809736d193a82a72 Mon Sep 17 00:00:00 2001 From: beeankha Date: Fri, 17 Jan 2020 15:08:05 -0500 Subject: [PATCH] Reduce number of requests running in the try/except block --- awx_collection/plugins/modules/tower_job_launch.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/awx_collection/plugins/modules/tower_job_launch.py b/awx_collection/plugins/modules/tower_job_launch.py index 303de65407..7a85e3c82c 100644 --- a/awx_collection/plugins/modules/tower_job_launch.py +++ b/awx_collection/plugins/modules/tower_job_launch.py @@ -125,11 +125,13 @@ def update_fields(module, p): job_template = params.get('job_template') extra_vars = params.get('extra_vars') try: - ask_extra_vars = tower_cli.get_resource('job_template').get(name=job_template)['ask_variables_on_launch'] - survey_enabled = tower_cli.get_resource('job_template').get(name=job_template)['survey_enabled'] + job_template_to_launch = tower_cli.get_resource('job_template').get(name=job_template) except (exc.NotFound) as excinfo: module.fail_json(msg='Unable to launch job, job_template/{0} was not found: {1}'.format(job_template, excinfo), changed=False) + ask_extra_vars = job_template_to_launch['ask_variables_on_launch'] + survey_enabled = job_template_to_launch['survey_enabled'] + if extra_vars and (ask_extra_vars or survey_enabled): params_update['extra_vars'] = [json.dumps(extra_vars)]