Issue 2160 - Ensure nagios_hostgroup() does not break reprovisioning

modules/nagios/manifests/server.pp

Index: modules/nagios/manifests/server.pp
===================================================================
--- a/modules/nagios/manifests/server.pp
+++ b/modules/nagios/manifests/server.pp
@@ -187,4 +187,17 @@
       hostgroups => $groups,
     }
   }
+
+  # This is a workaround removing "member" directives with absent values,
+  # which are created by Puppet when configuring a hostgroup without any
+  # explicit members (which is a valid scenario), but not properly parsed
+  # during the provisioning process:
+  Nagios_hostgroup <| |> ~>
+  exec {'nagios-hostgroup-fixup':
+    command => shellquote('sed', '-i',

[mathias, 2015/03/18 11:26:36]

I'd be glad if anybody could point out a better solution, maybe I just don't see
the wood for the trees here. Frankly, like I've indicated in the issue tracker,
it's quite a mess:

- A dummy value does not work because Nagios does not accept any unrecognized
input here
- An empty string with q ('""') results in Nagios complaining about an unknown
host
  named '""'.
- There is no explicit Nagios config keyword for the concept of a value being
absent or empty.
- Some sort of default host (e.g. "localhost") obviously makes no sense either.
- One cannot configure a Puppet exec { .. to rm(1) the file only before it's
  updated.
- Iterating through data is a mess in any Puppet version prior to 3.2 (which
introduces each(..)), simply because there is no such thing. 

The common workaround for the latter point is to write wrapper types and apply
the data as input to them - just as we did above. But in our case, I do not see
any way how that could help us circumventing the issue without creating our own,
e.g. template-based alternative for the builtin (!!) nagios_hostgroup(). That
would also be way more code and thus way more prone to error.

Another solution you may like more is just deleting the file at every run. That
of course triggers every run to be seen as an update here, but we've done
similar workarounds before. I actually prefer my sed(1) solution because I do
trust in my regex- and shell-foo.. ;-)

We can also remove this hack, document the issue and maintain the host group
associations manually. But that's basically the same as we did before and makes
even less sense.

+      '/^define\shostgroup\s{/,/^}/ { /^\s*members\s*$/d }',
+      '/etc/nagios3/conf.d/hostgroups.cfg'),
+    path => '/bin',
+    refreshonly => true,
+  }
 }