[NO-TICKET] Hide native code for memfd close (#4755)

Author: vpellan

lib/datadog/core/configuration/components.rb

@@ -210,7 +210,7 @@ def shutdown!(replacement = nil)
           telemetry.emit_closing! unless replacement&.telemetry&.enabled
           telemetry.shutdown!
 
-          Core::ProcessDiscovery._native_close_tracer_memfd(@process_discovery_fd, @logger) if @process_discovery_fd
+          @process_discovery_fd&.shutdown!
         end
 
         # Returns the current state of various components.

lib/datadog/core/process_discovery.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require 'datadog/core/process_discovery/tracer_memfd'
+
 module Datadog
   module Core
     # Class used to store tracer metadata in a native file descriptor.
@@ -10,7 +12,9 @@ def self.get_and_store_metadata(settings, logger)
           return
         end
         metadata = get_metadata(settings)
-        _native_store_tracer_metadata(logger, **metadata)
+        memfd = _native_store_tracer_metadata(logger, **metadata)
+        memfd.logger = logger if memfd
+        memfd
       end
 
       # According to the RFC, runtime_id, service_name, service_env, service_version are optional.

lib/datadog/core/process_discovery/tracer_memfd.rb

@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+
+module Datadog
+  module Core
+    class ProcessDiscovery
+      class TracerMemfd
+        attr_accessor :logger
+
+        def shutdown!
+          ProcessDiscovery._native_close_tracer_memfd(self, logger)
+        end
+      end
+    end
+  end
+end

sig/datadog/core/process_discovery/tracer_memfd.rbs

@@ -0,0 +1,11 @@
+module Datadog
+  module Core
+    class ProcessDiscovery
+      class TracerMemfd
+        attr_accessor logger: Datadog::Core::Logger
+
+        def shutdown!: () -> void
+      end
+    end
+  end
+end

spec/datadog/core/configuration/components_spec.rb

@@ -28,7 +28,6 @@
 RSpec.describe Datadog::Core::Configuration::Components do
   subject(:components) { described_class.new(settings) }
 
-  # Using a double does not give access to superclass methods like debug
   let(:logger) do
     instance_double(Datadog::Core::Logger).tap do |logger|
       allow(logger).to receive(:debug)