aspecto-opentelemetry 0.1.8 → 0.1.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c6dc0881d253f1a91182a09ae64a3464bc935b63744ea68654011bb317b1278b
-  data.tar.gz: b42327874bf790cfa4d4fbcea31f84a4bd9a8e31cf94362329f25c195202ec4f
+  metadata.gz: 005e783e20aa54623dca0a11daae40ef3e3d8a6d9b83a13b6fd44f11a0a7f00b
+  data.tar.gz: 9edaffde8afa0cbfb8f3547aec3fd4ab3fc236395e70e6ecb1e60b83f5d563b1
 SHA512:
-  metadata.gz: da5bc77255107ff1236861ab1ffcf61a98e27af86ad9a12787fa03f6d8d6ef3f32a7bb1de1032a717f0f51c647f59942924fd162e507c039d785f6b0272ea905
-  data.tar.gz: 905b157c77e02bb808ca5c4e5a2a644156b79be3fd8d39b2f8ebbc2a32c4cb7651d6ef4c6dec5b041d37ef574a05085a1d03dbc21f165ca5b32a217c613f6482
+  metadata.gz: a220142b3826b3cf0d95e73c559e0ac5ce0e29828f98883fd6e7f8e7a9c4a486bfe6620a49f73cbc938d23c3b0c48f8782147a69779256719520532112ef1b6d
+  data.tar.gz: fd9e2c8b3513d25599411f2718ff1f3aa79b75425fd9d81f75f32e1ceb7b69c4228dfc46abc1f0dc64a7f020c2916ed47f1bc29d11539cabf87e91945ce38851

data/lib/aspecto/opentelemetry/config/remote_config.rb

@@ -16,6 +16,8 @@ module Aspecto
         def initialize(aspecto_auth, service_name, env, fallback_sampler)
           @service_name = service_name
           @env = env
+          @error_reported = false
+          @got_remote_config = false
           @fallback_sampler = fallback_sampler
           aspecto_config_host = ENV.fetch("ASPECTO_CONFIG_HOST", "https://config.aspecto.io")
           @aspecto_config_url = URI("#{aspecto_config_host}/config/#{aspecto_auth}")
@@ -27,7 +29,10 @@ module Aspecto
             update_config
           end
 
-          ::OpenTelemetry.logger.info "[Aspecto] SDK client initialized"
+          ::OpenTelemetry.logger.info "[Aspecto] initialized remote config polling"
+        rescue StandardError => e
+          ::OpenTelemetry.logger.error "[Aspecto] failed to initialize remote config polling"
+          ::OpenTelemetry.logger.error e
         end
 
         def shutdown
@@ -50,7 +55,7 @@ module Aspecto
           @http_client.verify_mode = OpenSSL::SSL::VERIFY_PEER
         end
 
-        def update_config # rubocop:disable Metrics/AbcSize
+        def update_config
           ::OpenTelemetry::Common::Utilities.untraced do
             request = Net::HTTP::Get.new(@aspecto_config_url.path)
             request["If-None-Match"] = @latest_config_etag unless @latest_config_etag.nil?
@@ -60,7 +65,7 @@ module Aspecto
             return if response_code == 304
 
             if response_code >= 400
-              ::OpenTelemetry.logger.error("[Aspecto] error when trying to get remote config. will try again in #{@remote_config_poll_frequency}")
+              log_config_error "failed to get remote config with http response #{response_code}."
               return
             end
 
@@ -68,13 +73,14 @@ module Aspecto
             handle_new_config JSON.parse(response.body) if response_code < 300
           end
         rescue StandardError => e
-          ::OpenTelemetry.logger.error "[Aspecto] updating remote config failed. using previous remote config"
-          ::OpenTelemetry.logger.debug e
+          log_config_error "updating remote config failed with exception.", e
         end
 
         def handle_new_config(config)
-          ::OpenTelemetry.logger.info("[Aspecto] successfully received remote configuration")
           update_sampler config["samplingRules"]
+          @error_reported = false
+          @got_remote_config = true
+          ::OpenTelemetry.logger.info("[Aspecto] successfully updated remote configuration")
         end
 
         def update_sampler(sampler_config)
@@ -84,6 +90,16 @@ module Aspecto
           # updating the sampler should be thread safe as it's an atomic setter on a global value
           ::OpenTelemetry.tracer_provider.sampler = message_process_sampler
         end
+
+        def log_config_error(msg, err = nil)
+          return if @error_reported # report every issue only once
+
+          retry_msg = "will try again until success every #{@remote_config_poll_frequency}."
+          previous_config_msg = @got_remote_config ? "using previous remote config" : "no previous config is avialable"
+          ::OpenTelemetry.logger.info("[Aspecto]: #{msg} #{retry_msg} #{previous_config_msg}")
+          ::OpenTelemetry.logger.info(err) if err
+          @error_reported = true
+        end
       end
     end
   end

data/lib/aspecto/opentelemetry/version.rb

@@ -2,6 +2,6 @@
 
 module Aspecto
   module OpenTelemetry
-    VERSION = "0.1.8"
+    VERSION = "0.1.9"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: aspecto-opentelemetry
 version: !ruby/object:Gem::Version
-  version: 0.1.8
+  version: 0.1.9
 platform: ruby
 authors:
 - Aspecto
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2022-01-03 00:00:00.000000000 Z
+date: 2022-01-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: aspecto-opentelemetry-instrumentation-aws_sdk