Restart services when enabling cockpit

Signed-off-by: Eric D. Helms <[email protected]>

Author: ehelms

manifests/plugin/remote_execution/cockpit.pp

@@ -42,7 +42,10 @@
     service { 'foreman-cockpit':
       ensure    => running,
       enable    => true,
-      require   => Foreman::Plugin['remote_execution-cockpit'],
+      require   => [
+        Foreman::Plugin['remote_execution-cockpit'],
+        Foreman::Config::Apache::Fragment['cockpit'],
+      ],
       subscribe => File["${config_directory}/cockpit.conf", "${config_directory}/foreman-cockpit-session.yml"],
     }
   }
@@ -82,6 +85,7 @@
     foreman_config_entry { 'remote_execution_cockpit_url':
       value   => "${cockpit_path}/=%{host}",
       require => Class['foreman::database'],
+      notify  => Class['foreman::service'],
     }
   }
 }

spec/acceptance/foreman_rex_cockpit_spec.rb

@@ -17,4 +17,24 @@
   describe port(19090) do
     it { is_expected.to be_listening.on('127.0.0.1').with('tcp') }
   end
+
+  describe service('foreman-cockpit') do
+    it { is_expected.to be_running }
+    it { is_expected.to be_enabled }
+  end
+
+  describe service('httpd') do
+    it { is_expected.to be_running }
+    it { is_expected.to be_enabled }
+  end
+
+  describe file('/etc/httpd/conf.d/05-foreman-ssl.d/cockpit.conf') do
+    it { is_expected.to be_file }
+    its(:content) { should match(%r{<Location /webcon>}) }
+    its(:content) { should match(%r{ProxyPass http://127\.0\.0\.1:19090/webcon}) }
+  end
+
+  describe command('curl -k -s -o /dev/null -w "%{http_code}" https://localhost/webcon/') do
+    its(:stdout) { should match(/^(200|302|401)$/) }
+  end
 end

spec/classes/plugin/remote_execution_cockpit_spec.rb

@@ -23,11 +23,17 @@ class {'foreman':
           is_expected.to contain_foreman__plugin('tasks')
         end
 
-        it { is_expected.to contain_service('foreman-cockpit').with_ensure('running').with_enable('true') }
+        it do
+          is_expected.to contain_service('foreman-cockpit')
+            .with_ensure('running')
+            .with_enable('true')
+            .that_requires(['Foreman::Plugin[remote_execution-cockpit]', 'Foreman::Config::Apache::Fragment[cockpit]'])
+        end
 
         it do
           is_expected.to contain_foreman_config_entry('remote_execution_cockpit_url')
             .that_requires(['Class[foreman::database]', 'Foreman::Plugin[remote_execution-cockpit]'])
+            .that_notifies('Class[foreman::service]')
         end
 
         it 'creates configs' do

spec/defines/foreman_config_apache_fragment_spec.rb

@@ -33,7 +33,11 @@ class { 'foreman::config::apache':
             { content: '# config' }
           end
 
-          it { should contain_file("#{confd_dir}/05-foreman.d/test.conf").with_content('# config') }
+          it do
+            should contain_file("#{confd_dir}/05-foreman.d/test.conf")
+              .with_content('# config')
+              .that_notifies('Class[apache::service]')
+          end
           it { should contain_file("#{confd_dir}/05-foreman-ssl.d/test.conf").with_ensure(:absent) }
         end
 
@@ -62,7 +66,11 @@ class { 'foreman::config::apache':
           end
 
           it { should contain_file("#{confd_dir}/05-foreman.d/test.conf").with_ensure(:absent) }
-          it { should contain_file("#{confd_dir}/05-foreman-ssl.d/test.conf").with_content('# config') }
+          it do
+            should contain_file("#{confd_dir}/05-foreman-ssl.d/test.conf")
+              .with_content('# config')
+              .that_notifies('Class[apache::service]')
+          end
         end
       end
     end