Quick-Fix in the context of #765 and #760

modules/nagios/manifests/server.pp

 class nagios::server(
     $domain,
     $certificate,
     $private_key,
     $is_default=false,
     $htpasswd_source,
     $admins
   ) {
 
   File {
@@ skipping 64 matching lines @@
     require => Package['nagios3'],
     notify => Service['nagios3']
   }
 
   file {'/etc/nagios3/conf.d/generic-service.cfg':
     source => 'puppet:///modules/nagios/generic-service.cfg',
     require => Package['nagios3'],
     notify => Service['nagios3']
   }
 
   file {['/etc/nagios3/conf.d/contacts_nagios2.cfg',

[mathias, 2014/07/10 10:22:17]

Another possible optimization here: We could probably perform the deletion based
on a pattern (e.g. *_nagios2.cfg). That way we won't be surprised some day, when
the maintainer of the package introduces new files or something like that (see
also my next comment). On the other hand we would then have to ensure that we
won't use a matching filename ourselves.

[Wladimir Palant, 2014/07/10 10:38:20]

I don't think Puppet allows pattern matches for file resources. We could use a
shell command instead but that's ugly (especially checking whether that command
needs to run). And we aren't guaranteed that new files added by the package
maintainer will still follow that naming scheme. So I'd be in favor of leaving
things as they are. Note that we are using an LTS Ubuntu release exactly in
order to avoid such issues, at least for a while.

          '/etc/nagios3/conf.d/extinfo_nagios2.cfg',
          '/etc/nagios3/conf.d/generic-host_nagios2.cfg',
          '/etc/nagios3/conf.d/generic-service_nagios2.cfg',
          '/etc/nagios3/conf.d/hosts_nagios2.cfg',
          '/etc/nagios3/conf.d/hostgroups_nagios2.cfg',
          '/etc/nagios3/conf.d/localhost_nagios2.cfg',
          '/etc/nagios3/conf.d/services_nagios2.cfg']:
-    ensure => absent
+    ensure => absent,
+    require => Package['nagios3'],
+    before => Service['nagios3']

[mathias, 2014/07/10 10:22:17]

Almost sure.

As far as I can see these files are the defaults provided by the package
manager. Because they conflict with the ones puppet generates for us, they've
been removed in this directive (before my commit already). All I've tried to do
is a) ensure that this is done "early enough" - before the service is started
the first time - and b) "late enough" - after the package is installed.

I believe this issue existed for a while, but since execution order varies when
undefined the problem has gone unnoticed for a while.

   }
 
   resources {['nagios_contact', 'nagios_contactgroup', 'nagios_host',
               'nagios_hostgroup', 'nagios_service']:
     purge => true
   }
 
   Nagios_contact <| |> {
     target => '/etc/nagios3/conf.d/contacts.cfg',
     require => Package['nagios3'],
@@ skipping 33 matching lines @@
   file {['/etc/nagios3/conf.d/contacts.cfg',
          '/etc/nagios3/conf.d/contactgroups.cfg',
          '/etc/nagios3/conf.d/commands.cfg',
          '/etc/nagios3/conf.d/hosts.cfg',
          '/etc/nagios3/conf.d/hostgroups.cfg',
          '/etc/nagios3/conf.d/services.cfg']:
     require => Package['nagios3'],
     notify => Service['nagios3']
   }
 }