kennel 2.11.0 → 2.12.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 88495b67d113181455cb6dc751d2f7820eababca5679cfb714bb614dc9370ad6
-  data.tar.gz: a8703613913aa4557c585e605f35a1aef9555902c65a5f1c1772b1038e9d4d34
+  metadata.gz: 820cb0c4f746cbbafc79598199ba6c4e6d8d762e8505b432af9bf3c27de3ca74
+  data.tar.gz: d1c54c6ecd55bfd00e432b9258dfb6c6f16fa15c9181f862adca7bd885a0c4c2
 SHA512:
-  metadata.gz: d730ee50ff704f25eb39fdd8daa9c95a2c0b43f9a9534c7f1a05a245bd3881ab5ce36f12ebf226354eeafe94c4a961e843eebbbcf0312b2b548577b26a4debdd
-  data.tar.gz: ff487e51479e7a463464e37ec09a10ef7c1dc7d695bcf18f97303048ab0a75759d71eda14aa95e512aa69b66c25baf016d778be449455c0ef069e081f5957373
+  metadata.gz: e001bd09798e846ddabdd5dae8b47aa9174cdd19be8f4af77b52d2eb877ed2cb6043a2022c229f23e0c232700104c930902ec89ecb2db416e77de2e29074ef0b
+  data.tar.gz: bd930e26de4108718b8228056b726a5aeb41407ddef31175879b8126b250639e027b2d1e05cbabe47d5e94399aa8a8c609dddc3bb0163eeb920051d436186b1d

data/lib/kennel/models/monitor.rb

@@ -56,6 +56,7 @@ module Kennel
         # datadog UI sets this to false by default, but true is safer
         # except for log alerts which will always have "no error" gaps and should default to false
         notify_no_data: -> { !SKIP_NOTIFY_NO_DATA_TYPES.include?(type) },
+        no_data_timeframe: -> { MONITOR_OPTION_DEFAULTS.fetch(:no_data_timeframe) },
         notify_audit: -> { MONITOR_OPTION_DEFAULTS.fetch(:notify_audit) },
         new_host_delay: -> { MONITOR_OPTION_DEFAULTS.fetch(:new_host_delay) },
         new_group_delay: -> { nil },
@@ -149,16 +150,28 @@ module Kennel
       # and enforce that it is not set at the same time as on_missing_data
       def configure_no_data
         notify = notify_no_data
+        timeframe = no_data_timeframe
         action = on_missing_data
+        action ||= "default" if type == "event-v2 alert"
 
-        # on_missing_data cannot be used with notify_no_data or no_data_timeframe
-        if type == "event-v2 alert" || action
-          # TODO: mark setting notify_no_data or no_data_timeframe at all as invalid
+        # TODO: mark setting action && !notify.nil? at all as invalid
+        if action && timeframe
+          invalid! :invalid_no_data_config, "set either no_data_timeframe or on_missing_data"
+        end
+        if type == "composite" && action
+          invalid! :invalid_no_data_config, "cannot use on_missing_data with composite monitor"
+        end
+        if action == "resolve" && timeout_h.to_i != 0
+          invalid! :invalid_no_data_config, "timeout_h cannot be set and non-zero when on_missing_data is `resolve`"
+        end
+
+        # on_missing_data cannot be used with notify_no_data + no_data_timeframe
+        if action
           { on_missing_data: action || "default" }
         else
           {
             notify_no_data: notify,
-            no_data_timeframe: notify ? no_data_timeframe : nil
+            no_data_timeframe: notify ? no_data_timeframe || default_no_data_timeframe : nil
           }
         end
       end
@@ -184,7 +197,7 @@ module Kennel
       # deprecated this setting is no longer returned by dd for new monitors
       # datadog UI warns when setting no data timeframe to less than 2x the query window
       # limited to 24h because `no_data_timeframe must not exceed group retention` and max group retention is 24h
-      def no_data_timeframe
+      def default_no_data_timeframe
         default = 60
         if type == "query alert" && (minutes = query_window_minutes)
           (minutes * 2).clamp(default, 24 * 60)
@@ -305,9 +318,11 @@ module Kennel
         end
 
         # verify query includes critical value
-        if (query_value = data.fetch(:query)[/\s*[<>]=?\s*(\d+(\.\d+)?)\s*$/, 1])
-          if Float(query_value) != Float(data.dig(:options, :thresholds, :critical))
-            invalid! :critical_does_not_match_query, "critical and value used in query must match"
+        if (critical = data.dig(:options, :thresholds, :critical))
+          if (query_value = data.fetch(:query)[/\s*[<>]=?\s*(\d+(\.\d+)?)\s*$/, 1])
+            if Float(query_value) != Float(critical)
+              invalid! :critical_does_not_match_query, "critical and value used in query must match"
+            end
           end
         end
 

data/lib/kennel/version.rb

@@ -1,4 +1,4 @@
 # frozen_string_literal: true
 module Kennel
-  VERSION = "2.11.0"
+  VERSION = "2.12.1"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: kennel
 version: !ruby/object:Gem::Version
-  version: 2.11.0
+  version: 2.12.1
 platform: ruby
 authors:
 - Michael Grosser
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-01-09 00:00:00.000000000 Z
+date: 2026-01-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: diff-lcs