kennel 1.63.0 → 1.66.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7b8b30272574c9e7ad071340fefe2a6f020d2d00e499343cfa416322c1af91ea
-  data.tar.gz: b70c3cbfa377a9bbf023d3ada61c156519410c082702a8cd4693b95132198654
+  metadata.gz: 5d5c7df715aecb3223069ef066f070a0b78dd699c46feae0052e90facc2ef0f5
+  data.tar.gz: d88d4f7b8ff2ec7458490fd79cf269bc319abf589c17ace29a4d368a29626b2d
 SHA512:
-  metadata.gz: d9b8457368d955fc88a30608d746381d2bcbecac9997c6f9bbf650f4d7ec3487950b414a0876d7d72e3f349144ef221d1117942f9042e25e03fdc5f53e351ece
-  data.tar.gz: 57ce23d124ec7a8f2a6d2ff85e1adb9cd5725c19e80ba53130545924d265b6d8bedf89044f7c5e89ee49d0f060c365131b6ef7c912fe29b060dd39f6533f9eab
+  metadata.gz: f7a603b340355a6be34c57d7843729dbdf9d27f910d94511ab4dc6c5a71c32a477160ebfe188fc03675bc28261dc9beb82f27faaceb4be355aceba343bbf5c54
+  data.tar.gz: e320fdfb09fc4f2400cdcad2f16297e6715dbb9094959e8fd50784bc7b1a4b880e65de806caaaa1b3e016ce90a4d8217e1c3e174b757cf6d83895273e5652b80

data/Readme.md

@@ -184,6 +184,14 @@ Run `rake kennel:alerts TAG=service:my-service` to see all un-muted alerts for a
 
 `rake kennel:validate_mentions` should run as part of CI
 
+### Grepping through all of datadog
+
+`TYPE=monitor rake kennel:dump`
+
+### Find all monitors with No-Data
+
+`rake kennel:nodata TAG=team:foo`
+
 ## Examples
 
 ### Reusable monitors/dashes/etc
@@ -220,10 +228,10 @@ end
 ```Bash
 rake play
 cd template
-rake kennel:plan
+rake plan
 ```
 
-Then make changes to play around, do not commit changes and make sure to revert with a `rake kennel:update` after deleting everything.
+Then make changes to play around, do not commit changes and make sure to revert with a `rake kennel:update_datadog` after deleting everything.
 
 To make changes via the UI, make a new free datadog account and use it's credentaisl instead.
 

data/lib/kennel/api.rb

@@ -43,12 +43,12 @@ module Kennel
     end
 
     def delete(api_resource, id)
-      request :delete, "/api/v1/#{api_resource}/#{id}"
+      request :delete, "/api/v1/#{api_resource}/#{id}", ignore_404: true
     end
 
     private
 
-    def request(method, path, body: nil, params: {})
+    def request(method, path, body: nil, params: {}, ignore_404: false)
       params = params.merge(application_key: @app_key, api_key: @api_key)
       query = Faraday::FlatParamsEncoder.encode(params)
       response = nil
@@ -66,7 +66,7 @@ module Kennel
         Kennel.err.puts "Retrying on server error #{response.status} for #{path}"
       end
 
-      unless response.success?
+      if !response.success? && (response.status != 404 || !ignore_404)
         message = +"Error #{response.status} during #{method.upcase} #{path}\n"
         message << "request:\n#{JSON.pretty_generate(body)}\nresponse:\n" if body
         message << response.body

data/lib/kennel/github_reporter.rb

@@ -7,13 +7,13 @@ module Kennel
     class << self
       def report(token, &block)
         return yield unless token
-        new(token).report(&block)
+        new(token, Utils.capture_sh("git rev-parse HEAD").strip).report(&block)
       end
     end
 
-    def initialize(token)
+    def initialize(token, git_sha)
       @token = token
-      @git_sha = Utils.capture_sh("git rev-parse HEAD").strip
+      @git_sha = git_sha
       origin = ENV["PROJECT_REPOSITORY"] || Utils.capture_sh("git remote -v").split("\n").first
       @repo_part = origin[%r{github\.com[:/](.+?)(\.git|$)}, 1] || raise("no origin found")
     end
@@ -27,8 +27,6 @@ module Kennel
       comment "```\n#{output || "Error"}\n```"
     end
 
-    private
-
     # https://developer.github.com/v3/repos/comments/#create-a-commit-comment
     def comment(body)
       # truncate to maximum allowed comment size for github to avoid 422
@@ -39,6 +37,8 @@ module Kennel
       post "commits/#{@git_sha}/comments", body: body
     end
 
+    private
+
     def post(path, data)
       url = "https://api.github.com/repos/#{@repo_part}/#{path}"
       response = Faraday.post(url, data.to_json, authorization: "token #{@token}")

data/lib/kennel/models/monitor.rb

@@ -5,7 +5,8 @@ module Kennel
       include OptionalValidations
 
       RENOTIFY_INTERVALS = [0, 10, 20, 30, 40, 50, 60, 90, 120, 180, 240, 300, 360, 720, 1440].freeze # minutes
-      QUERY_INTERVALS = ["1m", "5m", "10m", "15m", "30m", "1h", "2h", "4h", "1d"].freeze
+      # 2d and 1w are valid timeframes for anomaly monitors
+      QUERY_INTERVALS = ["1m", "5m", "10m", "15m", "30m", "1h", "2h", "4h", "1d", "2d", "1w"].freeze
       OPTIONAL_SERVICE_CHECK_THRESHOLDS = [:ok, :warning].freeze
       READONLY_ATTRIBUTES = superclass::READONLY_ATTRIBUTES + [
         :multi, :matching_downtimes, :overall_state_modified, :overall_state, :restricted_roles
@@ -132,7 +133,7 @@ module Kennel
 
         case actual[:type]
         when "event alert"
-          # setting 0 results in thresholds not getting returned from the api
+          # setting nothing results in thresholds not getting returned from the api
           options[:thresholds] ||= { critical: 0 }
 
         when "service check"

data/lib/kennel/models/record.rb

@@ -42,7 +42,10 @@ module Kennel
 
         self.class.send(:normalize, expected, actual)
 
-        HashDiff.diff(actual, expected, use_lcs: false)
+        # strict: ignore Integer vs Float
+        # similarity: show diff when not 100% similar
+        # use_lcs: saner output
+        Hashdiff.diff(actual, expected, use_lcs: false, strict: false, similarity: 1)
       end
 
       def tracking_id

data/lib/kennel/syncer.rb

@@ -3,6 +3,7 @@ module Kennel
   class Syncer
     CACHE_FILE = "tmp/cache/details" # keep in sync with .travis.yml caching
     TRACKING_FIELDS = [:message, :description].freeze
+    DELETE_ORDER = ["dashboard", "slo", "monitor"].freeze # dashboards references monitors + slos, slos reference monitors
 
     def initialize(api, expected, project: nil)
       @api = api
@@ -98,6 +99,8 @@ module Kennel
         ensure_all_ids_found
         @create = @expected.map { |e| [nil, e] }
       end
+
+      @delete.sort_by! { |_, _, a| DELETE_ORDER.index a.fetch(:api_resource) }
     end
 
     # Make diff work even though we cannot mass-fetch definitions

data/lib/kennel/tasks.rb

@@ -111,14 +111,21 @@ namespace :kennel do
     end
   end
 
-  desc "Convert existing resources to copy-pastable definitions to import existing resources RESOURCE=dash ID=1234"
+  desc "Convert existing resources to copy-pastable definitions to import existing resources RESOURCE=[dashboard,monitor,slo] ID=1234"
   task import: :environment do
-    resource = ENV["RESOURCE"] || Kennel::Tasks.abort("Call with RESOURCE=dash") # TODO: add others
+    resource = ENV["RESOURCE"] || Kennel::Tasks.abort("Call with RESOURCE=dashboard or monitor or slo")
     id = ENV["ID"] || Kennel::Tasks.abort("Call with ID=1234")
     id = Integer(id) if id =~ /^\d+$/ # dashboards can have alphanumeric ids
     puts Kennel::Importer.new(Kennel.send(:api)).import(resource, id)
   end
 
+  desc "Dump ALL of datadog config as raw json ... useful for grep/search TYPE=slo|monitor|dashboard"
+  task dump: :environment do
+    Kennel.send(:api).list(ENV.fetch("TYPE")).each do |r|
+      Kennel.out.puts JSON.pretty_generate(r)
+    end
+  end
+
   task :environment do
     require "kennel"
     gem "dotenv"

data/lib/kennel/version.rb

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

data/template/Readme.md

@@ -166,6 +166,14 @@ Run `rake kennel:alerts TAG=service:my-service` to see all un-muted alerts for a
 
 `rake kennel:validate_mentions` should run as part of CI
 
+### Grepping through all of datadog
+
+`TYPE=monitor rake kennel:dump`
+
+### Find all monitors with No-Data
+
+`rake kennel:nodata TAG=team:foo`
+
 ## Examples
 
 ### Reusable monitors/dashes/etc

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: kennel
 version: !ruby/object:Gem::Version
-  version: 1.63.0
+  version: 1.66.2
 platform: ruby
 authors:
 - Michael Grosser
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-02-13 00:00:00.000000000 Z
+date: 2020-02-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday
@@ -30,14 +30,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.3.9
+        version: '1.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.3.9
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: net-http-persistent-retry
   requirement: !ruby/object:Gem::Requirement