diff options
Diffstat (limited to 'manifests')
| -rw-r--r-- | manifests/master.pp | 11 | ||||
| -rw-r--r-- | manifests/subsystems/munin.pp | 9 |
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; } } |