ddtrace 0.35.1 → 0.39.0

data/lib/ddtrace/workers/trace_writer.rb

@@ -72,6 +72,9 @@ module Datadog
         end
       end
 
+      # TODO: Register `Datadog::Diagnostics::EnvironmentLogger.log!`
+      # TODO: as a flush_completed subscriber when the `TraceWriter`
+      # TODO: instantiation code is implemented.
       def flush_completed
         @flush_completed ||= FlushCompleted.new
       end

data/lib/ddtrace/writer.rb

@@ -7,6 +7,7 @@ require 'ddtrace/transport/http'
 require 'ddtrace/transport/io'
 require 'ddtrace/encoding'
 require 'ddtrace/workers'
+require 'ddtrace/diagnostics/environment_logger'
 
 module Datadog
   # Processor that sends traces and metadata to the agent
@@ -43,9 +44,18 @@ module Datadog
       @worker = nil
     end
 
-    # spawns a worker for spans; they share the same transport which is thread-safe
     def start
-      @pid = Process.pid
+      @mutex_after_fork.synchronize do
+        pid = Process.pid
+        return if @worker && pid == @pid
+        @pid = pid
+        start_worker
+        true
+      end
+    end
+
+    # spawns a worker for spans; they share the same transport which is thread-safe
+    def start_worker
       @trace_handler = ->(items, transport) { send_spans(items, transport) }
       @worker = Datadog::Workers::AsyncTransport.new(
         transport: @transport,
@@ -57,14 +67,19 @@ module Datadog
       @worker.start
     end
 
-    # stops worker for spans.
     def stop
-      return if worker.nil?
+      @mutex_after_fork.synchronize { stop_worker }
+    end
+
+    def stop_worker
+      return if @worker.nil?
       @worker.stop
       @worker = nil
       true
     end
 
+    private :start_worker, :stop_worker
+
     # flush spans to the trace-agent, handles spans only
     def send_spans(traces, transport)
       return true if traces.empty?
@@ -83,6 +98,8 @@ module Datadog
       # Update priority sampler
       update_priority_sampler(responses.last)
 
+      record_environment_information!(responses)
+
       # Return if server error occurred.
       !responses.find(&:server_error?)
     end
@@ -106,13 +123,7 @@ module Datadog
       #
       # This check ensures that if a process doesn't own the current +Writer+, async workers
       # will be initialized again (but only once for each process).
-      pid = Process.pid
-      if pid != @pid # avoid using Mutex when pids are equal
-        @mutex_after_fork.synchronize do
-          # we should start threads because the worker doesn't own this
-          start if pid != @pid
-        end
-      end
+      start if @worker.nil? || @pid != Process.pid
 
       # TODO: Remove this, and have the tracer pump traces directly to runtime metrics
       #       instead of working through the trace writer.
@@ -121,7 +132,13 @@ module Datadog
         Datadog.runtime_metrics.associate_with_span(trace.first)
       end
 
-      @worker.enqueue_trace(trace)
+      worker_local = @worker
+
+      if worker_local
+        worker_local.enqueue_trace(trace)
+      else
+        Datadog.logger.debug('Writer either failed to start or was stopped before #write could complete')
+      end
     end
 
     # stats returns a dictionary of stats about the writer.
@@ -150,5 +167,9 @@ module Datadog
 
       priority_sampler.update(response.service_rates)
     end
+
+    def record_environment_information!(responses)
+      Diagnostics::EnvironmentLogger.log!(responses)
+    end
   end
 end

metadata

@@ -1,22 +1,22 @@
 --- !ruby/object:Gem::Specification
 name: ddtrace
 version: !ruby/object:Gem::Version
-  version: 0.35.1
+  version: 0.39.0
 platform: ruby
 authors:
 - Datadog, Inc.
-autorequire: 
+autorequire:
 bindir: exe
 cert_chain: []
-date: 2020-05-05 00:00:00.000000000 Z
+date: 2020-08-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: msgpack
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+  name: msgpack
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -25,12 +25,12 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: dogstatsd-ruby
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: 3.3.0
+  name: dogstatsd-ruby
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -39,12 +39,12 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 3.3.0
 - !ruby/object:Gem::Dependency
-  name: opentracing
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: 0.4.1
+  name: opentracing
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -53,12 +53,12 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 0.4.1
 - !ruby/object:Gem::Dependency
-  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '10.5'
+  name: rake
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -67,12 +67,12 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '10.5'
 - !ruby/object:Gem::Dependency
-  name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
         version: 0.49.1
+  name: rubocop
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -81,12 +81,12 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 0.49.1
 - !ruby/object:Gem::Dependency
-  name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3.0'
+  name: rspec
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -95,12 +95,12 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '3.0'
 - !ruby/object:Gem::Dependency
-  name: rspec-collection_matchers
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.1'
+  name: rspec-collection_matchers
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -109,12 +109,12 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '1.1'
 - !ruby/object:Gem::Dependency
-  name: minitest
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
         version: 5.10.1
+  name: minitest
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -123,12 +123,40 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 5.10.1
 - !ruby/object:Gem::Dependency
-  name: appraisal
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 0.5.0
+  name: minitest-around
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 0.5.0
+- !ruby/object:Gem::Dependency
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 1.0.2
+  name: minitest-stub_any_instance
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 1.0.2
+- !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '2.2'
+  name: appraisal
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -137,12 +165,12 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '2.2'
 - !ruby/object:Gem::Dependency
-  name: yard
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.9'
+  name: yard
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -151,12 +179,12 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0.9'
 - !ruby/object:Gem::Dependency
-  name: webmock
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '2.0'
+  name: webmock
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -165,12 +193,12 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '2.0'
 - !ruby/object:Gem::Dependency
-  name: builder
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+  name: builder
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -179,26 +207,26 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: sqlite3
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.3.6
+        version: '3'
+  name: jdbc-sqlite3
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.3.6
+        version: '3'
 - !ruby/object:Gem::Dependency
-  name: climate_control
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 0.2.0
+  name: climate_control
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -207,12 +235,12 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 0.2.0
 - !ruby/object:Gem::Dependency
-  name: addressable
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 2.4.0
+  name: addressable
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -221,26 +249,54 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 2.4.0
 - !ruby/object:Gem::Dependency
-  name: redcarpet
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.4'
+        version: '2.8'
+  name: benchmark-ips
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.4'
+        version: '2.8'
+- !ruby/object:Gem::Dependency
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.1'
+  name: benchmark-memory
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.1'
+- !ruby/object:Gem::Dependency
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.9'
+  name: memory_profiler
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.9'
 - !ruby/object:Gem::Dependency
-  name: pry
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 0.10.4
+  name: pry
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -249,12 +305,12 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 0.10.4
 - !ruby/object:Gem::Dependency
-  name: pry-stack_explorer
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 0.4.9.2
+  name: pry-stack_explorer
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -263,12 +319,12 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 0.4.9.2
 - !ruby/object:Gem::Dependency
-  name: warning
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1'
+  name: warning
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -300,6 +356,7 @@ files:
 - ".env"
 - ".github/CODEOWNERS"
 - ".gitignore"
+- ".gitlab-ci.yml"
 - ".rspec"
 - ".rubocop.yml"
 - ".yardopts"
@@ -475,7 +532,29 @@ files:
 - lib/ddtrace/contrib/http/integration.rb
 - lib/ddtrace/contrib/http/patcher.rb
 - lib/ddtrace/contrib/http_annotation_helper.rb
+- lib/ddtrace/contrib/httprb/configuration/settings.rb
+- lib/ddtrace/contrib/httprb/ext.rb
+- lib/ddtrace/contrib/httprb/instrumentation.rb
+- lib/ddtrace/contrib/httprb/integration.rb
+- lib/ddtrace/contrib/httprb/patcher.rb
 - lib/ddtrace/contrib/integration.rb
+- lib/ddtrace/contrib/kafka/configuration/settings.rb
+- lib/ddtrace/contrib/kafka/consumer_event.rb
+- lib/ddtrace/contrib/kafka/consumer_group_event.rb
+- lib/ddtrace/contrib/kafka/event.rb
+- lib/ddtrace/contrib/kafka/events.rb
+- lib/ddtrace/contrib/kafka/events/connection/request.rb
+- lib/ddtrace/contrib/kafka/events/consumer/process_batch.rb
+- lib/ddtrace/contrib/kafka/events/consumer/process_message.rb
+- lib/ddtrace/contrib/kafka/events/consumer_group/heartbeat.rb
+- lib/ddtrace/contrib/kafka/events/consumer_group/join_group.rb
+- lib/ddtrace/contrib/kafka/events/consumer_group/leave_group.rb
+- lib/ddtrace/contrib/kafka/events/consumer_group/sync_group.rb
+- lib/ddtrace/contrib/kafka/events/produce_operation/send_messages.rb
+- lib/ddtrace/contrib/kafka/events/producer/deliver_messages.rb
+- lib/ddtrace/contrib/kafka/ext.rb
+- lib/ddtrace/contrib/kafka/integration.rb
+- lib/ddtrace/contrib/kafka/patcher.rb
 - lib/ddtrace/contrib/mongodb/configuration/settings.rb
 - lib/ddtrace/contrib/mongodb/ext.rb
 - lib/ddtrace/contrib/mongodb/instrumentation.rb
@@ -569,6 +648,11 @@ files:
 - lib/ddtrace/contrib/sinatra/patcher.rb
 - lib/ddtrace/contrib/sinatra/tracer.rb
 - lib/ddtrace/contrib/sinatra/tracer_middleware.rb
+- lib/ddtrace/contrib/sneakers/configuration/settings.rb
+- lib/ddtrace/contrib/sneakers/ext.rb
+- lib/ddtrace/contrib/sneakers/integration.rb
+- lib/ddtrace/contrib/sneakers/patcher.rb
+- lib/ddtrace/contrib/sneakers/tracer.rb
 - lib/ddtrace/contrib/sucker_punch/configuration/settings.rb
 - lib/ddtrace/contrib/sucker_punch/exception_handler.rb
 - lib/ddtrace/contrib/sucker_punch/ext.rb
@@ -576,6 +660,7 @@ files:
 - lib/ddtrace/contrib/sucker_punch/integration.rb
 - lib/ddtrace/contrib/sucker_punch/patcher.rb
 - lib/ddtrace/correlation.rb
+- lib/ddtrace/diagnostics/environment_logger.rb
 - lib/ddtrace/diagnostics/health.rb
 - lib/ddtrace/distributed_tracing/headers/b3.rb
 - lib/ddtrace/distributed_tracing/headers/b3_single.rb
@@ -698,7 +783,7 @@ licenses:
 - BSD-3-Clause
 metadata:
   allowed_push_host: https://rubygems.org
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -713,8 +798,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: 2.0.0
 requirements: []
-rubygems_version: 3.1.2
-signing_key: 
+rubygems_version: 3.0.6
+signing_key:
 specification_version: 4
 summary: Datadog tracing code for your Ruby applications
 test_files: []