kennel 1.60.1 → 1.61.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d555de2128376fcfe551b2e7d95dfb724c819adc9f5b401ec155ba2d278607cb
-  data.tar.gz: bfa004cefabf58c7328118878e0cc1ffc10253098a6176b53ce3ae295adc41c6
+  metadata.gz: b27c8e5ce2f9b7ec5b649d582ff591bd05b2c874fdbf21207db795d3cc5b6c26
+  data.tar.gz: a1385e625b05f60d80d9cb5b6ca89a97cd9099187bb24896fee4e1bdcdcf51b3
 SHA512:
-  metadata.gz: c02a3b3308a8149e8efb2f014156017dc8e1cf8a5ad06507939f69f807c463f295554cddf48affd07706d0699e5df1a965fb44db2a8cb3459dd97401f511d70d
-  data.tar.gz: 06e3f9c53a1f2d9b7646a0ebc556eb6970957ff700ddcd448869844e46e0fa4564ae91bbc6ab57f5dd7eb713bce351171fbd038399d24de80a177dc7d8e5dee8
+  metadata.gz: 32273b702f893078015f99518c2798b3aa30dbd81abc68802ff35765ee489a2a16d0ad5f270c2d824c3d54fd44d4fba48d554486c19683a113f3a9e87f6ce1cd
+  data.tar.gz: 0f3c9290832283f5d1ae08865b926e84c7ba34fa8c79ebb01ddad91dad16eb592e54b9e246e432a00d7c2334908180b31cb2d343ce719a75c39b9cb4cf691ce2

data/lib/kennel/models/dashboard.rb

@@ -13,6 +13,7 @@ module Kennel
       REQUEST_DEFAULTS = {
         style: { line_width: "normal", palette: "dog_classic", line_type: "solid" }
       }.freeze
+      TIMESERIES_DEFAULTS = { show_legend: false, legend_size: "0" }.freeze
       SUPPORTED_DEFINITION_OPTIONS = [:events, :markers, :precision].freeze
 
       settings :title, :description, :definitions, :widgets, :layout_type
@@ -32,39 +33,68 @@ module Kennel
         def normalize(expected, actual)
           super
 
-          base_pairs(expected, actual).each do |pair|
+          widgets_pairs(expected, actual).each do |pair|
             # datadog always adds 2 to slo widget height
             # need to check fir layout since some monitors have height/width in their definition
-            pair.dig(1, :widgets)&.each do |widget|
+            pair[1].each do |widget|
               if widget.dig(:definition, :type) == "slo" && widget.dig(:layout, :height)
                 widget[:layout][:height] -= 2
               end
             end
 
             # conditional_formats ordering is randomly changed by datadog, compare a stable ordering
-            pair.each do |b|
-              b[:widgets]&.each do |w|
-                if formats = w.dig(:definition, :conditional_formats)
-                  w[:definition][:conditional_formats] = formats.sort_by(&:hash)
+            pair.each do |widgets|
+              widgets.each do |widget|
+                if formats = widget.dig(:definition, :conditional_formats)
+                  widget[:definition][:conditional_formats] = formats.sort_by(&:hash)
                 end
               end
             end
 
-            ignore_request_defaults(*pair, :widgets, :definition)
-            pair.each { |dash| dash[:widgets]&.each { |w| w.delete(:id) } }
+            ignore_definition_defaults_for_type pair, "timeseries", TIMESERIES_DEFAULTS
+
+            ignore_request_defaults(*pair)
+
+            # ids are kinda random so we always discard them
+            pair.each { |widgets| widgets.each { |w| w.delete(:id) } }
           end
         end
 
         private
 
+        def ignore_definition_defaults_for_type(pair, type, defaults)
+          pair.map(&:size).max.times do |i|
+            if pair.all? { |w| w.dig(i, :definition, :type) == type }
+              ignore_defaults(pair[0], pair[1], defaults, nesting: :definition)
+            end
+          end
+        end
+
+        # discard styles/conditional_formats/aggregator if nothing would change when we applied (both are default or nil)
+        def ignore_request_defaults(expected, actual)
+          [expected.size, actual.size].max.times do |i|
+            a_r = actual.dig(i, :definition, :requests) || []
+            e_r = expected.dig(i, :definition, :requests) || []
+            ignore_defaults e_r, a_r, REQUEST_DEFAULTS
+          end
+        end
+
+        def ignore_defaults(expected, actual, defaults, nesting: nil)
+          [expected.size, actual.size].max.times do |i|
+            e = expected.dig(i, *nesting) || {}
+            a = actual.dig(i, *nesting) || {}
+            ignore_default(e, a, defaults)
+          end
+        end
+
         # expand nested widgets into expected/actual pairs for default resolution
-        # [a, e] -> [[a, e], [aw1, ew1], ...]
-        def base_pairs(*pair)
-          result = [pair]
-          slots = pair.map { |d| d[:widgets]&.size }.compact.max.to_i
+        # [a, e] -> [[a-w, e-w], [a-w1-w1, e-w1-w1], ...]
+        def widgets_pairs(*pair)
+          result = [pair.map { |d| d[:widgets] || [] }]
+          slots = result.compact.map(&:size).max.to_i
           slots.times do |i|
-            nested = pair.map { |d| d.dig(:widgets, i, :definition) || {} }
-            result << nested if nested.any? { |d| d.key?(:widgets) }
+            nested = pair.map { |d| d.dig(:widgets, i, :definition, :widgets) || [] }
+            result << nested if nested.any?(&:any?)
           end
           result
         end

data/lib/kennel/models/record.rb

@@ -6,11 +6,6 @@ module Kennel
       READONLY_ATTRIBUTES = [
         :deleted, :id, :created, :created_at, :creator, :org_id, :modified, :modified_at, :api_resource
       ].freeze
-      REQUEST_DEFAULTS = {
-        style: { width: "normal", palette: "dog_classic", type: "solid" },
-        conditional_formats: [],
-        aggregator: "avg"
-      }.freeze
       API_LIST_INCOMPLETE = false
 
       settings :id, :kennel_id
@@ -22,25 +17,6 @@ module Kennel
           self::READONLY_ATTRIBUTES.each { |k| actual.delete k }
         end
 
-        # discard styles/conditional_formats/aggregator if nothing would change when we applied (both are default or nil)
-        def ignore_request_defaults(expected, actual, level1, level2)
-          actual = actual[level1] || {}
-          expected = expected[level1] || {}
-          [expected.size.to_i, actual.size.to_i].max.times do |i|
-            a_r = actual.dig(i, level2, :requests) || []
-            e_r = expected.dig(i, level2, :requests) || []
-            ignore_defaults e_r, a_r, self::REQUEST_DEFAULTS
-          end
-        end
-
-        def ignore_defaults(expected, actual, defaults)
-          [expected&.size.to_i, actual&.size.to_i].max.times do |i|
-            e = expected[i] || {}
-            a = actual[i] || {}
-            ignore_default(e, a, defaults)
-          end
-        end
-
         def ignore_default(expected, actual, defaults)
           definitions = [actual, expected]
           defaults.each do |key, default|

data/lib/kennel/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: kennel
 version: !ruby/object:Gem::Version
-  version: 1.60.1
+  version: 1.61.0
 platform: ruby
 authors:
 - Michael Grosser
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-12-18 00:00:00.000000000 Z
+date: 2019-12-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday