X-Git-Url: https://git.adam-barratt.org.uk/?a=blobdiff_plain;f=modules%2Fmunin%2Fmanifests%2Fcheck.pp;h=0ed10ea574d20ee983f0d68196e9087163978b67;hb=9e27d3f0aaa21b93d9d256414dcd6335b07c0f53;hp=ace6bb52e08d2583776eae0c5cf056314388cfa9;hpb=175c9bc26de888ba32a7677aa393621425d595f5;p=mirror%2Fdsa-puppet.git

diff --git a/modules/munin/manifests/check.pp b/modules/munin/manifests/check.pp
index ace6bb52e..0ed10ea57 100644
--- a/modules/munin/manifests/check.pp
+++ b/modules/munin/manifests/check.pp
@@ -1,22 +1,21 @@
-define munin::check($ensure = present, $script = undef) {
+# enable (or disable) a munin check
+# @param ensure check enabled/disabled
+# @param script check to synclink
+define munin::check(
+  Enum['present','absent'] $ensure = 'present',
+  String $script = $name
+) {
+  include munin
 
-	include munin
+  $link_target = $ensure ? {
+    present => link,
+    absent  => absent,
+  }
 
-	if $script {
-		$link = $script
-	} else {
-		$link = $name
-	}
-
-	$link_target = $ensure ? {
-		present => "/usr/share/munin/plugins/${link}",
-		absent  => absent,
-		default => fail ( "Unknown ensure value: '$ensure'" ),
-	}
-
-	file { "/etc/munin/plugins/${name}":
-		ensure  => $link_target,
-		require => Package['munin-node'],
-		notify  => Service['munin-node'],
-	}
+  file { "/etc/munin/plugins/${name}":
+    ensure  => $link_target,
+    target  => "/usr/share/munin/plugins/${script}",
+    require => Package['munin-node'],
+    notify  => Service['munin-node'],
+  }
 }