Fixes #32078 - explicitly notify dynflow core service on changes

Author: evgeni

manifests/plugin/dynflow.pp

@@ -54,12 +54,14 @@
     ~> file { '/etc/smart_proxy_dynflow_core/settings.yml':
       ensure  => file,
       content => template('foreman_proxy/plugin/dynflow_core.yml.erb'),
+      notify  => Service['smart_proxy_dynflow_core'],
     }
-    ~> file { '/etc/smart_proxy_dynflow_core/settings.d':
+    -> file { '/etc/smart_proxy_dynflow_core/settings.d':
       ensure => link,
       target => "${foreman_proxy::config_dir}/settings.d",
+      notify => Service['smart_proxy_dynflow_core'],
     }
-    ~> systemd::service_limits { 'smart_proxy_dynflow_core.service':
+    -> systemd::service_limits { 'smart_proxy_dynflow_core.service':
       limits          => {
         'LimitNOFILE' => $open_file_limit,
       },

spec/classes/foreman_proxy__plugin__dynflow_spec.rb

@@ -54,6 +54,13 @@
               ":ssl_private_key: /etc/puppetlabs/puppet/ssl/private_keys/foo.example.com.pem",
             ])
           end
+
+          it 'should restart external core' do
+            should contain_file("#{etc_dir}/smart_proxy_dynflow_core/settings.yml").
+              that_notifies('Service[smart_proxy_dynflow_core]')
+            should contain_file("#{etc_dir}/smart_proxy_dynflow_core/settings.d").
+              that_notifies('Service[smart_proxy_dynflow_core]')
+          end
         else
           it { should_not contain_foreman_proxy__plugin('dynflow_core') }
           it { should_not contain_file("#{etc_dir}/smart_proxy_dynflow_core/settings.d") }