Handle munin_allow and nagios_allow variables as arrays

munin/manifests/init.pp

@@ -3,11 +3,18 @@
 #
 # === Global variables
 #
+#   $munin_allow:
+#       Array of IP addresses allowed to connect to munin-node.
+#
 #   $munin_tls:
 #       Enable and require TLS if set to "true".
 #
 class munin::node {
 
+    if !$munin_allow {
+        $munin_allow = [ "127.0.0.1" ]
+    }
+
     package { "munin-node":
         ensure => installed,
     }

munin/templates/munin-node.conf.erb

@@ -37,8 +37,9 @@ host_name <%= @homename %>
 # regular expression, due to brain damage in Net::Server, which
 # doesn't understand CIDR-style network notation.  You may repeat
 # the allow line as many times as you'd like
-
-allow <%= @munin_allow %>
+<% @munin_allow.each do |allow| -%>
+allow <%= "^%s$" % allow.gsub('.', '\.') %>
+<% end -%>
 
 # Which address to bind to;
 host <%= @ipaddress %>

nagios/manifests/target.pp

@@ -179,10 +179,15 @@ define nagios::target::tcp($port, $description=undef) {
 
 # Configure nagios nrpe target.
 #
+#   === Global variables
+#
+#   $nagios_allow:
+#       Array of IP addresses allowed to connect to the nrpe service.
+#
 class nagios::target::nrpe inherits nagios::target {
 
     if !$nagios_allow {
-        $nagios_allow = "127.0.0.1"
+        $nagios_allow = [ "127.0.0.1" ]
     }
 
     include nagios::common
@@ -248,7 +253,7 @@ class nagios::target::nrpe inherits nagios::target {
             "openbsd" => "wheel",
             default   => "root",
         },
-        content => "allowed_hosts=${nagios_allow}\n",
+        content => inline_template("allowed_hosts=<%= @nagios_allow.join(',') %>\n"),
         require => File["/etc/nrpe.d"],
         notify  => Service["nrpe"],
     }