consul_syncer 1.2.0 → 1.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 875c3e3eee8a06872dde88a612d7dfb8ae33ac59377d441e54afa6585da787b7
-  data.tar.gz: 4e1d1d3701fc464772745858109ff04260b0a8568e7ce8ad8eeab6ad6de26032
+  metadata.gz: 0b2bef9d0c04323f933176da717b088e7179d8c1654bad10a5c91d1e83f957fb
+  data.tar.gz: 25ecfe6ce76a967aefdb2d89fd82137664a418848587645a29fc1346fc95165a
 SHA512:
-  metadata.gz: ae7f93e5464309a3ed828ba18e7f43bd8ff2ec8a111b513d225f598d2a84c6df80c480722d3f1545a7813c2a34fbf5de10083357432d6f3f9336bf4923a9f082
-  data.tar.gz: 3cdeb2e0566669014aa37ae01cebcb453593e993cf9cee5a4155d26649fddb8f5ca38e179cd911979824c08a5fa98d23345bf441c5b2eb8e5937001263aedba0
+  metadata.gz: 42d34c44dc46d220efff7759e423aa3c2e62ed69e9a63e91fc342111f09ed14ba073798be9edb22c956c3e8007cb0746105c144dedac5bdb1cca0b2a27e94653
+  data.tar.gz: 32eeeb25cb89d0f20802abc6a61cf9437bc8f6c44158fc5c84999cca42eb0f4761bfeaf1a97a9062d9ceedadc7588186c3f82e6fe97688138ff4032f1ad04107

data/lib/consul_syncer/version.rb

@@ -1,3 +1,3 @@
 class ConsulSyncer
-  VERSION = "1.2.0"
+  VERSION = "1.2.1"
 end

data/lib/consul_syncer.rb

@@ -64,31 +64,23 @@ class ConsulSyncer
       elsif remove_matching_service!(actual_definitions, expected, interesting)
         @logger.debug "Found #{description}"
       elsif remove_matching_service!(actual_definitions, expected, identifying)
-        @logger.info "Updating #{description}"
-        planned << [:register, [], expected]
+        planned << [:register, [], expected.merge(reason: "Updating #{description}")]
       else
-        @logger.info "Adding #{description}"
-        planned << [:register, [], expected]
+        planned << [:register, [], expected.merge(reason: "Adding #{description}")]
       end
     end
 
     # all definitions that are left did not match any expected definitions and are no longer needed
     actual_definitions.each do |actual|
-      @logger.info "Removing #{actual.fetch(:service)} / #{actual.fetch(:service_id)} on #{actual.fetch(:node)} in Consul"
-      planned << [:deregister, [actual.fetch(:node), actual.fetch(:service_id)], nil]
+      reason = "Removing #{actual.fetch(:service)} / #{actual.fetch(:service_id)} on #{actual.fetch(:node)} in Consul"
+      planned << [:deregister, [actual.fetch(:node), actual.fetch(:service_id)], {reason: reason}]
     end
 
     planned
   end
 
   def execute(planned)
-    planned.each do |m, args, kwargs|
-      if kwargs
-        send m, *args, **kwargs
-      else
-        send m, *args
-      end
-    end
+    planned.each { |m, args, kwargs| send m, *args, **kwargs }
   end
 
   private
@@ -116,7 +108,8 @@ class ConsulSyncer
   end
 
   # creates or updates based on node and service
-  def register(node:, service:, service_id:, service_address:, address:, tags:, port:)
+  def register(node:, service:, service_id:, service_address:, address:, tags:, port:, reason:)
+    @logger.info reason
     @consul.request(
       :put,
       '/v1/catalog/register',
@@ -132,7 +125,8 @@ class ConsulSyncer
     )
   end
 
-  def deregister(node, service_id)
+  def deregister(node, service_id, reason:)
+    @logger.info reason
     @consul.request(
       :put,
       '/v1/catalog/deregister',

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: consul_syncer
 version: !ruby/object:Gem::Version
-  version: 1.2.0
+  version: 1.2.1
 platform: ruby
 authors:
 - Michael Grosser
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-09-09 00:00:00.000000000 Z
+date: 2021-09-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday