make sure booleans are booleans and fix filter dependency

sensu · Feb 6, 2014 · 396e7fe · 396e7fe
1 parent d6d6dee
commit 396e7fe
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 5 deletions.
diff --git a/lib/puppet/type/sensu_check.rb b/lib/puppet/type/sensu_check.rb
@@ -97,16 +97,22 @@ def insync?(is)
     desc "Check timeout in seconds, after it fails"
   end
 
-  newproperty(:aggregate) do
+  newproperty(:aggregate, :boolean => true) do
     desc "Whether check is aggregate"
+
+    newvalues(:true, :false)
   end
 
-  newproperty(:handle) do
+  newproperty(:handle, :boolean => true) do
     desc "Whether check event send to a handler"
+
+    newvalues(:true, :false)
   end
 
-  newproperty(:publish) do
+  newproperty(:publish, :boolean => true) do
     desc "Whether check is unpublished"
+
+    newvalues(:true, :false)
   end
 
   autorequire(:package) do

diff --git a/lib/puppet/type/sensu_filter.rb b/lib/puppet/type/sensu_filter.rb
@@ -56,8 +56,10 @@ def insync?(is)
     defaultto {}
   end
 
-  newproperty(:negate) do
+  newproperty(:negate, :boolean => true) do
     desc ""
+
+    newvalues(:true, :false)
   end
 
   autorequire(:package) do

diff --git a/manifests/filter.pp b/manifests/filter.pp
@@ -38,7 +38,7 @@
     owner   => 'sensu',
     group   => 'sensu',
     mode    => '0444',
-    before  => Sensu_client_subscription[$name],
+    before  => Sensu_filter[$name],
   }
 
   sensu_filter { $name: