kennel 1.91.3 → 1.92.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7a85c311024f48c14e9e346eb863a47fbb67239f5d83cd82eddea6f53742ec74
-  data.tar.gz: 672ef53f76ee577f9f976ee9432f06f1e56d426bef74020d50850863e851635a
+  metadata.gz: a3460bf3995d61817a7d7db0ad209012cb6cd51b1d5ba6767f4d20bc1766243f
+  data.tar.gz: 6a70c8e91c2bbf3e67db5f157e3c7f85d4025d2494d04026f9e7dc2007635217
 SHA512:
-  metadata.gz: 8407d6564013f12ce1e00e1394eb742b7f721fe9302396d821c8c3441c62e4e30781b9681f430dcedb96e3c757a740b9604a2c56194afabdabd69ed52f8c1a27
-  data.tar.gz: efceaa72bc4ff084e1d8bd54ed79dbcaa021d6a95897fa66465dbec686b0712f05a153198b3c882029df08812b93353ed3061c34c5f4c5f1303b826bc3d17711
+  metadata.gz: 6d3554ec738b404eb6ef6ff6e32120b86d3e65d399c7a5c0ff9c828e3beecf066fb441d941f077b9537c50347b1c5694b54d94e1c4f1d1ba19bacb5afcb1af57
+  data.tar.gz: d3eb584c0b15e04e38021324ce5e861740c2660f978cf9f413635a6a76fc8a4f82bba09a07343e12b0821f4b56f098fe39a8387013ffbb287065281b54bbeca5

data/lib/kennel/github_reporter.rb

@@ -8,13 +8,15 @@ module Kennel
     class << self
       def report(token, &block)
         return yield unless token
-        new(token, Utils.capture_sh("git rev-parse HEAD").strip).report(&block)
+        new(token).report(&block)
       end
     end
 
-    def initialize(token, git_sha)
+    def initialize(token, ref: "HEAD")
       @token = token
-      @git_sha = git_sha
+      commit = Utils.capture_sh("git show #{ref}")
+      @sha = commit[/^Merge: \S+ (\S+)/, 1] || commit[/\Acommit (\S+)/, 1] || raise("Unable to find commit")
+      @pr = commit[/^\s+.*\(#(\d+)\)/, 1] # from squash
       @repo_part = ENV["GITHUB_REPOSITORY"] || begin
         origin = ENV["PROJECT_REPOSITORY"] || Utils.capture_sh("git remote -v").split("\n").first
         origin[%r{github\.com[:/](\S+?)(\.git|$)}, 1] || raise("no origin found in #{origin}")
@@ -37,13 +39,14 @@ module Kennel
         body = body.byteslice(0, MAX_COMMENT_SIZE - TRUNCATED_MSG.bytesize) + TRUNCATED_MSG
       end
 
-      post "commits/#{@git_sha}/comments", body: body
+      path = (@pr ? "/repos/#{@repo_part}/issues/#{@pr}/comments" : "/repos/#{@repo_part}/commits/#{@sha}/comments")
+      post path, body: body
     end
 
     private
 
     def post(path, data)
-      url = "https://api.github.com/repos/#{@repo_part}/#{path}"
+      url = "https://api.github.com#{path}"
       response = Faraday.post(url, data.to_json, authorization: "token #{@token}")
       raise "failed to POST to github:\n#{url} -> #{response.status}\n#{response.body}" unless response.status == 201
     end

data/lib/kennel/models/dashboard.rb

@@ -46,7 +46,6 @@ module Kennel
           font_size: "14"
         },
         "query_value" => {
-          autoscale: true,
           time: {},
           title_align: "left",
           title_size: "16"

data/lib/kennel/syncer.rb

@@ -3,6 +3,7 @@ module Kennel
   class Syncer
     DELETE_ORDER = ["dashboard", "slo", "monitor", "synthetics/tests"].freeze # dashboards references monitors + slos, slos reference monitors
     LINE_UP = "\e[1A\033[K" # go up and clear
+    DEFAULT_BRANCH = "master"
 
     def initialize(api, expected, project: nil)
       @api = api
@@ -26,11 +27,7 @@ module Kennel
     def confirm
       return false if noop?
       return true if ENV["CI"] || !STDIN.tty?
-      if @delete.any? && @project_filter
-        Kennel.out.puts(
-          Utils.color(:red, "WARNING: deleting resources that had `id: -> { ...` breaks master branch")
-        )
-      end
+      warn_about_deleting_resources_with_id if @project_filter
       Utils.ask("Execute Plan ?")
     end
 
@@ -63,6 +60,30 @@ module Kennel
 
     private
 
+    # this is brittle/hacky since it relies on knowledge from the generation + git + branch knowledge
+    def warn_about_deleting_resources_with_id
+      @delete.each do |_, _, a|
+        tracking_id = a.fetch(:tracking_id)
+        api_resource = a.fetch(:klass).api_resource
+
+        file = "generated/#{tracking_id.sub(":", "/")}.json"
+        old = `true && git show #{DEFAULT_BRANCH}:#{file.shellescape} 2>&1` # true && to not crash on missing git
+
+        next unless $?.success?
+        old =
+          begin
+            JSON.parse(old)
+          rescue StandardError
+            false
+          end
+        next if !old || !old["id"]
+
+        Kennel.out.puts(
+          Utils.color(:red, "WARNING: deleting #{api_resource} #{tracking_id} will break #{DEFAULT_BRANCH} branch")
+        )
+      end
+    end
+
     # loop over items until everything is resolved or crash when we get stuck
     # this solves cases like composite monitors depending on each other or monitor->monitor slo->slo monitor chains
     def each_resolved(list)

data/lib/kennel/tasks.rb

@@ -48,7 +48,7 @@ namespace :kennel do
       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
+      Kennel::Tasks.abort ENV["KNOWN_WARNING"]
     end
   end
 

data/lib/kennel/version.rb

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

data/lib/kennel.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 require "faraday"
 require "json"
+require "shellwords"
 require "English"
 
 require "kennel/version"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: kennel
 version: !ruby/object:Gem::Version
-  version: 1.91.3
+  version: 1.92.3
 platform: ruby
 authors:
 - Michael Grosser
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-07-17 00:00:00.000000000 Z
+date: 2021-09-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday