Enhancing comment about munin::host variable issues

diff --git a/manifests/master.pp b/manifests/master.pp
index c5c23da2bb1bd72cfb5b61dfd807507e38320918..82801f4c187561d94a68650e904866fc2ebefc5a 100644 (file)
--- a/manifests/master.pp
+++ b/manifests/master.pp
@@ -9,6 +9,13 @@ class nodo::master {
 
     # 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 = $global_munin_allow
 
     include munin::host