kennel 1.58.3 → 1.60.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d6eb8595b53bd765cb23745ba69474537f573d5359eec2d381e21bfba4d906c6
-  data.tar.gz: 57868385f03594a028c6140389be0f92b5899ec6953cd0d4a7f504122da2301d
+  metadata.gz: f6b4a0efd1cc9562ad3fe039254ba48b197a392556105b3d56f66985105a8806
+  data.tar.gz: 8277eccfa9f48eec9ee8fe6dcf516fd4ad1e36c5760ec850dc4d60d71714ddcd
 SHA512:
-  metadata.gz: 72304befa0ab9ad0a0ac92b0112d3a37f2b0176f7940d6167eb761be8aadec2de30c12942ad6b495dd39da35a2d47153b769e2d2b7be41f3f71635c7a1afeee2
-  data.tar.gz: f927605bda707be196eeb7ff18987f4fd6737d2ba6c619e331b8c12832079051663445477ff57ffe2bfd79ca5bf38851b79c379f90d136eba22d448252f1517f
+  metadata.gz: 405531a24f9f801592c687f4f2a2ca448785b8907e549eafde3d7ea02935db63aa010d955d0f0d86d50f75ae1f42344fcf5c7904500e5e6d049ba9cbb1b6483a
+  data.tar.gz: 6c1d454d24dd9b0fc313e0a208a52c2a8efc77258171aefc4df8a115beca66d681925095a4f4ac544792452d7e2e5c09557350a72f467cc38fcbfbceb699350d

data/lib/kennel.rb

@@ -29,6 +29,10 @@ require "kennel/models/project"
 require "kennel/models/team"
 
 module Kennel
+  MISSING_ID = 1
+  class ValidationError < RuntimeError
+  end
+
   @out = $stdout
   @err = $stderr
 
@@ -68,7 +72,7 @@ module Kennel
         Progress.progress "Generating" do
           load_all
           parts = Models::Project.recursive_subclasses.flat_map do |project_class|
-            project_class.new.parts
+            project_class.new.validated_parts
           end
           parts.map(&:tracking_id).group_by { |id| id }.select do |id, same|
             raise "#{id} is defined #{same.size} times" if same.size != 1

data/lib/kennel/api.rb

@@ -13,7 +13,24 @@ module Kennel
     end
 
     def list(api_resource, params = {})
-      request :get, "/api/v1/#{api_resource}", params: params
+      if api_resource == "slo"
+        raise ArgumentError if params[:limit] || params[:offset]
+        limit = 1000
+        offset = 0
+        all = []
+
+        loop do
+          result = request :get, "/api/v1/#{api_resource}", params: params.merge(limit: limit, offset: offset)
+          data = result.fetch(:data)
+          all.concat data
+          break all if data.size < limit
+          offset += limit
+        end
+      else
+        result = request :get, "/api/v1/#{api_resource}", params: params
+        result = result.fetch(:dashboards) if api_resource == "dashboard"
+        result
+      end
     end
 
     def create(api_resource, attributes)

data/lib/kennel/models/dashboard.rb

@@ -101,16 +101,16 @@ module Kennel
           when "uptime"
             if ids = definition[:monitor_ids]
               definition[:monitor_ids] = ids.map do |id|
-                tracking_id?(id) ? resolve_link(id, id_map, force: false) : id
+                tracking_id?(id) ? resolve_link(id, id_map) : id
               end
             end
           when "alert_graph"
             if (id = definition[:alert_id]) && tracking_id?(id)
-              definition[:alert_id] = resolve_link(id, id_map, force: false).to_s
+              definition[:alert_id] = resolve_link(id, id_map).to_s
             end
           when "slo"
             if (id = definition[:slo_id]) && tracking_id?(id)
-              definition[:slo_id] = resolve_link(id, id_map, force: false).to_s
+              definition[:slo_id] = resolve_link(id, id_map).to_s
             end
           end
         end

data/lib/kennel/models/monitor.rb

@@ -105,6 +105,7 @@ module Kennel
       def resolve_linked_tracking_ids(id_map)
         if as_json[:type] == "composite"
           as_json[:query] = as_json[:query].gsub(/%\{(.*?)\}/) do
+            # need force here since it validates the id exists
             resolve_link($1, id_map, force: true)
           end
         end

data/lib/kennel/models/project.rb

@@ -14,6 +14,18 @@ module Kennel
           instance_method(method_in_file).source_location.first.sub("#{Bundler.root}/", "")
         end
       end
+
+      def validated_parts
+        all = parts
+        validate_parts(all)
+        all
+      end
+
+      private
+
+      # hook for users to add custom validations via `prepend`
+      def validate_parts(parts)
+      end
     end
   end
 end

data/lib/kennel/models/record.rb

@@ -13,9 +13,6 @@ module Kennel
       }.freeze
       API_LIST_INCOMPLETE = false
 
-      class ValidationError < RuntimeError
-      end
-
       settings :id, :kennel_id
 
       class << self
@@ -81,10 +78,19 @@ module Kennel
 
       private
 
-      def resolve_link(id, id_map, force:)
-        id_map[id] || begin
-          message = "Unable to find #{id} in existing monitors (they need to be created first to link them)"
-          force ? invalid!(message) : Kennel.err.puts(message)
+      def resolve_link(id, id_map, force: false)
+        found = id_map[id]
+        return found if found && found != :new
+
+        if found == :new
+          if force
+            invalid! "Monitor #{id} will be created in the current run and can only be used after that"
+          else
+            Kennel.err.puts "Monitor #{id} will be created in the current run, the next run will link it properly"
+            Kennel::MISSING_ID
+          end
+        else
+          invalid! "Unable to find monitor #{id} (does not exist and is not being created by the current run)"
         end
       end
 

data/lib/kennel/models/slo.rb

@@ -35,7 +35,7 @@ module Kennel
           description: description,
           thresholds: thresholds,
           monitor_ids: monitor_ids,
-          tags: tags,
+          tags: tags.uniq,
           type: type
         }
 
@@ -55,7 +55,7 @@ module Kennel
 
       def resolve_linked_tracking_ids(id_map)
         as_json[:monitor_ids] = as_json[:monitor_ids].map do |id|
-          id.is_a?(String) ? resolve_link(id, id_map, force: false) || 1 : id
+          id.is_a?(String) ? resolve_link(id, id_map) : id
         end
       end
 

data/lib/kennel/syncer.rb

@@ -38,7 +38,6 @@ module Kennel
     def update
       @create.each do |_, e|
         reply = @api.create e.class.api_resource, e.as_json
-        reply = unnest(e.class.api_resource, reply)
         Kennel.out.puts "Created #{e.class.api_resource} #{tracking_id(e.as_json)} #{e.url(reply.fetch(:id))}"
       end
 
@@ -101,21 +100,16 @@ module Kennel
       end
     end
 
-    # Hack to get diff to work until we can mass-fetch definitions
+    # Make diff work even though we cannot mass-fetch definitions
     def fill_details(a, cache)
       resource = a.fetch(:api_resource)
       args = [resource, a.fetch(:id)]
       full = cache.fetch(args, a[:modified] || a.fetch(:modified_at)) do
-        unnest(resource, @api.show(*args))
+        @api.show(*args)
       end
       a.merge!(full)
     end
 
-    # dashes are nested, others are not
-    def unnest(api_resource, result)
-      result[api_resource.to_sym] || result[:data] || result
-    end
-
     def details_cache(&block)
       cache = FileCache.new CACHE_FILE, Kennel::VERSION
       cache.open(&block)
@@ -195,6 +189,7 @@ module Kennel
 
     def resolve_linked_tracking_ids(actual)
       map = actual.each_with_object({}) { |a, lookup| lookup[tracking_id(a)] = a.fetch(:id) }
+      @expected.each { |e| map[e.tracking_id] ||= :new }
       @expected.each { |e| e.resolve_linked_tracking_ids(map) }
     end
 

data/lib/kennel/tasks.rb

@@ -18,7 +18,7 @@ end
 namespace :kennel do
   desc "Ensure there are no uncommited changes that would be hidden from PR reviewers"
   task no_diff: :generate do
-    result = `git status --porcelain`.strip
+    result = `git status --porcelain generated/`.strip
     Kennel::Tasks.abort "Diff found:\n#{result}\nrun `rake generate` and commit the diff to fix" unless result == ""
     Kennel::Tasks.abort "Error during diffing" unless $CHILD_STATUS.success?
   end
@@ -45,8 +45,7 @@ namespace :kennel do
     end
 
     if bad.any?
-      subdomain = ENV["DATADOG_SUBDOMAIN"]
-      url = (subdomain ? "https://zendesk.datadoghq.com" : "") + "/account/settings"
+      url = Kennel::Utils.path_to_url "/account/settings"
       puts "Invalid mentions found, either ignore them by adding to `KNOWN` env var or add them via #{url}"
       bad.each { |f, v| puts "Invalid mention #{v} in monitor message of #{f}" }
       Kennel::Tasks.abort

data/lib/kennel/version.rb

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

metadata

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