ddtrace 1.12.1 → 1.13.0

data/lib/datadog/tracing/contrib/action_pack/ext.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Datadog
   module Tracing
     module Contrib
@@ -5,14 +7,14 @@ module Datadog
         # ActionPack integration constants
         # @public_api Changing resource names, tag names, or environment variables creates breaking changes.
         module Ext
-          ENV_ENABLED = 'DD_TRACE_ACTION_PACK_ENABLED'.freeze
-          ENV_ANALYTICS_ENABLED = 'DD_TRACE_ACTION_PACK_ANALYTICS_ENABLED'.freeze
-          ENV_ANALYTICS_SAMPLE_RATE = 'DD_TRACE_ACTION_PACK_ANALYTICS_SAMPLE_RATE'.freeze
-          SPAN_ACTION_CONTROLLER = 'rails.action_controller'.freeze
-          TAG_COMPONENT = 'action_pack'.freeze
-          TAG_OPERATION_CONTROLLER = 'controller'.freeze
-          TAG_ROUTE_ACTION = 'rails.route.action'.freeze
-          TAG_ROUTE_CONTROLLER = 'rails.route.controller'.freeze
+          ENV_ENABLED = 'DD_TRACE_ACTION_PACK_ENABLED'
+          ENV_ANALYTICS_ENABLED = 'DD_TRACE_ACTION_PACK_ANALYTICS_ENABLED'
+          ENV_ANALYTICS_SAMPLE_RATE = 'DD_TRACE_ACTION_PACK_ANALYTICS_SAMPLE_RATE'
+          SPAN_ACTION_CONTROLLER = 'rails.action_controller'
+          TAG_COMPONENT = 'action_pack'
+          TAG_OPERATION_CONTROLLER = 'controller'
+          TAG_ROUTE_ACTION = 'rails.route.action'
+          TAG_ROUTE_CONTROLLER = 'rails.route.controller'
         end
       end
     end

data/lib/datadog/tracing/contrib/action_view/configuration/settings.rb

@@ -10,18 +10,21 @@ module Datadog
           # @public_api
           class Settings < Contrib::Configuration::Settings
             option :enabled do |o|
-              o.default { env_to_bool(Ext::ENV_ENABLED, true) }
-              o.lazy
+              o.type :bool
+              o.env Ext::ENV_ENABLED
+              o.default true
             end
 
             option :analytics_enabled do |o|
-              o.default { env_to_bool(Ext::ENV_ANALYTICS_ENABLED, false) }
-              o.lazy
+              o.type :bool
+              o.env Ext::ENV_ANALYTICS_ENABLED
+              o.default false
             end
 
             option :analytics_sample_rate do |o|
-              o.default { env_to_float(Ext::ENV_ANALYTICS_SAMPLE_RATE, 1.0) }
-              o.lazy
+              o.type :float
+              o.env Ext::ENV_ANALYTICS_SAMPLE_RATE
+              o.default 1.0
             end
 
             option :service_name

data/lib/datadog/tracing/contrib/action_view/ext.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Datadog
   module Tracing
     module Contrib
@@ -5,16 +7,16 @@ module Datadog
         # ActionView integration constants
         # @public_api Changing resource names, tag names, or environment variables creates breaking changes.
         module Ext
-          ENV_ENABLED = 'DD_TRACE_ACTION_VIEW_ENABLED'.freeze
-          ENV_ANALYTICS_ENABLED = 'DD_TRACE_ACTION_VIEW_ANALYTICS_ENABLED'.freeze
-          ENV_ANALYTICS_SAMPLE_RATE = 'DD_TRACE_ACTION_VIEW_ANALYTICS_SAMPLE_RATE'.freeze
-          SPAN_RENDER_PARTIAL = 'rails.render_partial'.freeze
-          SPAN_RENDER_TEMPLATE = 'rails.render_template'.freeze
-          TAG_COMPONENT = 'action_view'.freeze
-          TAG_LAYOUT = 'rails.layout'.freeze
-          TAG_OPERATION_RENDER_PARTIAL = 'render_partial'.freeze
-          TAG_OPERATION_RENDER_TEMPLATE = 'render_template'.freeze
-          TAG_TEMPLATE_NAME = 'rails.template_name'.freeze
+          ENV_ENABLED = 'DD_TRACE_ACTION_VIEW_ENABLED'
+          ENV_ANALYTICS_ENABLED = 'DD_TRACE_ACTION_VIEW_ANALYTICS_ENABLED'
+          ENV_ANALYTICS_SAMPLE_RATE = 'DD_TRACE_ACTION_VIEW_ANALYTICS_SAMPLE_RATE'
+          SPAN_RENDER_PARTIAL = 'rails.render_partial'
+          SPAN_RENDER_TEMPLATE = 'rails.render_template'
+          TAG_COMPONENT = 'action_view'
+          TAG_LAYOUT = 'rails.layout'
+          TAG_OPERATION_RENDER_PARTIAL = 'render_partial'
+          TAG_OPERATION_RENDER_TEMPLATE = 'render_template'
+          TAG_TEMPLATE_NAME = 'rails.template_name'
         end
       end
     end

data/lib/datadog/tracing/contrib/active_job/configuration/settings.rb

@@ -13,22 +13,28 @@ module Datadog
           # @public_api
           class Settings < Contrib::Configuration::Settings
             option :enabled do |o|
-              o.default { env_to_bool(Ext::ENV_ENABLED, true) }
-              o.lazy
+              o.type :bool
+              o.env Ext::ENV_ENABLED
+              o.default true
             end
 
             option :analytics_enabled do |o|
-              o.default { env_to_bool(Ext::ENV_ANALYTICS_ENABLED, false) }
-              o.lazy
+              o.type :bool
+              o.env Ext::ENV_ANALYTICS_ENABLED
+              o.default false
             end
 
             option :analytics_sample_rate do |o|
-              o.default { env_to_float(Ext::ENV_ANALYTICS_SAMPLE_RATE, 1.0) }
-              o.lazy
+              o.type :float
+              o.env Ext::ENV_ANALYTICS_SAMPLE_RATE
+              o.default 1.0
             end
 
             option :service_name
-            option :error_handler, default: Tracing::SpanOperation::Events::DEFAULT_ON_ERROR
+            option :error_handler do |o|
+              o.type :proc
+              o.experimental_default_proc(&Tracing::SpanOperation::Events::DEFAULT_ON_ERROR)
+            end
           end
         end
       end

data/lib/datadog/tracing/contrib/active_job/ext.rb

@@ -1,35 +1,37 @@
+# frozen_string_literal: true
+
 module Datadog
   module Tracing
     module Contrib
       module ActiveJob
         # @public_api Changing resource names, tag names, or environment variables creates breaking changes.
         module Ext
-          ENV_ENABLED = 'DD_TRACE_ACTIVE_JOB_ENABLED'.freeze
-          ENV_ANALYTICS_ENABLED = 'DD_TRACE_ACTIVE_JOB_ANALYTICS_ENABLED'.freeze
-          ENV_ANALYTICS_SAMPLE_RATE = 'DD_TRACE_ACTIVE_JOB_ANALYTICS_SAMPLE_RATE'.freeze
+          ENV_ENABLED = 'DD_TRACE_ACTIVE_JOB_ENABLED'
+          ENV_ANALYTICS_ENABLED = 'DD_TRACE_ACTIVE_JOB_ANALYTICS_ENABLED'
+          ENV_ANALYTICS_SAMPLE_RATE = 'DD_TRACE_ACTIVE_JOB_ANALYTICS_SAMPLE_RATE'
 
-          SPAN_DISCARD = 'active_job.discard'.freeze
-          SPAN_ENQUEUE = 'active_job.enqueue'.freeze
-          SPAN_ENQUEUE_RETRY = 'active_job.enqueue_retry'.freeze
-          SPAN_PERFORM = 'active_job.perform'.freeze
-          SPAN_RETRY_STOPPED = 'active_job.retry_stopped'.freeze
+          SPAN_DISCARD = 'active_job.discard'
+          SPAN_ENQUEUE = 'active_job.enqueue'
+          SPAN_ENQUEUE_RETRY = 'active_job.enqueue_retry'
+          SPAN_PERFORM = 'active_job.perform'
+          SPAN_RETRY_STOPPED = 'active_job.retry_stopped'
 
-          TAG_COMPONENT = 'active_job'.freeze
-          TAG_OPERATION_DISCARD = 'discard'.freeze
-          TAG_OPERATION_ENQUEUE = 'enqueue'.freeze
-          TAG_OPERATION_ENQUEUE_AT = 'enqueue_at'.freeze
-          TAG_OPERATION_ENQUEUE_RETRY = 'enqueue_retry'.freeze
-          TAG_OPERATION_PERFORM = 'perform'.freeze
-          TAG_OPERATION_RETRY_STOPPED = 'retry_stopped'.freeze
+          TAG_COMPONENT = 'active_job'
+          TAG_OPERATION_DISCARD = 'discard'
+          TAG_OPERATION_ENQUEUE = 'enqueue'
+          TAG_OPERATION_ENQUEUE_AT = 'enqueue_at'
+          TAG_OPERATION_ENQUEUE_RETRY = 'enqueue_retry'
+          TAG_OPERATION_PERFORM = 'perform'
+          TAG_OPERATION_RETRY_STOPPED = 'retry_stopped'
 
-          TAG_ADAPTER = 'active_job.adapter'.freeze
-          TAG_JOB_ERROR = 'active_job.job.error'.freeze
-          TAG_JOB_EXECUTIONS = 'active_job.job.executions'.freeze
-          TAG_JOB_ID = 'active_job.job.id'.freeze
-          TAG_JOB_PRIORITY = 'active_job.job.priority'.freeze
-          TAG_JOB_QUEUE = 'active_job.job.queue'.freeze
-          TAG_JOB_RETRY_WAIT = 'active_job.job.retry_wait'.freeze
-          TAG_JOB_SCHEDULED_AT = 'active_job.job.scheduled_at'.freeze
+          TAG_ADAPTER = 'active_job.adapter'
+          TAG_JOB_ERROR = 'active_job.job.error'
+          TAG_JOB_EXECUTIONS = 'active_job.job.executions'
+          TAG_JOB_ID = 'active_job.job.id'
+          TAG_JOB_PRIORITY = 'active_job.job.priority'
+          TAG_JOB_QUEUE = 'active_job.job.queue'
+          TAG_JOB_RETRY_WAIT = 'active_job.job.retry_wait'
+          TAG_JOB_SCHEDULED_AT = 'active_job.job.scheduled_at'
         end
       end
     end

data/lib/datadog/tracing/contrib/active_job/log_injection.rb

@@ -9,7 +9,7 @@ module Datadog
           def self.included(base)
             base.class_eval do
               around_perform do |_, block|
-                if logger.respond_to?(:tagged)
+                if Datadog.configuration.tracing.log_injection && logger.respond_to?(:tagged)
                   logger.tagged(Tracing.log_correlation, &block)
                 else
                   block.call

data/lib/datadog/tracing/contrib/active_job/patcher.rb

@@ -21,7 +21,7 @@ module Datadog
 
           def patch
             Events.subscribe!
-            inject_log_correlation if Datadog.configuration.tracing.log_injection
+            inject_log_correlation
           end
 
           def inject_log_correlation

data/lib/datadog/tracing/contrib/active_model_serializers/configuration/settings.rb

@@ -12,18 +12,21 @@ module Datadog
           # @public_api
           class Settings < Contrib::Configuration::Settings
             option :enabled do |o|
-              o.default { env_to_bool(Ext::ENV_ENABLED, true) }
-              o.lazy
+              o.type :bool
+              o.env Ext::ENV_ENABLED
+              o.default true
             end
 
             option :analytics_enabled do |o|
-              o.default { env_to_bool(Ext::ENV_ANALYTICS_ENABLED, false) }
-              o.lazy
+              o.type :bool
+              o.env Ext::ENV_ANALYTICS_ENABLED
+              o.default false
             end
 
             option :analytics_sample_rate do |o|
-              o.default { env_to_float(Ext::ENV_ANALYTICS_SAMPLE_RATE, 1.0) }
-              o.lazy
+              o.type :float
+              o.env Ext::ENV_ANALYTICS_SAMPLE_RATE
+              o.default 1.0
             end
           end
         end

data/lib/datadog/tracing/contrib/active_model_serializers/ext.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Datadog
   module Tracing
     module Contrib
@@ -5,16 +7,16 @@ module Datadog
         # ActiveModelSerializers integration constants
         # @public_api Changing resource names, tag names, or environment variables creates breaking changes.
         module Ext
-          ENV_ENABLED = 'DD_TRACE_ACTIVE_MODEL_SERIALIZERS_ENABLED'.freeze
-          ENV_ANALYTICS_ENABLED = 'DD_TRACE_ACTIVE_MODEL_SERIALIZERS_ANALYTICS_ENABLED'.freeze
-          ENV_ANALYTICS_SAMPLE_RATE = 'DD_TRACE_ACTIVE_MODEL_SERIALIZERS_ANALYTICS_SAMPLE_RATE'.freeze
-          SPAN_RENDER = 'active_model_serializers.render'.freeze
-          SPAN_SERIALIZE = 'active_model_serializers.serialize'.freeze
-          TAG_ADAPTER = 'active_model_serializers.adapter'.freeze
-          TAG_COMPONENT = 'active_model_serializers'.freeze
-          TAG_OPERATION_RENDER = 'render'.freeze
-          TAG_OPERATION_SERIALIZE = 'serialize'.freeze
-          TAG_SERIALIZER = 'active_model_serializers.serializer'.freeze
+          ENV_ENABLED = 'DD_TRACE_ACTIVE_MODEL_SERIALIZERS_ENABLED'
+          ENV_ANALYTICS_ENABLED = 'DD_TRACE_ACTIVE_MODEL_SERIALIZERS_ANALYTICS_ENABLED'
+          ENV_ANALYTICS_SAMPLE_RATE = 'DD_TRACE_ACTIVE_MODEL_SERIALIZERS_ANALYTICS_SAMPLE_RATE'
+          SPAN_RENDER = 'active_model_serializers.render'
+          SPAN_SERIALIZE = 'active_model_serializers.serialize'
+          TAG_ADAPTER = 'active_model_serializers.adapter'
+          TAG_COMPONENT = 'active_model_serializers'
+          TAG_OPERATION_RENDER = 'render'
+          TAG_OPERATION_SERIALIZE = 'serialize'
+          TAG_SERIALIZER = 'active_model_serializers.serializer'
         end
       end
     end

data/lib/datadog/tracing/contrib/active_record/configuration/settings.rb

@@ -13,18 +13,21 @@ module Datadog
           # @public_api
           class Settings < Contrib::Configuration::Settings
             option :enabled do |o|
-              o.default { env_to_bool(Ext::ENV_ENABLED, true) }
-              o.lazy
+              o.type :bool
+              o.env Ext::ENV_ENABLED
+              o.default true
             end
 
             option :analytics_enabled do |o|
-              o.default { env_to_bool(Ext::ENV_ANALYTICS_ENABLED, false) }
-              o.lazy
+              o.type :bool
+              o.env Ext::ENV_ANALYTICS_ENABLED
+              o.default false
             end
 
             option :analytics_sample_rate do |o|
-              o.default { env_to_float(Ext::ENV_ANALYTICS_SAMPLE_RATE, 1.0) }
-              o.lazy
+              o.type :float
+              o.env Ext::ENV_ANALYTICS_SAMPLE_RATE
+              o.default 1.0
             end
 
             option :service_name do |o|
@@ -34,7 +37,6 @@ module Datadog
                   Utils.adapter_name
                 )
               end
-              o.lazy
             end
           end
         end

data/lib/datadog/tracing/contrib/active_record/events/sql.rb

@@ -45,14 +45,6 @@ module Datadog
               span.set_tag(Tracing::Metadata::Ext::TAG_COMPONENT, Ext::TAG_COMPONENT)
               span.set_tag(Tracing::Metadata::Ext::TAG_OPERATION, Ext::TAG_OPERATION_SQL)
 
-              # Tag as an external peer service
-              if Contrib::SpanAttributeSchema.default_span_attribute_schema?
-                span.set_tag(Tracing::Metadata::Ext::TAG_PEER_SERVICE, span.service)
-              end
-
-              # TODO: Populate hostname for JDBC connections
-              span.set_tag(Tracing::Metadata::Ext::TAG_PEER_HOSTNAME, config[:host]) if config[:host]
-
               # Set analytics sample rate
               if Contrib::Analytics.enabled?(configuration[:analytics_enabled])
                 Contrib::Analytics.set_sample_rate(span, configuration[:analytics_sample_rate])

data/lib/datadog/tracing/contrib/active_record/ext.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Datadog
   module Tracing
     module Contrib
@@ -5,21 +7,21 @@ module Datadog
         # ActiveRecord integration constants
         # @public_api Changing resource names, tag names, or environment variables creates breaking changes.
         module Ext
-          ENV_ENABLED = 'DD_TRACE_ACTIVE_RECORD_ENABLED'.freeze
-          ENV_ANALYTICS_ENABLED = 'DD_TRACE_ACTIVE_RECORD_ANALYTICS_ENABLED'.freeze
-          ENV_ANALYTICS_SAMPLE_RATE = 'DD_TRACE_ACTIVE_RECORD_ANALYTICS_SAMPLE_RATE'.freeze
-          SERVICE_NAME = 'active_record'.freeze
-          SPAN_INSTANTIATION = 'active_record.instantiation'.freeze
-          SPAN_SQL = 'active_record.sql'.freeze
-          SPAN_TYPE_INSTANTIATION = 'custom'.freeze
-          TAG_COMPONENT = 'active_record'.freeze
-          TAG_OPERATION_INSTANTIATION = 'instantiation'.freeze
-          TAG_OPERATION_SQL = 'sql'.freeze
-          TAG_DB_CACHED = 'active_record.db.cached'.freeze
-          TAG_DB_NAME = 'active_record.db.name'.freeze
-          TAG_DB_VENDOR = 'active_record.db.vendor'.freeze
-          TAG_INSTANTIATION_CLASS_NAME = 'active_record.instantiation.class_name'.freeze
-          TAG_INSTANTIATION_RECORD_COUNT = 'active_record.instantiation.record_count'.freeze
+          ENV_ENABLED = 'DD_TRACE_ACTIVE_RECORD_ENABLED'
+          ENV_ANALYTICS_ENABLED = 'DD_TRACE_ACTIVE_RECORD_ANALYTICS_ENABLED'
+          ENV_ANALYTICS_SAMPLE_RATE = 'DD_TRACE_ACTIVE_RECORD_ANALYTICS_SAMPLE_RATE'
+          SERVICE_NAME = 'active_record'
+          SPAN_INSTANTIATION = 'active_record.instantiation'
+          SPAN_SQL = 'active_record.sql'
+          SPAN_TYPE_INSTANTIATION = 'custom'
+          TAG_COMPONENT = 'active_record'
+          TAG_OPERATION_INSTANTIATION = 'instantiation'
+          TAG_OPERATION_SQL = 'sql'
+          TAG_DB_CACHED = 'active_record.db.cached'
+          TAG_DB_NAME = 'active_record.db.name'
+          TAG_DB_VENDOR = 'active_record.db.vendor'
+          TAG_INSTANTIATION_CLASS_NAME = 'active_record.instantiation.class_name'
+          TAG_INSTANTIATION_RECORD_COUNT = 'active_record.instantiation.record_count'
         end
       end
     end

data/lib/datadog/tracing/contrib/active_support/cache/instrumentation.rb

@@ -42,11 +42,6 @@ module Datadog
               span.set_tag(Tracing::Metadata::Ext::TAG_COMPONENT, Ext::TAG_COMPONENT)
               span.set_tag(Tracing::Metadata::Ext::TAG_OPERATION, Ext::TAG_OPERATION_CACHE)
 
-              if Contrib::SpanAttributeSchema.default_span_attribute_schema?
-                # Tag as an external peer service
-                span.set_tag(Tracing::Metadata::Ext::TAG_PEER_SERVICE, service)
-              end
-
               tracing_context[:dd_cache_span] = span
             rescue StandardError => e
               Datadog.logger.debug(e.message)

data/lib/datadog/tracing/contrib/active_support/configuration/settings.rb

@@ -12,18 +12,21 @@ module Datadog
           # @public_api
           class Settings < Contrib::Configuration::Settings
             option :enabled do |o|
-              o.default { env_to_bool(Ext::ENV_ENABLED, true) }
-              o.lazy
+              o.type :bool
+              o.env Ext::ENV_ENABLED
+              o.default true
             end
 
             option :analytics_enabled do |o|
-              o.default { env_to_bool(Ext::ENV_ANALYTICS_ENABLED, false) }
-              o.lazy
+              o.type :bool
+              o.env Ext::ENV_ANALYTICS_ENABLED
+              o.default false
             end
 
             option :analytics_sample_rate do |o|
-              o.default { env_to_float(Ext::ENV_ANALYTICS_SAMPLE_RATE, 1.0) }
-              o.lazy
+              o.type :float
+              o.env Ext::ENV_ANALYTICS_SAMPLE_RATE
+              o.default 1.0
             end
 
             option :cache_service do |o|
@@ -33,7 +36,6 @@ module Datadog
                   Ext::SERVICE_CACHE
                 )
               end
-              o.lazy
             end
           end
         end

data/lib/datadog/tracing/contrib/active_support/ext.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Datadog
   module Tracing
     module Contrib
@@ -5,23 +7,23 @@ module Datadog
         # ActiveSupport integration constants
         # @public_api Changing resource names, tag names, or environment variables creates breaking changes.
         module Ext
-          ENV_ENABLED = 'DD_TRACE_ACTIVE_SUPPORT_ENABLED'.freeze
-          ENV_ANALYTICS_ENABLED = 'DD_TRACE_ACTIVE_SUPPORT_ANALYTICS_ENABLED'.freeze
-          ENV_ANALYTICS_SAMPLE_RATE = 'DD_TRACE_ACTIVE_SUPPORT_ANALYTICS_SAMPLE_RATE'.freeze
+          ENV_ENABLED = 'DD_TRACE_ACTIVE_SUPPORT_ENABLED'
+          ENV_ANALYTICS_ENABLED = 'DD_TRACE_ACTIVE_SUPPORT_ANALYTICS_ENABLED'
+          ENV_ANALYTICS_SAMPLE_RATE = 'DD_TRACE_ACTIVE_SUPPORT_ANALYTICS_SAMPLE_RATE'
           QUANTIZE_CACHE_MAX_KEY_SIZE = 300
-          RESOURCE_CACHE_DELETE = 'DELETE'.freeze
-          RESOURCE_CACHE_GET = 'GET'.freeze
-          RESOURCE_CACHE_MGET = 'MGET'.freeze
-          RESOURCE_CACHE_SET = 'SET'.freeze
-          RESOURCE_CACHE_MSET = 'MSET'.freeze
-          SERVICE_CACHE = 'active_support-cache'.freeze
-          SPAN_CACHE = 'rails.cache'.freeze
-          SPAN_TYPE_CACHE = 'cache'.freeze
-          TAG_CACHE_BACKEND = 'rails.cache.backend'.freeze
-          TAG_CACHE_KEY = 'rails.cache.key'.freeze
-          TAG_CACHE_KEY_MULTI = 'rails.cache.keys'.freeze
-          TAG_COMPONENT = 'active_support'.freeze
-          TAG_OPERATION_CACHE = 'cache'.freeze
+          RESOURCE_CACHE_DELETE = 'DELETE'
+          RESOURCE_CACHE_GET = 'GET'
+          RESOURCE_CACHE_MGET = 'MGET'
+          RESOURCE_CACHE_SET = 'SET'
+          RESOURCE_CACHE_MSET = 'MSET'
+          SERVICE_CACHE = 'active_support-cache'
+          SPAN_CACHE = 'rails.cache'
+          SPAN_TYPE_CACHE = 'cache'
+          TAG_CACHE_BACKEND = 'rails.cache.backend'
+          TAG_CACHE_KEY = 'rails.cache.key'
+          TAG_CACHE_KEY_MULTI = 'rails.cache.keys'
+          TAG_COMPONENT = 'active_support'
+          TAG_OPERATION_CACHE = 'cache'
         end
       end
     end

data/lib/datadog/tracing/contrib/aws/configuration/settings.rb

@@ -13,18 +13,21 @@ module Datadog
           # @public_api
           class Settings < Contrib::Configuration::Settings
             option :enabled do |o|
-              o.default { env_to_bool(Ext::ENV_ENABLED, true) }
-              o.lazy
+              o.type :bool
+              o.env Ext::ENV_ENABLED
+              o.default true
             end
 
             option :analytics_enabled do |o|
-              o.default { env_to_bool(Ext::ENV_ANALYTICS_ENABLED, false) }
-              o.lazy
+              o.type :bool
+              o.env Ext::ENV_ANALYTICS_ENABLED
+              o.default false
             end
 
             option :analytics_sample_rate do |o|
-              o.default { env_to_float(Ext::ENV_ANALYTICS_SAMPLE_RATE, 1.0) }
-              o.lazy
+              o.type :float
+              o.env Ext::ENV_ANALYTICS_SAMPLE_RATE
+              o.default 1.0
             end
 
             option :service_name do |o|
@@ -34,7 +37,11 @@ module Datadog
                   Ext::DEFAULT_PEER_SERVICE_NAME
                 )
               end
-              o.lazy
+            end
+
+            option :peer_service do |o|
+              o.type :string, nilable: true
+              o.env Ext::ENV_PEER_SERVICE
             end
           end
         end

data/lib/datadog/tracing/contrib/aws/ext.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Datadog
   module Tracing
     module Contrib
@@ -5,30 +7,41 @@ module Datadog
         # AWS integration constants
         # @public_api Changing resource names, tag names, or environment variables creates breaking changes.
         module Ext
-          ENV_ENABLED = 'DD_TRACE_AWS_ENABLED'.freeze
-          ENV_SERVICE_NAME = 'DD_TRACE_AWS_SERVICE_NAME'.freeze
-          ENV_ANALYTICS_ENABLED = 'DD_TRACE_AWS_ANALYTICS_ENABLED'.freeze
-          ENV_ANALYTICS_SAMPLE_RATE = 'DD_TRACE_AWS_ANALYTICS_SAMPLE_RATE'.freeze
-          DEFAULT_PEER_SERVICE_NAME = 'aws'.freeze
-          SPAN_COMMAND = 'aws.command'.freeze
-          TAG_AGENT = 'aws.agent'.freeze
-          TAG_COMPONENT = 'aws'.freeze
-          TAG_DEFAULT_AGENT = 'aws-sdk-ruby'.freeze
-          TAG_HOST = 'host'.freeze
-          TAG_OPERATION = 'aws.operation'.freeze
-          TAG_OPERATION_COMMAND = 'command'.freeze
-          TAG_PATH = 'path'.freeze
-          TAG_AWS_REGION = 'aws.region'.freeze
-          TAG_REGION = 'region'.freeze
-          TAG_AWS_SERVICE = 'aws_service'.freeze
-          TAG_AWS_ACCOUNT = 'aws_account'.freeze
-          TAG_QUEUE_NAME = 'queuename'.freeze
-          TAG_TOPIC_NAME = 'topicname'.freeze
-          TAG_TABLE_NAME = 'tablename'.freeze
-          TAG_STREAM_NAME = 'streamname'.freeze
-          TAG_RULE_NAME = 'rulename'.freeze
-          TAG_STATE_MACHINE_NAME = 'statemachinename'.freeze
-          TAG_BUCKET_NAME = 'bucketname'.freeze
+          ENV_ENABLED = 'DD_TRACE_AWS_ENABLED'
+          ENV_SERVICE_NAME = 'DD_TRACE_AWS_SERVICE_NAME'
+          ENV_PEER_SERVICE = 'DD_TRACE_AWS_PEER_SERVICE'
+          ENV_ANALYTICS_ENABLED = 'DD_TRACE_AWS_ANALYTICS_ENABLED'
+          ENV_ANALYTICS_SAMPLE_RATE = 'DD_TRACE_AWS_ANALYTICS_SAMPLE_RATE'
+          DEFAULT_PEER_SERVICE_NAME = 'aws'
+          SPAN_COMMAND = 'aws.command'
+          TAG_AGENT = 'aws.agent'
+          TAG_COMPONENT = 'aws'
+          TAG_DEFAULT_AGENT = 'aws-sdk-ruby'
+          TAG_HOST = 'host'
+          TAG_OPERATION = 'aws.operation'
+          TAG_OPERATION_COMMAND = 'command'
+          TAG_PATH = 'path'
+          TAG_AWS_REGION = 'aws.region'
+          TAG_REGION = 'region'
+          TAG_AWS_SERVICE = 'aws_service'
+          TAG_AWS_ACCOUNT = 'aws_account'
+          TAG_QUEUE_NAME = 'queuename'
+          TAG_TOPIC_NAME = 'topicname'
+          TAG_TABLE_NAME = 'tablename'
+          TAG_STREAM_NAME = 'streamname'
+          TAG_RULE_NAME = 'rulename'
+          TAG_STATE_MACHINE_NAME = 'statemachinename'
+          TAG_BUCKET_NAME = 'bucketname'
+          PEER_SERVICE_SOURCES = Array[TAG_QUEUE_NAME,
+            TAG_TOPIC_NAME,
+            TAG_STREAM_NAME,
+            TAG_TABLE_NAME,
+            TAG_BUCKET_NAME,
+            TAG_RULE_NAME,
+            TAG_STATE_MACHINE_NAME,
+            Tracing::Metadata::Ext::TAG_PEER_HOSTNAME,
+            Tracing::Metadata::Ext::NET::TAG_DESTINATION_NAME,
+            Tracing::Metadata::Ext::NET::TAG_TARGET_HOST,].freeze
         end
       end
     end

data/lib/datadog/tracing/contrib/aws/instrumentation.rb

@@ -40,16 +40,18 @@ module Datadog
               handler.add_tags(span, params)
             end
 
+            if configuration[:peer_service]
+              span.set_tag(
+                Tracing::Metadata::Ext::TAG_PEER_SERVICE,
+                configuration[:peer_service]
+              )
+            end
+
             span.set_tag(Tracing::Metadata::Ext::TAG_KIND, Tracing::Metadata::Ext::SpanKind::TAG_CLIENT)
 
             span.set_tag(Tracing::Metadata::Ext::TAG_COMPONENT, Ext::TAG_COMPONENT)
             span.set_tag(Tracing::Metadata::Ext::TAG_OPERATION, Ext::TAG_OPERATION_COMMAND)
 
-            # Tag as an external peer service
-            if Contrib::SpanAttributeSchema.default_span_attribute_schema?
-              span.set_tag(Tracing::Metadata::Ext::TAG_PEER_SERVICE, span.service)
-            end
-
             span.set_tag(Tracing::Metadata::Ext::TAG_PEER_HOSTNAME, context.safely(:host))
 
             # Set analytics sample rate
@@ -66,6 +68,8 @@ module Datadog
             span.set_tag(Ext::TAG_HOST, context.safely(:host))
             span.set_tag(Tracing::Metadata::Ext::HTTP::TAG_METHOD, context.safely(:http_method))
             span.set_tag(Tracing::Metadata::Ext::HTTP::TAG_STATUS_CODE, context.safely(:status_code))
+
+            Contrib::SpanAttributeSchema.set_peer_service!(span, Ext::PEER_SERVICE_SOURCES)
           end
 
           def configuration

data/lib/datadog/tracing/contrib/concurrent_ruby/configuration/settings.rb

@@ -12,8 +12,9 @@ module Datadog
           # @public_api
           class Settings < Contrib::Configuration::Settings
             option :enabled do |o|
-              o.default { env_to_bool(Ext::ENV_ENABLED, true) }
-              o.lazy
+              o.type :bool
+              o.env Ext::ENV_ENABLED
+              o.default true
             end
           end
         end

data/lib/datadog/tracing/contrib/concurrent_ruby/ext.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Datadog
   module Tracing
     module Contrib
@@ -5,8 +7,8 @@ module Datadog
         # ConcurrentRuby integration constants
         # @public_api Changing resource names, tag names, or environment variables creates breaking changes.
         module Ext
-          APP = 'concurrent-ruby'.freeze
-          ENV_ENABLED = 'DD_TRACE_CONCURRENT_RUBY_ENABLED'.freeze
+          APP = 'concurrent-ruby'
+          ENV_ENABLED = 'DD_TRACE_CONCURRENT_RUBY_ENABLED'
         end
       end
     end