ensure the collection pkg to be installed

theforeman · Feb 28, 2023 · e398f8c · e398f8c
1 parent 946a422
commit e398f8c
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 0 deletions.
diff --git a/manifests/plugin/ansible.pp b/manifests/plugin/ansible.pp
@@ -58,6 +58,10 @@
     mode    => '0640',
   }
 
+  if ($facts['os']['family'] in ['RedHat', 'Debian'] and $foreman_proxy::plugin::ansible::callback == 'theforeman.foreman.foreman') {
+    ensure_packages(['ansible-collection-theforeman-foreman'])
+  }
+
   include foreman_proxy::plugin::dynflow
   include foreman_proxy::plugin::remote_execution::script
   if $install_runner {

diff --git a/spec/acceptance/ansible_spec.rb b/spec/acceptance/ansible_spec.rb
@@ -7,6 +7,10 @@
 
   it_behaves_like 'the default foreman proxy application'
 
+  describe package('ansible-collection-theforeman-foreman') do
+    it { is_expected.to be_installed }
+  end
+
   package_name = ['debian', 'ubuntu'].include?(os[:family]) ? 'python3-ansible-runner' : 'ansible-runner'
   describe package(package_name) do
     it { is_expected.to be_installed }

diff --git a/spec/classes/foreman_proxy__plugin__ansible_spec.rb b/spec/classes/foreman_proxy__plugin__ansible_spec.rb
@@ -14,13 +14,15 @@
         case os
         when 'debian-11-x86_64'
           it { should contain_package('python3-ansible-runner').with_ensure('installed') }
+          it { should contain_package('ansible-collection-theforeman-foreman').with_ensure('installed') }
         when 'redhat-7-x86_64'
           it 'should include ansible-runner upstream repo' do
             should contain_yumrepo('ansible-runner')
                    .with_ensure('absent')
                    .that_comes_before('Package[ansible-runner]')
           end
           it { should contain_package('ansible-runner').with_ensure('installed') }
+          it { should contain_package('ansible-collection-theforeman-foreman').with_ensure('installed') }
         end