kennel 1.112.0 → 1.113.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d0daaf865eea8c23b97ab763827ecf7a7addbe7cc3f8683d9c2ec0ba2b5115d5
-  data.tar.gz: ffbffb714491fae59ef58738fb90dea0533571b1694a372cfd99269a9c74c321
+  metadata.gz: b199d98b445f146f6e5ac3a2746d715c4286a75da4ddd9cf2268875e41a4f7f4
+  data.tar.gz: cc748229520b855819210413905525b962b7ea21ff13053258a011f59abaf001
 SHA512:
-  metadata.gz: d59cbb2e220bfc906d3842618704ddd09ed1636eab7923985efd75d5631b9c8a065f5026ec74a77b4da4112190b8798675aa538a6b97a2aecf96d0c90611a532
-  data.tar.gz: 7e55958dc660217e230f5b1670c890d81619b042b181d29f68288115a4f05cf5d7aff9571aabf8937886379c30cb74188d139bbdbc9d7420e3f27c5faad939ec
+  metadata.gz: a26a733f32e1b0c167d950d4da1c8deb69717306e6428636e2b5ab03b450eb6db6f6797eb3a054a66083b91d351a6ca875703255cea2adec62f6022452d1e90c
+  data.tar.gz: cedb46594f03b8e2ebf89915f4bd8945f0378b51808e36460b5af8c7b44266e1661de771acb6febaffa2ca1226b88a533fccbea870e47e025b30677b7f91f5a4

data/Readme.md

@@ -307,7 +307,8 @@ https://foo.datadog.com/monitor/123
 ## Development
 
 ### Benchmarking
-Setting `FORCE_GET_CACHE=true` will cache all get requests, which makes benchmarking improvements more reliable.
+- Setting `FORCE_GET_CACHE=true` will cache all get requests, which makes benchmarking improvements more reliable.
+- Setting `STORE=false` will make `rake plan` not update the files on disk and save a bit of time
 
 ### Integration testing
 ```Bash

data/lib/kennel/models/record.rb

@@ -109,11 +109,8 @@ module Kennel
       private
 
       def resolve(value, type, id_map, force:)
-        if tracking_id?(value)
-          return resolve_link(value, type, id_map, force: force)
-        end
-
-        value
+        return value unless tracking_id?(value)
+        resolve_link(value, type, id_map, force: force)
       end
 
       def tracking_id?(id)

data/lib/kennel/models/slo.rb

@@ -68,8 +68,8 @@ module Kennel
       end
 
       def resolve_linked_tracking_ids!(id_map, **args)
-        return unless as_json[:monitor_ids] # ignore_default can remove it
-        as_json[:monitor_ids] = as_json[:monitor_ids].map do |id|
+        return unless ids = as_json[:monitor_ids] # ignore_default can remove it
+        as_json[:monitor_ids] = ids.map do |id|
           resolve(id, :monitor, id_map, **args) || id
         end
       end

data/lib/kennel/syncer.rb

@@ -7,7 +7,7 @@ module Kennel
     def initialize(api, expected, project_filter: nil)
       @api = api
       @project_filter = project_filter
-      @expected = expected
+      @expected = Set.new expected # need set to speed up deletion
       calculate_diff
       validate_plan
       prevent_irreversible_partial_updates
@@ -110,9 +110,10 @@ module Kennel
 
         @expected.each(&:add_tracking_id) # avoid diff with actual
 
+        lookup_map = matching_expected_lookup_map
         items = actual.map do |a|
-          e = matching_expected(a)
-          if e && @expected.delete(e)
+          e = matching_expected(a, lookup_map)
+          if e && @expected.delete?(e)
             [e, a]
           else
             [nil, a]
@@ -127,7 +128,7 @@ module Kennel
         items.each do |e, a|
           id = a.fetch(:id)
           if e
-            diff = e.diff(a)
+            diff = e.diff(a) # slow ...
             @update << [id, e, a, diff] if diff.any?
           elsif a.fetch(:tracking_id) # was previously managed
             @delete << [id, nil, a]
@@ -166,9 +167,9 @@ module Kennel
       end
     end
 
-    def matching_expected(a)
-      # index list by all the thing we look up by: tracking id and actual id
-      @lookup_map ||= @expected.each_with_object({}) do |e, all|
+    # index list by all the thing we look up by: tracking id and actual id
+    def matching_expected_lookup_map
+      @expected.each_with_object({}) do |e, all|
         keys = [e.tracking_id]
         keys << "#{e.class.api_resource}:#{e.id}" if e.id
         keys.compact.each do |key|
@@ -176,9 +177,11 @@ module Kennel
           all[key] = e
         end
       end
+    end
 
+    def matching_expected(a, map)
       klass = a.fetch(:klass)
-      @lookup_map["#{klass.api_resource}:#{a.fetch(:id)}"] || @lookup_map[a.fetch(:tracking_id)]
+      map["#{klass.api_resource}:#{a.fetch(:id)}"] || map[a.fetch(:tracking_id)]
     end
 
     def print_plan(step, list, color)

data/lib/kennel/utils.rb

@@ -149,7 +149,7 @@ module Kennel
       # TODO: use awesome-print or similar, but it has too many monkey-patches
       # https://github.com/amazing-print/amazing_print/issues/36
       def pretty_inspect(object)
-        string = object.inspect
+        string = object.inspect.dup
         string.gsub!(/:([a-z_]+)=>/, "\\1: ")
         10.times do
           string.gsub!(/{(\S.*?\S)}/, "{ \\1 }") || break

data/lib/kennel/version.rb

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

data/lib/kennel.rb

@@ -47,7 +47,9 @@ module Kennel
     attr_accessor :out, :err, :strict_imports
 
     def generate
-      store generated
+      out = generated
+      store out if ENV["STORE"] != "false" # quicker when debugging
+      out
     end
 
     def plan
@@ -127,6 +129,11 @@ module Kennel
               use a different `kennel_id` when defining multiple projects/monitors/dashboards to avoid this conflict
             ERROR
           end
+
+          # trigger json caching here so it counts into generating
+          # somehow threading helps reduce this ~25%
+          Utils.parallel(parts, &:as_json)
+
           parts
         end
       end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: kennel
 version: !ruby/object:Gem::Version
-  version: 1.112.0
+  version: 1.113.0
 platform: ruby
 authors:
 - Michael Grosser
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-04-22 00:00:00.000000000 Z
+date: 2022-05-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday
@@ -117,7 +117,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.16
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: Keep datadog monitors/dashboards/etc in version control, avoid chaotic management