From 198b84bfbe5396e289764a03f9996845d3e47b86 Mon Sep 17 00:00:00 2001
From: Micah Anderson <micah@riseup.net>
Date: Mon, 4 Jan 2010 21:46:32 -0500
Subject: change the nagios2::passive_service check so that it uses the $name
 variable instead of the $real_host variable because $real_host prevents us
 from creating more than one rdiff-backup define on a server because this will
 cause a duplicate defintion. The $name variable allows for multiple without
 creating duplicate passive_service checks

---
 manifests/server.pp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'manifests/server.pp')

diff --git a/manifests/server.pp b/manifests/server.pp
index 7cd5dd1..291ffed 100644
--- a/manifests/server.pp
+++ b/manifests/server.pp
@@ -97,7 +97,7 @@ class backupninja::server {
 
     if $nagios_server {
       # configure a passive service check for backups
-      nagios2::passive_service { "backups-$real_host": nagios2_host_name => $real_host, nagios2_description => 'backups', servicegroups => "backups" }
+      nagios2::passive_service { "backups-$name": nagios2_host_name => $real_host, nagios2_description => 'backups', servicegroups => "backups" }
     }
     
     if !defined(File["$real_dir"]) {
-- 
cgit v1.2.3


From 47733659ae1c79cd8249096455f4b0d863dcad75 Mon Sep 17 00:00:00 2001
From: Micah Anderson <micah@riseup.net>
Date: Tue, 5 Jan 2010 10:34:16 -0500
Subject: let the user override the nagios2_description, which is necessary for
 multiple backup defines on one host

---
 manifests/server.pp | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

(limited to 'manifests/server.pp')

diff --git a/manifests/server.pp b/manifests/server.pp
index 291ffed..a64e854 100644
--- 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"]) {
-- 
cgit v1.2.3


From 8766fcdd82cdb67eb4b09f4b2f3f959c0c0ec3d0 Mon Sep 17 00:00:00 2001
From: Micah Anderson <micah@riseup.net>
Date: Tue, 5 Jan 2010 21:48:19 -0500
Subject: enclose namevar in curly braces

---
 manifests/server.pp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'manifests/server.pp')

diff --git a/manifests/server.pp b/manifests/server.pp
index a64e854..a0bcbf2 100644
--- a/manifests/server.pp
+++ b/manifests/server.pp
@@ -102,7 +102,7 @@ class backupninja::server {
 
     if $nagios_server {
       # configure a passive service check for backups
-      nagios2::passive_service { "backups-$name": nagios2_host_name => $real_host, nagios2_description => $real_nagios2_description, servicegroups => "backups" }
+      nagios2::passive_service { "backups-${name}": nagios2_host_name => $real_host, nagios2_description => $real_nagios2_description, servicegroups => "backups" }
     }
     
     if !defined(File["$real_dir"]) {
-- 
cgit v1.2.3