From 6473b88ae3dee2fb08b933cbe165374ae935bac2 Mon Sep 17 00:00:00 2001 From: Nick Carboni Date: Wed, 17 Jul 2019 17:55:34 -0400 Subject: [PATCH] Pass verbosity through for ansible playbook automate methods --- .../embedded_ansible/automation_manager/playbook_runner.rb | 2 +- .../automation_manager/playbook_runner_spec.rb | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/app/models/manageiq/providers/embedded_ansible/automation_manager/playbook_runner.rb b/app/models/manageiq/providers/embedded_ansible/automation_manager/playbook_runner.rb index 8ca45efb080d..8bdd0975e5d3 100644 --- a/app/models/manageiq/providers/embedded_ansible/automation_manager/playbook_runner.rb +++ b/app/models/manageiq/providers/embedded_ansible/automation_manager/playbook_runner.rb @@ -38,7 +38,7 @@ def create_job_template def launch_ansible_tower_job set_status('launching tower job') - launch_options = options.slice(:extra_vars, :limit) + launch_options = options.slice(:extra_vars, :limit, :verbosity) launch_options[:hosts] = hosts_array(options[:hosts]) tower_job = ManageIQ::Providers::EmbeddedAnsible::AutomationManager::Job.create_job(temp_configuration_script, launch_options) options[:tower_job_id] = tower_job.id diff --git a/spec/models/manageiq/providers/embedded_ansible/automation_manager/playbook_runner_spec.rb b/spec/models/manageiq/providers/embedded_ansible/automation_manager/playbook_runner_spec.rb index 3b6725651651..b983dc8b9d58 100644 --- a/spec/models/manageiq/providers/embedded_ansible/automation_manager/playbook_runner_spec.rb +++ b/spec/models/manageiq/providers/embedded_ansible/automation_manager/playbook_runner_spec.rb @@ -83,10 +83,10 @@ end context 'with launch options' do - let(:options) { {:job_template_ref => 'jt1', :extra_vars => {:thing => "stuff"}} } + let(:options) { {:job_template_ref => 'jt1', :extra_vars => {:thing => "stuff"}, :verbosity => "4"} } it 'passes them to the job' do expect(ManageIQ::Providers::EmbeddedAnsible::AutomationManager::Job).to receive(:create_job) - .with(an_instance_of(ManageIQ::Providers::EmbeddedAnsible::AutomationManager::ConfigurationScript), :hosts => ["localhost"], :extra_vars => {:thing => "stuff"}) + .with(an_instance_of(ManageIQ::Providers::EmbeddedAnsible::AutomationManager::ConfigurationScript), :hosts => ["localhost"], :extra_vars => {:thing => "stuff"}, :verbosity => "4") .and_return(double(:id => 'jb1')) expect(subject).to receive(:queue_signal) subject.launch_ansible_tower_job