aspecto-opentelemetry-instrumentation-aws_sdk 0.1.8 → 0.4.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fd22dbff8fc844572b0134bc017e7202b7b0a0b117b3eceab65249449493d57e
-  data.tar.gz: c9466be404ec13f25017f7db758819500ae4f109ad2826923d86907ce72ba13a
+  metadata.gz: 4e06be223969bfa905a6ea8b8f8dad2c8efd7532df475e35f862c25dd103dd30
+  data.tar.gz: 6e21b2bb8abd75b523f5a3ca56f53b01fdd065bf187a4fc8a9b23a35db9f2e69
 SHA512:
-  metadata.gz: 1337b7ad6217685438f65de8655865f2f699b3ac12c7cd17d78e292f8fade4afddbfe1e7ee35e974abd4645e042d6209b2fc52fdda4e1291b2ca61ae6d129eb4
-  data.tar.gz: 03fb4f63e6b7a485b6acd925e351d6d289ccdcc008b891f584662830d21c3a0900b03e41a7fc1575e436d003309a8bebdf1b548a499242e9e903ad3ea020f91e
+  metadata.gz: cf2d059c6f60d808c248b211560012f3633c34ce7dd925d786e60f6b1aa2f8e7514042ad966933f5d70615758dc3dcead7bfda9ea935f3f228f01cb577da4f86
+  data.tar.gz: c8649583f19d785df378fa1b661ee9092436e54c08067c2e14c4974f922c82f2169f044d72ee40db1fb5d594545da93082adc763eb286d2118ee3fba17e52b1b

data/CHANGELOG.md

@@ -1,4 +1,20 @@
-# Release History: opentelemetry-instrumentation-aws_sdk
+# Release History: opentelemetry-instrumentation-aws_sdk
+
+### v0.2.3 / 2022-05-02
+
+* FIXED: RubyGems Fallback 
+
+### v0.2.2 / 2022-01-26
+
+* (No significant changes)
+
+### v0.2.1 / 2022-01-21
+
+* ADDED: attach HTTP status code to AWS spans
+
+### v0.2.0 / 2022-01-20
+
+* ADDED: SQS / SNS messaging attributes and context propagation
 
 ### v0.1.0 / 2021-12-01
 

data/README.md

@@ -19,8 +19,8 @@ To install the instrumentation, call `use` with the name of the instrumentation.
 ```ruby
 OpenTelemetry::SDK.configure do |c|
   c.use 'OpenTelemetry::Instrumentation::AwsSdk', {
-    extract_messaging_context: true,
     inject_messaging_context: true,
+    extract_messaging_context: true,
     suppress_internal_instrumentation: true
   }
 end

data/lib/opentelemetry/instrumentation/aws_sdk/handler.rb

@@ -17,7 +17,7 @@ module OpenTelemetry
         def call(context) # rubocop:disable Metrics/AbcSize, Metrics/MethodLength, Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
           return super unless context
 
-          service_name = context.client.class.api.metadata['serviceId'] || context.client.class.to_s.split('::')[1] rescue context.client.class.to_s.split('::')[1] # rubocop:disable Style/RescueModifier
+          service_name = service_name(context)
           operation = context.operation&.name
           client_method = "#{service_name}.#{operation}"
           attributes = {
@@ -39,12 +39,15 @@ module OpenTelemetry
             else
               super
             end.tap do |response|
+              span.set_attribute(OpenTelemetry::SemanticConventions::Trace::HTTP_STATUS_CODE,
+                                 context.http_response.status_code)
+
               if (err = response.error)
                 span.record_exception(err)
                 span.status = Trace::Status.error(err.to_s)
               end
 
-              MessagingHelper.create_sqs_processing_spans(context, tracer, response.messages) if client_method == SQS_RECEIVE_MESSAGE && response.respond_to?(:messages)
+              MessagingHelper.create_sqs_processing_spans(context, tracer, response.messages) if instrumentation_config[:extract_messaging_context] && client_method == SQS_RECEIVE_MESSAGE && response.respond_to?(:messages)
             end
           end
         end
@@ -59,8 +62,16 @@ module OpenTelemetry
           AwsSdk::Instrumentation.instance.config
         end
 
+        def service_name(context) # rubocop:disable Metrics/AbcSize
+          # Support aws-sdk v2.0.x, which 'metadata' has a setter method only
+          return context.client.class.to_s.split('::')[1] if ::Seahorse::Model::Api.instance_method(:metadata).parameters.length.positive?
+
+          context.client.class.api.metadata['serviceId'] || context.client.class.to_s.split('::')[1]
+        end
+
+        SEND_MESSAGE_CLIENT_METHODS = [SQS_SEND_MESSAGE, SQS_SEND_MESSAGE_BATCH, SNS_PUBLISH].freeze
         def inject_context(context, client_method)
-          return unless [SQS_SEND_MESSAGE, SQS_SEND_MESSAGE_BATCH, SNS_PUBLISH].include? client_method
+          return unless SEND_MESSAGE_CLIENT_METHODS.include? client_method
           return unless instrumentation_config[:inject_messaging_context]
 
           if client_method == SQS_SEND_MESSAGE_BATCH

data/lib/opentelemetry/instrumentation/aws_sdk/instrumentation.rb

@@ -9,7 +9,7 @@ module OpenTelemetry
     module AwsSdk
       # Instrumentation class that detects and installs the AwsSdk instrumentation
       class Instrumentation < OpenTelemetry::Instrumentation::Base
-        MINIMUM_VERSION = Gem::Version.new('2.0')
+        MINIMUM_VERSION = Gem::Version.new('2.0.0')
 
         install do |_config|
           require_dependencies
@@ -21,7 +21,7 @@ module OpenTelemetry
         end
 
         compatible do
-          gem_version >= MINIMUM_VERSION
+          !gem_version.nil? && gem_version >= MINIMUM_VERSION
         end
 
         option :extract_messaging_context, default: false, validate: :boolean
@@ -33,6 +33,8 @@ module OpenTelemetry
             Gem.loaded_specs['aws-sdk'].version
           elsif Gem.loaded_specs['aws-sdk-core']
             Gem.loaded_specs['aws-sdk-core'].version
+          elsif defined?(::Aws::CORE_GEM_VERSION)
+            Gem::Version.new(::Aws::CORE_GEM_VERSION)
           end
         end
 

data/lib/opentelemetry/instrumentation/aws_sdk/messaging_helper.rb

@@ -10,23 +10,19 @@ module OpenTelemetry
       # MessagingHelper class provides methods for calculating messaging span attributes
       class MessagingHelper
         class << self
-          def destination_name(context) # rubocop:disable Metrics/CyclomaticComplexity
+          def destination_name(context)
             topic_arn = context.params[:topic_arn]
             target_arn = context.params[:target_arn]
-            phone_number = context.params[:phone_number]
-            queue_url = context.params[:queue_url]
 
             if topic_arn || target_arn
               arn = topic_arn || target_arn
-              begin
-                return arn.split(':')[-1]
-              rescue StandardError
-                return arn
-              end
+              return arn.split(':')[-1]
             end
 
+            phone_number = context.params[:phone_number]
             return 'phone_number' if phone_number
 
+            queue_url = context.params[:queue_url]
             return queue_url.split('/')[-1] if queue_url
 
             'unknown'
@@ -36,7 +32,7 @@ module OpenTelemetry
             attributes[SemanticConventions::Trace::MESSAGING_SYSTEM] = 'aws.sqs'
             attributes[SemanticConventions::Trace::MESSAGING_DESTINATION_KIND] = 'queue'
             attributes[SemanticConventions::Trace::MESSAGING_DESTINATION] = destination_name(context)
-            attributes[SemanticConventions::Trace::MESSAGING_URL] = context.params[:queue_url]
+            attributes[SemanticConventions::Trace::MESSAGING_URL] = context.params[:queue_url] if context.params[:queue_url]
 
             attributes[SemanticConventions::Trace::MESSAGING_OPERATION] = 'receive' if client_method == 'SQS.ReceiveMessage'
           end

data/lib/opentelemetry/instrumentation/aws_sdk/version.rb

@@ -7,7 +7,7 @@
 module OpenTelemetry
   module Instrumentation
     module AwsSdk
-      VERSION = '0.1.8'
+      VERSION = '0.4.2'
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: aspecto-opentelemetry-instrumentation-aws_sdk
 version: !ruby/object:Gem::Version
-  version: 0.1.8
+  version: 0.4.2
 platform: ruby
 authors:
-- Aspecto Authors
-autorequire: 
+- OpenTelemetry Authors
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-12-21 00:00:00.000000000 Z
+date: 2022-07-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: opentelemetry-api
@@ -100,14 +100,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '1.1'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '1.1'
+- !ruby/object:Gem::Dependency
+  name: opentelemetry-test-helpers
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: pry
   requirement: !ruby/object:Gem::Requirement
@@ -222,7 +236,7 @@ dependencies:
         version: 0.1.6
 description: AWS SDK instrumentation for the OpenTelemetry framework
 email:
-- info@aspecto.io
+- cncf-opentelemetry-contributors@lists.cncf.io
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -240,15 +254,15 @@ files:
 - lib/opentelemetry/instrumentation/aws_sdk/messaging_helper.rb
 - lib/opentelemetry/instrumentation/aws_sdk/services.rb
 - lib/opentelemetry/instrumentation/aws_sdk/version.rb
-homepage: https://aspecto.io
+homepage: https://github.com/open-telemetry/opentelemetry-ruby
 licenses:
 - Apache-2.0
 metadata:
-  changelog_uri: https://open-telemetry.github.io/opentelemetry-ruby/opentelemetry-instrumentation-aws_sdk/v0.1.8/file.CHANGELOG.html
+  changelog_uri: https://open-telemetry.github.io/opentelemetry-ruby/opentelemetry-instrumentation-aws_sdk/v0.4.2/file.CHANGELOG.html
   source_code_uri: https://github.com/open-telemetry/opentelemetry-ruby/tree/main/instrumentation/aws_sdk
   bug_tracker_uri: https://github.com/open-telemetry/opentelemetry-ruby/issues
-  documentation_uri: https://open-telemetry.github.io/opentelemetry-ruby/opentelemetry-instrumentation-aws_sdk/v0.1.8
-post_install_message: 
+  documentation_uri: https://open-telemetry.github.io/opentelemetry-ruby/opentelemetry-instrumentation-aws_sdk/v0.4.2
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -256,15 +270,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.5.0
+      version: 2.6.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.1.4
+signing_key:
 specification_version: 4
 summary: AWS SDK instrumentation for the OpenTelemetry framework
 test_files: []