From 4ced0c5bbe605c322dc4103c3572ae18e69fa278 Mon Sep 17 00:00:00 2001
From: Matt Taggart <taggart@riseup.net>
Date: Thu, 22 Jan 2015 13:10:10 -0800
Subject: improve comments for ps stuff

---
 manifests/ps.pp | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

(limited to 'manifests/ps.pp')

diff --git a/manifests/ps.pp b/manifests/ps.pp
index c84f583..1171a13 100644
--- a/manifests/ps.pp
+++ b/manifests/ps.pp
@@ -6,8 +6,12 @@ define check_mk::ps (
   $levels = '1, 1, 1, 1',
   $user = undef
 ) {
+  # This class is called on check-mk agent machines in order to create
+  # checks using the built-in ps check type. They create stored configs
+  # and then the check_mk::server::collect_ps class on the server
+  # generates the config file to set them up
 
-  # lines look like
+  # lines in the ps.mk config file look like
   # ( "foo.example.com", "ps", "NAME", ( "/usr/sbin/foo", 1, 1, 1, 1 ) )
   # or with a user
   # ( "foo.example.com", "ps", "NAME", ( "/usr/sbin/foo", "user", 1, 1, 1, 1 ) )
@@ -17,9 +21,10 @@ define check_mk::ps (
     $check = "  ( \"${host}\", \"ps\", \"${desc}\", ( \"${procname}\", ${levels} ) ),\n"
   }
 
-  # FIXME: we could be smarter about this and consolidate host checks that have
-  #  identical settings and that would make the config file make more sense
-  #  for humans. but for now we'll just do separate lines.
+  # FIXME: we could be smarter about this and consolidate host checks
+  # that have identical settings and that would make the config file
+  # make more sense for humans. but for now we'll just do separate
+  # lines (which may result in a very large file, but check-mk is fine)
   concat::fragment { "check_mk_ps-${host}_${desc}":
     target  => $target,
     content => $check,
-- 
cgit v1.2.3