kennel 1.68.0 → 1.69.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cb534c9bfadda4d374201e3e6b00eddbe5596af1376b8e11e071b27c7d1eaa28
-  data.tar.gz: 33566de8715eb98743470f5d40f9e77744a1932de3900652cf8f01b72ffbe5f6
+  metadata.gz: daadbdb69c86de7a0dcdec2bc1eeaedd4b0c2221e2059d241db57e717a8876b6
+  data.tar.gz: 1d93147d3d88468564c6c9fcfd0a1fa3926df23d44961b876273e24ac642c851
 SHA512:
-  metadata.gz: 57b28c3702ce5bf31b81dcfc104e61cb4ef936564df0575ef65812dc0d2224202c71d120f53b82527464dd50f545388e7d130600941b2fb75e9e2361b1dc499a
-  data.tar.gz: 97b1954e09f674ce834ad717fa90112f8e1e905d244040b0d2360b41f0bfde0955da5d69a86985964525efd2aa89ee13fdcbd22beffcbea2911aa5a795906788
+  metadata.gz: 7737e951ba0e6f34bdec5c4c000e64852acfcc1939d19385a7a92d9b88821c5d6efe0f29f1d8a292692c3ae8b85bd248af0f05ed3e591595da5d2ee41382fd37
+  data.tar.gz: 62989fb72a5bc60f052449ecf4d380f59f7e945a8d1857e7f3e789549d8625eac8ffd51c2a946b4f0367cd2d439b2ecb02eb99f79cb42202bb020dadf8a6b612

data/lib/kennel/syncer.rb

@@ -19,6 +19,7 @@ module Kennel
       end
       @expected.each { |e| add_tracking_id e }
       calculate_diff
+      prevent_irreversible_partial_updates
     end
 
     def plan
@@ -42,7 +43,6 @@ module Kennel
         Kennel.out.puts "Created #{e.class.api_resource} #{tracking_id(e.as_json)} #{e.url(reply.fetch(:id))}"
       end
 
-      block_irreversible_partial_updates
       @update.each do |id, e|
         @api.update e.class.api_resource, id, e.as_json
         Kennel.out.puts "Updated #{e.class.api_resource} #{tracking_id(e.as_json)} #{e.url(id)}"
@@ -151,7 +151,7 @@ module Kennel
       return if list.empty?
       list.each do |_, e, a, diff|
         api_resource = (e ? e.class.api_resource : a.fetch(:api_resource))
-        Kennel.out.puts Utils.color(color, "#{step} #{api_resource} #{tracking_id(e&.as_json || a)}")
+        Kennel.out.puts Utils.color(color, "#{step} #{api_resource} #{e&.tracking_id || tracking_id(a)}")
         print_diff(diff) if diff # only for update
       end
     end
@@ -159,12 +159,12 @@ module Kennel
     def print_diff(diff)
       diff.each do |type, field, old, new|
         if type == "+"
-          temp = new.inspect
-          new = old.inspect
+          temp = Utils.pretty_inspect(new)
+          new = Utils.pretty_inspect(old)
           old = temp
         else # ~ and -
-          old = old.inspect
-          new = new.inspect
+          old = Utils.pretty_inspect(old)
+          new = Utils.pretty_inspect(new)
         end
 
         if (old + new).size > 100
@@ -177,18 +177,30 @@ module Kennel
       end
     end
 
-    def block_irreversible_partial_updates
+    # Do not add tracking-id when working with existing ids on a branch,
+    # so resource do not get deleted from merges to master.
+    # Also make sure the diff still makes sense, by kicking out the now noop-update.
+    #
+    # Note: ideally we'd never add tracking in the first place, but at that point we do not know the diff yet
+    def prevent_irreversible_partial_updates
       return unless @project_filter
-      return if @update.none? do |_, e, _, diff|
-        e.id && diff.any? do |_, field, old, new = nil|
-          TRACKING_FIELDS.include?(field.to_sym) && tracking_value(old) != tracking_value(new)
+      @update.select! do |_, e, _, diff|
+        next true unless e.id # short circuit for performance
+
+        diff.select! do |field_diff|
+          (_, field, old, new) = field_diff
+          next true unless tracking_field?(field)
+
+          if (old_tracking = tracking_value(old))
+            old_tracking == tracking_value(new) || raise("do not update! (atm unreachable)")
+          else
+            field_diff[3] = remove_tracking_id(e) # make plan output match update
+            old != field_diff[3]
+          end
         end
+
+        !diff.empty?
       end
-      raise <<~TEXT
-        Updates with PROJECT= filter should not update tracking id in #{TRACKING_FIELDS.join("/")} of resources with a set `id:`,
-        since this makes them get deleted by a full update.
-        Remove the `id:` to test them out, which will result in a copy being created and later deleted.
-      TEXT
     end
 
     def resolve_linked_tracking_ids(actual)
@@ -212,6 +224,13 @@ module Kennel
       json[field] = "#{json[field]}\n-- Managed by kennel #{e.tracking_id} in #{e.project.class.file_location}, do not modify manually".lstrip
     end
 
+    def remove_tracking_id(e)
+      json = e.as_json
+      field = tracking_field(json)
+      value = json[field]
+      json[field] = value.dup.sub!(/\n-- Managed by kennel .*/, "") || raise("did not find tracking id in #{value}")
+    end
+
     def tracking_id(a)
       tracking_value a[tracking_field(a)]
     end
@@ -223,5 +242,9 @@ module Kennel
     def tracking_field(a)
       TRACKING_FIELDS.detect { |f| a.key?(f) }
     end
+
+    def tracking_field?(field)
+      TRACKING_FIELDS.include?(field.to_sym)
+    end
   end
 end

data/lib/kennel/utils.rb

@@ -142,6 +142,17 @@ module Kennel
         else []
         end
       end
+
+      # 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.gsub!(/:([a-z_]+)=>/, "\\1: ")
+        10.times do
+          string.gsub!(/{(\S.*?\S)}/, "{ \\1 }") || break
+        end
+        string
+      end
     end
   end
 end

data/lib/kennel/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: kennel
 version: !ruby/object:Gem::Version
-  version: 1.68.0
+  version: 1.69.0
 platform: ruby
 authors:
 - Michael Grosser
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-05-11 00:00:00.000000000 Z
+date: 2020-05-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday