let the user override the nagios2_description, which is necessary for

multiple backup defines on one host

diff --git a/manifests/rdiff.pp b/manifests/rdiff.pp
index 38a975db040c5d60a6cfba6b4c171a2d5d2565a4..587c02dd65c83af31d75f2f4ecb46becece29735 100644 (file)
--- a/manifests/rdiff.pp
+++ b/manifests/rdiff.pp
@@ -25,7 +25,7 @@ define backupninja::rdiff(
                "/home", "/usr/local/*bin", "/var/lib/dpkg/status*" ],
   $vsinclude = false, $keep = 30, $sshoptions = false, $options = '--force', $ssh_dir_manage = true,
   $ssh_dir = false, $authorized_keys_file = false, $installuser = true, $installkey = true, $key = false,
-  $backuptag = false, $home = false, $backupkeytype = "rsa", $backupkeystore = false, $extras = false)
+  $backuptag = false, $home = false, $backupkeytype = "rsa", $backupkeystore = false, $extras = false, $nagios2_description = 'backups')
 {
   include backupninja::client::rdiff_backup
 
@@ -49,6 +49,7 @@ define backupninja::rdiff(
         manage_ssh_dir => $ssh_dir_manage, ssh_dir => $ssh_dir, key => $key,
         authorized_keys_file => $authorized_keys_file, installuser => $installuser,
         backuptag => $real_backuptag, keytype => $backupkeytype, backupkeys => $backupkeystore,
+        nagios2_description => $nagios2_description
       }
      
       backupninja::client::key

diff --git a/manifests/server.pp b/manifests/server.pp
index 291ffede8f8e7e1c1a7831a8622c2d7d12f8c8a1..a64e8546fc4671a8bcc12a534329d0091282a185 100644 (file)
--- a/manifests/server.pp
+++ b/manifests/server.pp
@@ -62,7 +62,7 @@ class backupninja::server {
   define sandbox(
     $user = false, $host = false, $installuser = true, $dir = false, $manage_ssh_dir = true,
     $ssh_dir = false, $authorized_keys_file = false, $key = false, $keytype = 'dss', $backupkeys = false, $uid = false,
-    $gid = "backupninjas", $backuptag = false)
+    $gid = "backupninjas", $backuptag = false, $nagios2_description = "backups")
   {
     
     $real_user = $user ? {
@@ -95,9 +95,14 @@ class backupninja::server {
       default => $backuptag,
     }
 
+    $real_nagios2_description = $nagios2_description ? {
+      false => "backups",
+      default => $nagios2_description,
+    }
+
     if $nagios_server {
       # configure a passive service check for backups
-      nagios2::passive_service { "backups-$name": nagios2_host_name => $real_host, nagios2_description => 'backups', servicegroups => "backups" }
+      nagios2::passive_service { "backups-$name": nagios2_host_name => $real_host, nagios2_description => $real_nagios2_description, servicegroups => "backups" }
     }
     
     if !defined(File["$real_dir"]) {