aboutsummaryrefslogtreecommitdiff
path: root/manifests
diff options
context:
space:
mode:
Diffstat (limited to 'manifests')
-rw-r--r--manifests/master.pp11
-rw-r--r--manifests/subsystems/munin.pp9
2 files changed, 5 insertions, 15 deletions
diff --git a/manifests/master.pp b/manifests/master.pp
index 28fbdf1..118e28a 100644
--- a/manifests/master.pp
+++ b/manifests/master.pp
@@ -7,17 +7,6 @@ class nodo::master {
main => true,
}
- # This is a workaround to correctly set allowed hosts on munin
- # client when the host is also present.
- #
- # Somewhow the inclusion of munin::host makes both $munin_allow
- # and $munin_port to get blank.
- #
- # Right now we don't need to bother with fixing $munin_port as
- # we are running the munin host in the main master node but it's
- # mandatory to fix $munin_allow so munin can connect into the node.
- $munin_allow = hiera('nodo::munin_node::host', '')
-
include munin::host
include munin::plugins::muninhost
diff --git a/manifests/subsystems/munin.pp b/manifests/subsystems/munin.pp
index a0a534e..a0eb7d9 100644
--- a/manifests/subsystems/munin.pp
+++ b/manifests/subsystems/munin.pp
@@ -9,10 +9,11 @@ define munin_node(
'': { fail("Please set nodo::munin_node::allow in your site config") }
}
- $munin_port = $port
- $munin_allow = $allow
- $munin_host = $host
+ class { 'munin::client':
+ port => $port,
+ allow => $allow,
+ host => $host,
+ }
- include munin::client
munin::plugin { apt_all: ensure => present; }
}