From f6bfdef34d9a06dcd410e18a44b2541f6f138188 Mon Sep 17 00:00:00 2001 From: John Westcott IV Date: Wed, 25 Mar 2020 14:09:30 -0400 Subject: [PATCH] Removed old secho comment from Tower-CLI Fixed job name for tests --- awx_collection/plugins/modules/tower_job_wait.py | 4 ---- .../tests/integration/targets/tower_job_wait/tasks/main.yml | 2 +- 2 files changed, 1 insertion(+), 5 deletions(-) diff --git a/awx_collection/plugins/modules/tower_job_wait.py b/awx_collection/plugins/modules/tower_job_wait.py index c0ee082cc1..640eb1b525 100644 --- a/awx_collection/plugins/modules/tower_job_wait.py +++ b/awx_collection/plugins/modules/tower_job_wait.py @@ -146,10 +146,6 @@ def main(): start = time.time() # Poll the Ansible Tower instance for status, and print the status to the outfile (usually standard out). - # - # Note that this is one of the few places where we use `secho` even though we're in a function that might - # theoretically be imported and run in Python. This seems fine; outfile can be set to /dev/null and very - # much the normal use for this method should be CLI monitoring. result = check_job(module, job_url) last_poll = time.time() diff --git a/awx_collection/tests/integration/targets/tower_job_wait/tasks/main.yml b/awx_collection/tests/integration/targets/tower_job_wait/tasks/main.yml index 29520b0281..92457889c9 100644 --- a/awx_collection/tests/integration/targets/tower_job_wait/tasks/main.yml +++ b/awx_collection/tests/integration/targets/tower_job_wait/tasks/main.yml @@ -33,7 +33,7 @@ - name: Launch Demo Job Template (take happy path) tower_job_launch: - job_template: "Jenkins Export Vars" + job_template: "Demo Job Template" register: job - assert: