kennel 2.10.0 → 2.11.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9213ac86e78dd85ff280d2048bd20fe3d074c3d6218cc45021b8c11eee7c720c
-  data.tar.gz: b2506ab43bb450a0496db0985ffa990b378ad591fa7be2a1c8db32223863680e
+  metadata.gz: 88495b67d113181455cb6dc751d2f7820eababca5679cfb714bb614dc9370ad6
+  data.tar.gz: a8703613913aa4557c585e605f35a1aef9555902c65a5f1c1772b1038e9d4d34
 SHA512:
-  metadata.gz: 73331755a7074cd0eeb95c92901cb8bf477380672a553cd7ef4722ec4cb6ea6f2fd56634eef442fb4a59de961d102c7b8b0f5b39788d006c24d960573eb27500
-  data.tar.gz: fa9ee34a57cce815f63eba43208daac2232cd04456f1200adc4181e6a381407b93962d52cbc5f9b27398caa863c8ab371758f5c216613b0218f8b9e958de6bc5
+  metadata.gz: d730ee50ff704f25eb39fdd8daa9c95a2c0b43f9a9534c7f1a05a245bd3881ab5ce36f12ebf226354eeafe94c4a961e843eebbbcf0312b2b548577b26a4debdd
+  data.tar.gz: ff487e51479e7a463464e37ec09a10ef7c1dc7d695bcf18f97303048ab0a75759d71eda14aa95e512aa69b66c25baf016d778be449455c0ef069e081f5957373

data/lib/kennel/models/monitor.rb

@@ -6,7 +6,8 @@ module Kennel
 
       OPTIONAL_SERVICE_CHECK_THRESHOLDS = [:ok, :warning].freeze
       READONLY_ATTRIBUTES = superclass::READONLY_ATTRIBUTES + [
-        :multi, :matching_downtimes, :overall_state_modified, :overall_state, :restricted_roles, :draft_status, :assets
+        :multi, :matching_downtimes, :overall_state_modified, :overall_state, :restricted_roles, :draft_status, :assets,
+        :enable_logs_sample
       ]
       TRACKING_FIELD = :message
 
@@ -25,7 +26,7 @@ module Kennel
         group_retention_duration: nil,
         groupby_simple_monitor: false,
         variables: nil,
-        on_missing_data: "default", # "default" is "evaluate as zero"
+        on_missing_data: nil,
         notification_preset_name: nil,
         notify_by: nil
       }.freeze
@@ -74,6 +75,8 @@ module Kennel
       )
 
       def build_json
+        no_data_options = configure_no_data
+
         data = super.merge(
           name: "#{name}#{LOCK}",
           type: type,
@@ -83,8 +86,7 @@ module Kennel
           priority: priority,
           options: {
             timeout_h: timeout_h,
-            notify_no_data: notify_no_data,
-            no_data_timeframe: notify_no_data ? no_data_timeframe : nil,
+            **no_data_options.except(:on_missing_data),
             notify_audit: notify_audit,
             require_full_window: require_full_window,
             new_host_delay: new_host_delay,
@@ -92,33 +94,14 @@ module Kennel
             include_tags: true,
             escalation_message: Utils.presence(escalation_message.strip),
             evaluation_delay: evaluation_delay,
-            locked: false, # deprecated: setting this to true will likely fail
             renotify_interval: renotify_interval || 0,
-            variables: variables
+            variables: variables,
+            **configure_thresholds,
+            **no_data_options.slice(:on_missing_data) # moved here to avoid generated diff
           }
         )
 
         options = data[:options]
-        if data.fetch(:type) != "composite"
-          thresholds = (options[:thresholds] = { critical: critical })
-
-          # warning, ok, critical_recovery, and warning_recovery are optional
-          [:warning, :ok, :critical_recovery, :warning_recovery].each do |key|
-            if (value = send(key))
-              thresholds[key] = value
-            end
-          end
-
-          thresholds[:critical] = critical unless
-          case data.fetch(:type)
-          when "service check"
-            # avoid diff for default values of 1
-            OPTIONAL_SERVICE_CHECK_THRESHOLDS.each { |t| thresholds[t] ||= 1 }
-          when "query alert"
-            # metric and query values are stored as float by datadog
-            thresholds.each { |k, v| thresholds[k] = Float(v) }
-          end
-        end
 
         # set without causing lots of nulls to be stored
         if (notify_by_value = notify_by)
@@ -149,30 +132,37 @@ module Kennel
         # Add in statuses where we would re notify on. Possible values: alert, no data, warn
         if options[:renotify_interval] != 0
           statuses = ["alert"]
-          statuses << "no data" if options[:notify_no_data]
+          statuses << "no data" if options[:notify_no_data] || options[:on_missing_data] == "show_and_notify_no_data"
           statuses << "warn" if options.dig(:thresholds, :warning)
           options[:renotify_statuses] = statuses
         end
 
-        # on_missing_data cannot be used with notify_no_data or no_data_timeframe
-        # TODO migrate everything to only use on_missing_data
-        if data.fetch(:type) == "event-v2 alert" || on_missing_data != "default"
-          options[:on_missing_data] = on_missing_data
-          options[:notify_no_data] = false # cannot set nil or it's an endless update loop
-          options.delete :no_data_timeframe
-        end
-
         # only set when needed to avoid big diff
         if (notification_preset_name = notification_preset_name())
           options[:notification_preset_name] = notification_preset_name
         end
 
-        # locked is deprecated, will fail if used
-        options.delete :locked
-
         data
       end
 
+      # TODO: migrate everything to only use on_missing_data by only sending notify_no_data when it was set by a user
+      # and enforce that it is not set at the same time as on_missing_data
+      def configure_no_data
+        notify = notify_no_data
+        action = on_missing_data
+
+        # 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
+          { on_missing_data: action || "default" }
+        else
+          {
+            notify_no_data: notify,
+            no_data_timeframe: notify ? no_data_timeframe : nil
+          }
+        end
+      end
+
       def resolve_linked_tracking_ids!(id_map, **args)
         case as_json[:type]
         when "composite", "slo alert"
@@ -206,7 +196,8 @@ module Kennel
       # 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)
+        sending_no_data_notifications = (on_missing_data ? on_missing_data == "show_and_notify_no_data" : notify_no_data)
+        sending_no_data_notifications ? 24 : MONITOR_OPTION_DEFAULTS.fetch(:timeout_h)
       end
 
       def self.api_resource
@@ -235,7 +226,9 @@ module Kennel
         ignore_default(expected, actual, MONITOR_DEFAULTS)
 
         options = actual.fetch(:options)
-        options.delete(:silenced) # we do not manage silenced, so ignore it when diffing
+
+        # we do not manage silenced: ignore it when diffing
+        options.delete(:silenced)
 
         # fields are not returned when set to true
         if ["service check", "event alert"].include?(actual[:type])
@@ -265,13 +258,38 @@ module Kennel
           options.delete(:escalation_message)
           expected_options.delete(:escalation_message)
         end
+
         # locked is deprecated: ignored when diffing
         options.delete(:locked)
-        expected_options.delete(:locked)
       end
 
       private
 
+      def configure_thresholds
+        return {} if type == "composite"
+
+        thresholds = { critical: critical }
+
+        # set optional variables
+        [:warning, :ok, :critical_recovery, :warning_recovery].each do |key|
+          if (value = send(key))
+            thresholds[key] = value
+          end
+        end
+
+        # custom logic for some types
+        case type
+        when "service check"
+          # avoid diff for default values of 1
+          OPTIONAL_SERVICE_CHECK_THRESHOLDS.each { |t| thresholds[t] ||= 1 }
+        when "query alert"
+          # metric and query values are stored as float by datadog
+          thresholds.each { |k, v| thresholds[k] = Float(v) }
+        end
+
+        { thresholds: thresholds }
+      end
+
       def validate_json(data)
         super
 

data/lib/kennel/version.rb

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

metadata

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