ddtrace 1.7.0 → 1.9.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 38409b6f228216767057d98b4acb4cc0effeecd34e6b98e202dde946aa4aec99
-  data.tar.gz: 6a5b909f6a73516c965c1125b7c70fe212972bf89ac829e0cabc356d82a353a0
+  metadata.gz: 1ba4a81593411a12abfa1665b3910e49e35429b1b266779e93accb0f84a478d2
+  data.tar.gz: e72362dc1a1e682a3c52b8af8017fcd12e60d353843cdd129f65a55dbb0c312d
 SHA512:
-  metadata.gz: 73d2114938c7a420a6846042c3172f4b08b5a9fb5b68efc6aa155441fa80d6221201ff1a09afbcbd8ccb12655590622326510cf17995443cd1e5cadca0fedbb3
-  data.tar.gz: 6c8005ac09cb122bbcf32ee569799f61ada81f8e7a735b2b754e0b0f2969a77980b95a1d8871440efe8e617a3c53c77b4f286dceb98be00dd446684ed5bec43b
+  metadata.gz: a083b23cfadccc78e67a5aa2a3ae067bc868e4721d92d1fb9fcbadc7536332fabd20f6638fb157712e5e909afd3e3e505a2011838a4f0e98dbebaa64a38f7db6
+  data.tar.gz: 81b27dbbf685c01ee99ab79a8774aefca4a2feb93363a41e1a3465b01b1b07f571d5bdd75a73880658ae05254a3d742651bdf8df98fd48c040f73bb6a16702d0

data/CHANGELOG.md

@@ -2,6 +2,69 @@
 
 ## [Unreleased]
 
+## [1.9.0] - 2023-01-30
+
+As of ddtrace 1.9.0, CPU Profiling 2.0 is now in opt-in (that is, disabled by default) public beta. For more details, check the release notes.
+
+Release notes: https://github.com/DataDog/dd-trace-rb/releases/tag/v1.9.0
+
+### Added
+
+* Tracing: Add `Stripe` instrumentation ([#2557][])
+* Tracing: Add configurable response codes considered as errors for `Net/HTTP`, `httprb` and `httpclient` ([#2501][], [#2576][])([@caramcc][])
+* Tracing: Flexible header matching for HTTP propagator ([#2504][])
+* Tracing: `OpenTelemetry` Traces support ([#2496][])
+* Tracing: W3C: Propagate unknown values as-is ([#2485][])
+* Appsec: Add event kit API ([#2512][])
+* Profiling: Allow profiler development on arm64 macOS ([#2573][])
+* Core: Add `profiling_enabled` state to environment logger output ([#2541][])
+* Core: Add 'type' to `OptionDefinition` ([#2493][])
+* Allow `debase-ruby_core_source` 3.2.0 to be used ([#2526][])
+
+### Changed
+
+* Profiling: Upgrade to `libdatadog` to `1.0.1.1.0` ([#2530][])
+* Appsec: Update appsec rules `1.4.3` ([#2580][])
+* Ci: Update CI Visibility metadata extraction ([#2586][])
+
+### Fixed
+
+* Profiling: Fix wrong `libdatadog` version being picked during profiler build ([#2531][])
+* Tracing: Support `PG` calls with a block ([#2522][])
+* Ci: Fix error in `teamcity` env vars ([#2562][])
+
+## [1.8.0] - 2022-12-14
+
+Release notes: https://github.com/DataDog/dd-trace-rb/releases/tag/v1.8.0
+
+As of ddtrace 1.8.0, CPU Profiling 2.0 is now in opt-in (that is, disabled by default) public beta. For more details,
+check the release notes.
+
+### Added
+
+* Core: Profiling: [PROF-6559] Mark Ruby CPU Profiling 2.0 as being in beta ([#2489][])
+* Tracing: Attempt to parse future version of TraceContext ([#2473][])
+* Tracing: Add DD_TRACE_PROPAGATION_STYLE option ([#2466][])
+* Integrations: Tracing: SQL comment propagation full mode with traceparent ([#2464][])
+* Integrations: Tracing: Wire W3C propagator to HTTP & gRPC propagation ([#2458][])
+* Integrations: Tracing: Auto-instrumentation with service_name from environmental variable ([#2455][])
+* Core: Integrations: Tracing: Deprecation notice for B3 propagation configuration ([#2454][])
+* Tracing: Add W3C Trace Context propagator ([#2451][])
+* Integrations: Tracing: Redis 5 Instrumentation ([#2428][])
+
+### Changed
+
+* Tracing: Changes `error.msg` to `error.message` for UNC ([#2469][])
+* Tracing: Semicolons not allowed in 'origin' ([#2461][])
+* Core: Dev/refactor: Tracing: Dev/internal: Move Utils#next_id and constants to Tracing::Utils ([#2463][])
+* Core: Dev/refactor: Tracing: Dev/internal: Move Tracing config settings from Core to Tracing ([#2459][])
+* Core: Dev/refactor: Tracing: Dev/internal: Move Tracing diagnostic code from Core to Tracing ([#2453][])
+
+### Fixed
+
+* Integrations: Tracing: Improve redis integration patching ([#2470][])
+* Tracing: Extra testing from W3C spec ([#2460][])
+
 ## [1.7.0] - 2022-11-29
 
 ### Added
@@ -2218,7 +2281,9 @@ Release notes: https://github.com/DataDog/dd-trace-rb/releases/tag/v0.3.1
 
 Git diff: https://github.com/DataDog/dd-trace-rb/compare/v0.3.0...v0.3.1
 
-[Unreleased]: https://github.com/DataDog/dd-trace-rb/compare/v1.7.0...master
+[Unreleased]: https://github.com/DataDog/dd-trace-rb/compare/v1.9.0...master
+[1.9.0]: https://github.com/DataDog/dd-trace-rb/compare/v1.8.0...v1.9.0
+[1.8.0]: https://github.com/DataDog/dd-trace-rb/compare/v1.7.0...v1.8.0
 [1.7.0]: https://github.com/DataDog/dd-trace-rb/compare/v1.6.1...v1.7.0
 [1.6.1]: https://github.com/DataDog/dd-trace-rb/compare/v1.6.0...v1.6.1
 [1.6.0]: https://github.com/DataDog/dd-trace-rb/compare/v1.5.2...v1.6.0
@@ -3161,7 +3226,40 @@ Git diff: https://github.com/DataDog/dd-trace-rb/compare/v0.3.0...v0.3.1
 [#2413]: https://github.com/DataDog/dd-trace-rb/issues/2413
 [#2419]: https://github.com/DataDog/dd-trace-rb/issues/2419
 [#2420]: https://github.com/DataDog/dd-trace-rb/issues/2420
+[#2428]: https://github.com/DataDog/dd-trace-rb/issues/2428
 [#2435]: https://github.com/DataDog/dd-trace-rb/issues/2435
+[#2451]: https://github.com/DataDog/dd-trace-rb/issues/2451
+[#2453]: https://github.com/DataDog/dd-trace-rb/issues/2453
+[#2454]: https://github.com/DataDog/dd-trace-rb/issues/2454
+[#2455]: https://github.com/DataDog/dd-trace-rb/issues/2455
+[#2458]: https://github.com/DataDog/dd-trace-rb/issues/2458
+[#2459]: https://github.com/DataDog/dd-trace-rb/issues/2459
+[#2460]: https://github.com/DataDog/dd-trace-rb/issues/2460
+[#2461]: https://github.com/DataDog/dd-trace-rb/issues/2461
+[#2463]: https://github.com/DataDog/dd-trace-rb/issues/2463
+[#2464]: https://github.com/DataDog/dd-trace-rb/issues/2464
+[#2466]: https://github.com/DataDog/dd-trace-rb/issues/2466
+[#2469]: https://github.com/DataDog/dd-trace-rb/issues/2469
+[#2470]: https://github.com/DataDog/dd-trace-rb/issues/2470
+[#2473]: https://github.com/DataDog/dd-trace-rb/issues/2473
+[#2485]: https://github.com/DataDog/dd-trace-rb/issues/2485
+[#2489]: https://github.com/DataDog/dd-trace-rb/issues/2489
+[#2493]: https://github.com/DataDog/dd-trace-rb/issues/2493
+[#2496]: https://github.com/DataDog/dd-trace-rb/issues/2496
+[#2501]: https://github.com/DataDog/dd-trace-rb/issues/2501
+[#2504]: https://github.com/DataDog/dd-trace-rb/issues/2504
+[#2512]: https://github.com/DataDog/dd-trace-rb/issues/2512
+[#2522]: https://github.com/DataDog/dd-trace-rb/issues/2522
+[#2526]: https://github.com/DataDog/dd-trace-rb/issues/2526
+[#2530]: https://github.com/DataDog/dd-trace-rb/issues/2530
+[#2531]: https://github.com/DataDog/dd-trace-rb/issues/2531
+[#2541]: https://github.com/DataDog/dd-trace-rb/issues/2541
+[#2557]: https://github.com/DataDog/dd-trace-rb/issues/2557
+[#2562]: https://github.com/DataDog/dd-trace-rb/issues/2562
+[#2573]: https://github.com/DataDog/dd-trace-rb/issues/2573
+[#2576]: https://github.com/DataDog/dd-trace-rb/issues/2576
+[#2580]: https://github.com/DataDog/dd-trace-rb/issues/2580
+[#2586]: https://github.com/DataDog/dd-trace-rb/issues/2586
 [@AdrianLC]: https://github.com/AdrianLC
 [@Azure7111]: https://github.com/Azure7111
 [@BabyGroot]: https://github.com/BabyGroot
@@ -3200,6 +3298,7 @@ Git diff: https://github.com/DataDog/dd-trace-rb/compare/v0.3.0...v0.3.1
 [@brafales]: https://github.com/brafales
 [@bzf]: https://github.com/bzf
 [@callumj]: https://github.com/callumj
+[@caramcc]: https://github.com/caramcc
 [@carlallen]: https://github.com/carlallen
 [@chychkan]: https://github.com/chychkan
 [@cjford]: https://github.com/cjford

data/README.md

@@ -1,5 +1,3 @@
-**We've recently released the 1.x version series. If you're upgrading from a 0.x version, check out our [upgrade guide](https://github.com/DataDog/dd-trace-rb/blob/master/docs/UpgradeGuide.md#from-0x-to-10).**
-
 # Datadog Trace Client
 
 [![Gem](https://img.shields.io/gem/v/ddtrace)](https://rubygems.org/gems/ddtrace/)
@@ -12,6 +10,8 @@ databases and microservices so that developers have great visiblity into bottlen
 
 ## Getting started
 
+**If you're upgrading from a 0.x version, check out our [upgrade guide](https://github.com/DataDog/dd-trace-rb/blob/master/docs/UpgradeGuide.md#from-0x-to-10).**
+
 For a basic product overview, check out our [setup documentation][setup docs].
 
 For installation, configuration, and details about using the API, check out our [API documentation][api docs] and [gem documentation][gem docs].

data/ext/ddtrace_profiling_loader/extconf.rb

@@ -34,7 +34,7 @@ add_compiler_flag '-Werror' if ENV['DDTRACE_CI'] == 'true'
 #   (https://github.com/msgpack/msgpack-ruby/blob/18ce08f6d612fe973843c366ac9a0b74c4e50599/ext/msgpack/extconf.rb#L8)
 add_compiler_flag '-std=gnu99'
 
-# Gets really noisy when we include the MJIT header, let's omit it
+# Gets really noisy when we include the MJIT header, let's omit it (TODO: Use #pragma GCC diagnostic instead?)
 add_compiler_flag '-Wno-unused-function'
 
 # Allow defining variables at any point in a function
@@ -55,6 +55,9 @@ add_compiler_flag '-Wunused-parameter'
 # For more details see https://gcc.gnu.org/wiki/Visibility
 add_compiler_flag '-fvisibility=hidden'
 
+# Avoid legacy C definitions
+add_compiler_flag '-Wold-style-definition'
+
 # Enable all other compiler warnings
 add_compiler_flag '-Wall'
 add_compiler_flag '-Wextra'

data/ext/ddtrace_profiling_native_extension/NativeExtensionDesign.md

@@ -57,7 +57,7 @@ Non-exhaustive list of APIs that cause exceptions to be raised:
 
 * `Check_TypedStruct`, `Check_Type`, `ENFORCE_TYPE`
 * `rb_funcall`
-* `rb_thread_call_without_gvl`, `rb_thread_call_without_gvl2`
+* `rb_thread_call_without_gvl`
 * [Numeric conversion APIs, e.g. `NUM2LONG`, `NUM2INT`, etc.](https://silverhammermba.github.io/emberb/c/?#translation)
 * Our `char_slice_from_ruby_string` helper
 

data/ext/ddtrace_profiling_native_extension/clock_id_from_pthread.c

@@ -7,11 +7,12 @@
 #include <pthread.h>
 #include <time.h>
 #include <errno.h>
-
 #include <ruby.h>
+
 #include "helpers.h"
 #include "private_vm_api_access.h"
 #include "clock_id.h"
+#include "time_helpers.h"
 
 // Validate that our home-cooked pthread_id_for() matches pthread_self() for the current thread
 void self_test_clock_id(void) {
@@ -68,7 +69,7 @@ thread_cpu_time thread_cpu_time_for(thread_cpu_time_id time_id) {
   // TODO: Include the error code in some way in the output?
   if (clock_gettime(time_id.clock_id, &current_cpu) != 0) return error;
 
-  return (thread_cpu_time) {.valid = true, .result_ns = current_cpu.tv_nsec + (current_cpu.tv_sec * 1000 * 1000 * 1000)};
+  return (thread_cpu_time) {.valid = true, .result_ns = current_cpu.tv_nsec + SECONDS_AS_NS(current_cpu.tv_sec)};
 }
 
 #endif

data/ext/ddtrace_profiling_native_extension/collectors_cpu_and_wall_time.c

@@ -1,11 +1,13 @@
 #include <ruby.h>
-#include "helpers.h"
+
 #include "clock_id.h"
+#include "collectors_cpu_and_wall_time.h"
 #include "collectors_stack.h"
+#include "helpers.h"
 #include "libdatadog_helpers.h"
 #include "private_vm_api_access.h"
 #include "stack_recorder.h"
-#include "collectors_cpu_and_wall_time.h"
+#include "time_helpers.h"
 
 // Used to periodically sample threads, recording elapsed CPU-time and Wall-time between samples.
 //
@@ -60,8 +62,6 @@
 
 #define INVALID_TIME -1
 #define THREAD_ID_LIMIT_CHARS 44 // Why 44? "#{2**64} (#{2**64})".size + 1 for \0
-#define RAISE_ON_FAILURE true
-#define DO_NOT_RAISE_ON_FAILURE false
 #define IS_WALL_TIME true
 #define IS_NOT_WALL_TIME false
 #define MISSING_TRACER_CONTEXT_KEY 0
@@ -128,9 +128,8 @@ struct trace_identifiers {
 
   bool valid;
   ddog_CharSlice local_root_span_id;
-  ddog_CharSlice span_id;
+  uint64_t span_id;
   char local_root_span_id_buffer[MAXIMUM_LENGTH_64_BIT_IDENTIFIER];
-  char span_id_buffer[MAXIMUM_LENGTH_64_BIT_IDENTIFIER];
   VALUE trace_endpoint;
 };
 
@@ -148,7 +147,7 @@ static void trigger_sample_for_thread(
   struct cpu_and_wall_time_collector_state *state,
   VALUE thread,
   struct per_thread_context *thread_context,
-  ddog_Slice_i64 metric_values_slice,
+  ddog_Slice_I64 metric_values_slice,
   sample_type type
 );
 static VALUE _native_thread_list(VALUE self);
@@ -164,7 +163,6 @@ static int remove_if_dead_thread(st_data_t key_thread, st_data_t value_context,
 static VALUE _native_per_thread_context(VALUE self, VALUE collector_instance);
 static long update_time_since_previous_sample(long *time_at_previous_sample_ns, long current_time_ns, long gc_start_time_ns, bool is_wall_time);
 static long cpu_time_now_ns(struct per_thread_context *thread_context);
-static long wall_time_now_ns(bool raise_on_failure);
 static long thread_id_for(VALUE thread);
 static VALUE _native_stats(VALUE self, VALUE collector_instance);
 static void trace_identifiers_for(struct cpu_and_wall_time_collector_state *state, VALUE thread, struct trace_identifiers *trace_identifiers_result);
@@ -300,7 +298,7 @@ static VALUE _native_initialize(DDTRACE_UNUSED VALUE _self, VALUE collector_inst
 // This method exists only to enable testing Datadog::Profiling::Collectors::CpuAndWallTime behavior using RSpec.
 // It SHOULD NOT be used for other purposes.
 static VALUE _native_sample(DDTRACE_UNUSED VALUE _self, VALUE collector_instance) {
-  cpu_and_wall_time_collector_sample(collector_instance);
+  cpu_and_wall_time_collector_sample(collector_instance, monotonic_wall_time_now_ns(RAISE_ON_FAILURE));
   return Qtrue;
 }
 
@@ -332,12 +330,11 @@ static VALUE _native_sample_after_gc(DDTRACE_UNUSED VALUE self, VALUE collector_
 // Assumption 3: This function IS NOT called from a signal handler. This function is not async-signal-safe.
 // Assumption 4: This function IS NOT called in a reentrant way.
 // Assumption 5: This function is called from the main Ractor (if Ruby has support for Ractors).
-VALUE cpu_and_wall_time_collector_sample(VALUE self_instance) {
+void cpu_and_wall_time_collector_sample(VALUE self_instance, long current_monotonic_wall_time_ns) {
   struct cpu_and_wall_time_collector_state *state;
   TypedData_Get_Struct(self_instance, struct cpu_and_wall_time_collector_state, &cpu_and_wall_time_collector_typed_data, state);
 
   VALUE threads = ddtrace_thread_list();
-  long current_wall_time_ns = wall_time_now_ns(RAISE_ON_FAILURE);
 
   const long thread_count = RARRAY_LEN(threads);
   for (long i = 0; i < thread_count; i++) {
@@ -354,7 +351,7 @@ VALUE cpu_and_wall_time_collector_sample(VALUE self_instance) {
     );
     long wall_time_elapsed_ns = update_time_since_previous_sample(
       &thread_context->wall_time_at_previous_sample_ns,
-      current_wall_time_ns,
+      current_monotonic_wall_time_ns,
       thread_context->gc_tracking.wall_time_at_start_ns,
       IS_WALL_TIME
     );
@@ -369,19 +366,16 @@ VALUE cpu_and_wall_time_collector_sample(VALUE self_instance) {
       state,
       thread,
       thread_context,
-      (ddog_Slice_i64) {.ptr = metric_values, .len = ENABLED_VALUE_TYPES_COUNT},
+      (ddog_Slice_I64) {.ptr = metric_values, .len = ENABLED_VALUE_TYPES_COUNT},
       SAMPLE_REGULAR
     );
   }
 
   state->sample_count++;
 
-  // TODO: This seems somewhat overkill and inefficient to do often; right now we just doing every few samples
+  // TODO: This seems somewhat overkill and inefficient to do often; right now we just do it every few samples
   // but there's probably a better way to do this if we actually track when threads finish
   if (state->sample_count % 100 == 0) remove_context_for_dead_threads(state);
-
-  // Return a VALUE to make it easier to call this function from Ruby APIs that expect a return value (such as rb_rescue2)
-  return Qnil;
 }
 
 // This function gets called when Ruby is about to start running the Garbage Collector on the current thread.
@@ -425,7 +419,7 @@ void cpu_and_wall_time_collector_on_gc_start(VALUE self_instance) {
     thread_context->gc_tracking.wall_time_at_finish_ns != INVALID_TIME) return;
 
   // Here we record the wall-time first and in on_gc_finish we record it second to avoid having wall-time be slightly < cpu-time
-  thread_context->gc_tracking.wall_time_at_start_ns = wall_time_now_ns(DO_NOT_RAISE_ON_FAILURE);
+  thread_context->gc_tracking.wall_time_at_start_ns = monotonic_wall_time_now_ns(DO_NOT_RAISE_ON_FAILURE);
   thread_context->gc_tracking.cpu_time_at_start_ns = cpu_time_now_ns(thread_context);
 }
 
@@ -461,7 +455,7 @@ void cpu_and_wall_time_collector_on_gc_finish(VALUE self_instance) {
 
   // Here we record the wall-time second and in on_gc_start we record it first to avoid having wall-time be slightly < cpu-time
   thread_context->gc_tracking.cpu_time_at_finish_ns = cpu_time_now_ns(thread_context);
-  thread_context->gc_tracking.wall_time_at_finish_ns = wall_time_now_ns(DO_NOT_RAISE_ON_FAILURE);
+  thread_context->gc_tracking.wall_time_at_finish_ns = monotonic_wall_time_now_ns(DO_NOT_RAISE_ON_FAILURE);
 }
 
 // This function gets called shortly after Ruby has finished running the Garbage Collector.
@@ -526,7 +520,7 @@ VALUE cpu_and_wall_time_collector_sample_after_gc(VALUE self_instance) {
       state,
       thread,
       thread_context,
-      (ddog_Slice_i64) {.ptr = metric_values, .len = ENABLED_VALUE_TYPES_COUNT},
+      (ddog_Slice_I64) {.ptr = metric_values, .len = ENABLED_VALUE_TYPES_COUNT},
       SAMPLE_IN_GC
     );
 
@@ -555,24 +549,24 @@ static void trigger_sample_for_thread(
   struct cpu_and_wall_time_collector_state *state,
   VALUE thread,
   struct per_thread_context *thread_context,
-  ddog_Slice_i64 metric_values_slice,
+  ddog_Slice_I64 metric_values_slice,
   sample_type type
 ) {
   int max_label_count =
     1 + // thread id
     1 + // thread name
     2;  // local root span id and span id
-  ddog_Label labels[max_label_count];
+  ddog_prof_Label labels[max_label_count];
   int label_pos = 0;
 
-  labels[label_pos++] = (ddog_Label) {
+  labels[label_pos++] = (ddog_prof_Label) {
     .key = DDOG_CHARSLICE_C("thread id"),
     .str = thread_context->thread_id_char_slice
   };
 
   VALUE thread_name = thread_name_for(thread);
   if (thread_name != Qnil) {
-    labels[label_pos++] = (ddog_Label) {
+    labels[label_pos++] = (ddog_prof_Label) {
       .key = DDOG_CHARSLICE_C("thread name"),
       .str = char_slice_from_ruby_string(thread_name)
     };
@@ -582,8 +576,8 @@ static void trigger_sample_for_thread(
   trace_identifiers_for(state, thread, &trace_identifiers_result);
 
   if (trace_identifiers_result.valid) {
-    labels[label_pos++] = (ddog_Label) {.key = DDOG_CHARSLICE_C("local root span id"), .str = trace_identifiers_result.local_root_span_id};
-    labels[label_pos++] = (ddog_Label) {.key = DDOG_CHARSLICE_C("span id"), .str = trace_identifiers_result.span_id};
+    labels[label_pos++] = (ddog_prof_Label) {.key = DDOG_CHARSLICE_C("local root span id"), .str = trace_identifiers_result.local_root_span_id};
+    labels[label_pos++] = (ddog_prof_Label) {.key = DDOG_CHARSLICE_C("span id"), .num = trace_identifiers_result.span_id};
 
     if (trace_identifiers_result.trace_endpoint != Qnil) {
       // The endpoint gets recorded in a different way because it is mutable in the tracer and can change during a
@@ -616,7 +610,7 @@ static void trigger_sample_for_thread(
     state->sampling_buffer,
     state->recorder_instance,
     metric_values_slice,
-    (ddog_Slice_label) {.ptr = labels, .len = label_pos},
+    (ddog_prof_Slice_Label) {.ptr = labels, .len = label_pos},
     type
   );
 }
@@ -693,8 +687,6 @@ static VALUE per_thread_context_st_table_as_ruby_hash(struct cpu_and_wall_time_c
   return result;
 }
 
-#define VALUE_COUNT(array) (sizeof(array) / sizeof(VALUE))
-
 static int per_thread_context_as_ruby_hash(st_data_t key_thread, st_data_t value_context, st_data_t result_hash) {
   VALUE thread = (VALUE) key_thread;
   struct per_thread_context *thread_context = (struct per_thread_context*) value_context;
@@ -723,8 +715,8 @@ static VALUE stats_as_ruby_hash(struct cpu_and_wall_time_collector_state *state)
   // Update this when modifying state struct (stats inner struct)
   VALUE stats_as_hash = rb_hash_new();
   VALUE arguments[] = {
-    ID2SYM(rb_intern("gc_samples")),                               /* => */ INT2NUM(state->stats.gc_samples),
-    ID2SYM(rb_intern("gc_samples_missed_due_to_missing_context")), /* => */ INT2NUM(state->stats.gc_samples_missed_due_to_missing_context),
+    ID2SYM(rb_intern("gc_samples")),                               /* => */ UINT2NUM(state->stats.gc_samples),
+    ID2SYM(rb_intern("gc_samples_missed_due_to_missing_context")), /* => */ UINT2NUM(state->stats.gc_samples_missed_due_to_missing_context),
   };
   for (long unsigned int i = 0; i < VALUE_COUNT(arguments); i += 2) rb_hash_aset(stats_as_hash, arguments[i], arguments[i+1]);
   return stats_as_hash;
@@ -794,18 +786,6 @@ static long update_time_since_previous_sample(long *time_at_previous_sample_ns,
   return elapsed_time_ns;
 }
 
-// Safety: This function is assumed never to raise exceptions by callers when raise_on_failure == false
-static long wall_time_now_ns(bool raise_on_failure) {
-  struct timespec current_monotonic;
-
-  if (clock_gettime(CLOCK_MONOTONIC, &current_monotonic) != 0) {
-    if (raise_on_failure) rb_sys_fail("Failed to read CLOCK_MONOTONIC");
-    else return 0;
-  }
-
-  return current_monotonic.tv_nsec + (current_monotonic.tv_sec * 1000 * 1000 * 1000);
-}
-
 // Safety: This function is assumed never to raise exceptions by callers
 static long cpu_time_now_ns(struct per_thread_context *thread_context) {
   thread_cpu_time cpu_time = thread_cpu_time_for(thread_context->thread_cpu_time_id);
@@ -872,19 +852,13 @@ static void trace_identifiers_for(struct cpu_and_wall_time_collector_state *stat
   if (numeric_local_root_span_id == Qnil || numeric_span_id == Qnil) return;
 
   unsigned long long local_root_span_id = NUM2ULL(numeric_local_root_span_id);
-  unsigned long long span_id = NUM2ULL(numeric_span_id);
-
   snprintf(trace_identifiers_result->local_root_span_id_buffer, MAXIMUM_LENGTH_64_BIT_IDENTIFIER, "%llu", local_root_span_id);
-  snprintf(trace_identifiers_result->span_id_buffer, MAXIMUM_LENGTH_64_BIT_IDENTIFIER, "%llu", span_id);
 
   trace_identifiers_result->local_root_span_id = (ddog_CharSlice) {
     .ptr = trace_identifiers_result->local_root_span_id_buffer,
     .len = strlen(trace_identifiers_result->local_root_span_id_buffer)
   };
-  trace_identifiers_result->span_id = (ddog_CharSlice) {
-    .ptr = trace_identifiers_result->span_id_buffer,
-    .len = strlen(trace_identifiers_result->span_id_buffer)
-  };
+  trace_identifiers_result->span_id = NUM2ULL(numeric_span_id);
 
   trace_identifiers_result->valid = true;
 

data/ext/ddtrace_profiling_native_extension/collectors_cpu_and_wall_time.h

@@ -2,7 +2,7 @@
 
 #include <ruby.h>
 
-VALUE cpu_and_wall_time_collector_sample(VALUE self_instance);
+void cpu_and_wall_time_collector_sample(VALUE self_instance, long current_monotonic_wall_time_ns);
 VALUE cpu_and_wall_time_collector_sample_after_gc(VALUE self_instance);
 void cpu_and_wall_time_collector_on_gc_start(VALUE self_instance);
 void cpu_and_wall_time_collector_on_gc_finish(VALUE self_instance);