kennel 2.8.2 → 2.10.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ec30baa34fa9cd6ea5ada65125548347ec92fa0b97a3c1e58ea4bb590850bf48
-  data.tar.gz: 650ffef2c083b084a71e7c04083a2405be8d03da974e4c45374a56cfd5c3ac0b
+  metadata.gz: 9213ac86e78dd85ff280d2048bd20fe3d074c3d6218cc45021b8c11eee7c720c
+  data.tar.gz: b2506ab43bb450a0496db0985ffa990b378ad591fa7be2a1c8db32223863680e
 SHA512:
-  metadata.gz: 0053b7f8135a408f04db94cb055640c5dc5a928381e70736f41a5d79581fadd3071a1cdc1e7d3f8f6c867c972b18843204474d103b81377b5ed6b15183999aaa
-  data.tar.gz: e32878d2a099fb1067f5af7f06c7f09bf69b41f82b1f407ceb5edce8d3ab5a3c50da97d7ce8df1b86ae1b225cef7c9d23fb038bf1c09c9fd698d254b93bff480
+  metadata.gz: 73331755a7074cd0eeb95c92901cb8bf477380672a553cd7ef4722ec4cb6ea6f2fd56634eef442fb4a59de961d102c7b8b0f5b39788d006c24d960573eb27500
+  data.tar.gz: fa9ee34a57cce815f63eba43208daac2232cd04456f1200adc4181e6a381407b93962d52cbc5f9b27398caa863c8ab371758f5c216613b0218f8b9e958de6bc5

data/lib/kennel/api.rb

@@ -5,8 +5,6 @@ module Kennel
   class Api
     CACHE_FILE = ENV.fetch("KENNEL_API_CACHE_FILE", "tmp/cache/details")
 
-    RateLimitParams = Data.define(:limit, :period, :remaining, :reset, :name)
-
     def self.with_tracking(api_resource, reply)
       klass = Models::Record.api_resource_map[api_resource]
       return reply unless klass # do not blow up on unknown models
@@ -125,20 +123,8 @@ module Kennel
             end
           end
 
-          rate_limit = RateLimitParams.new(
-            limit: response.headers["x-ratelimit-limit"],
-            period: response.headers["x-ratelimit-period"],
-            remaining: response.headers["x-ratelimit-remaining"],
-            reset: response.headers["x-ratelimit-reset"],
-            name: response.headers["x-ratelimit-name"]
-          )
-
           if response.status == 429
-            message = "Datadog rate limit #{rate_limit.name.inspect} hit"
-            message += " (#{rate_limit.limit} requests per #{rate_limit.period} seconds)"
-            message += "; sleeping #{rate_limit.reset} seconds before trying again"
-            Kennel.err.puts message
-            sleep rate_limit.reset.to_f
+            sleep_until_rate_limit_resets(response)
             redo
           end
 
@@ -161,6 +147,20 @@ module Kennel
       end
     end
 
+    def sleep_until_rate_limit_resets(response)
+      limit = response.headers["x-ratelimit-limit"]
+      period = response.headers["x-ratelimit-period"]
+      reset = response.headers["x-ratelimit-reset"]
+      name = response.headers["x-ratelimit-name"]
+
+      message = "Datadog rate limit #{name.inspect} hit"
+      message += " (#{limit} requests per #{period} seconds)"
+      message += "; sleeping #{reset} seconds before trying again"
+
+      Kennel.err.puts message
+      sleep reset.to_f
+    end
+
     # allow caching all requests to speedup/benchmark logic that includes repeated requests
     def with_cache(enabled, key)
       return yield unless enabled

data/lib/kennel/models/monitor.rb

@@ -32,6 +32,12 @@ module Kennel
       DEFAULT_ESCALATION_MESSAGE = ["", nil].freeze
       ALLOWED_PRIORITY_CLASSES = [NilClass, Integer].freeze
       SKIP_NOTIFY_NO_DATA_TYPES = ["event alert", "event-v2 alert", "log alert"].freeze
+      MINUTES_PER_UNIT = {
+        "m" => 1,
+        "h" => 60,
+        "d" => 60 * 24,
+        "w" => 60 * 24 * 7
+      }.freeze
 
       settings(
         :query, :name, :message, :escalation_message, :critical, :type, :renotify_interval, :warning, :timeout_h, :evaluation_delay,
@@ -49,13 +55,11 @@ 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: -> { 60 },
         notify_audit: -> { MONITOR_OPTION_DEFAULTS.fetch(:notify_audit) },
         new_host_delay: -> { MONITOR_OPTION_DEFAULTS.fetch(:new_host_delay) },
         new_group_delay: -> { nil },
         group_retention_duration: -> { MONITOR_OPTION_DEFAULTS.fetch(:group_retention_duration) },
         tags: -> { @project.tags },
-        timeout_h: -> { MONITOR_OPTION_DEFAULTS.fetch(:timeout_h) },
         evaluation_delay: -> { MONITOR_OPTION_DEFAULTS.fetch(:evaluation_delay) },
         critical_recovery: -> { nil },
         warning_recovery: -> { nil },
@@ -187,6 +191,24 @@ module Kennel
         "cannot update type from #{actual_type} to #{type}"
       end
 
+      # 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
+        default = 60
+        if type == "query alert" && (minutes = query_window_minutes)
+          (minutes * 2).clamp(default, 24 * 60)
+        else
+          default
+        end
+      end
+
+      # validate that monitors that alert on no data resolve in external services by using timeout_h, so it sends a
+      # notification when the no data group is removed from the monitor, which datadog does automatically after 24h
+      def timeout_h
+        notify_no_data && on_missing_data != "resolve" ? 24 : MONITOR_OPTION_DEFAULTS.fetch(:timeout_h)
+      end
+
       def self.api_resource
         "monitor"
       end
@@ -372,6 +394,11 @@ module Kennel
         else # do nothing
         end
       end
+
+      def query_window_minutes
+        return unless (match = query.match(/^\s*\w+\(last_(?<count>\d+)(?<unit>[mhdw])\):/))
+        Integer(match["count"]) * MINUTES_PER_UNIT.fetch(match["unit"])
+      end
     end
   end
 end

data/lib/kennel/models/project.rb

@@ -11,7 +11,7 @@ module Kennel
       def self.file_location
         return @file_location if defined?(@file_location)
         if (location = instance_methods(false).first)
-          @file_location = instance_method(location).source_location.first.sub("#{Bundler.root}/", "")
+          @file_location = force_relative_path(instance_method(location).source_location.first)
         else
           @file_location = nil
         end
@@ -29,6 +29,13 @@ module Kennel
 
       private
 
+      private_class_method def self.force_relative_path(path)
+        return path unless File.absolute_path?(path)
+        path.dup.sub!("#{Bundler.root}/", "") ||
+        path.dup.sub!("#{Dir.pwd}/", "") ||
+        raise("Unable to make path #{path} relative with bundler root #{Bundler.root} or pwd #{Dir.pwd}")
+      end
+
       # hook for users to add custom filtering via `prepend`
       def filter_parts(parts)
         parts

data/lib/kennel/version.rb

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

metadata

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