fix compiler warnings by removing unused TracerSignature field

Author: zacharycmontoya

src/datadog/config_manager.cpp

@@ -124,7 +124,6 @@ ConfigManager::Update parse_dynamic_config(const nlohmann::json& j) {
 namespace rc = datadog::remote_config;
 
 ConfigManager::ConfigManager(const FinalizedTracerConfig& config,
-                             const TracerSignature& tracer_signature,
                              const std::shared_ptr<TracerTelemetry>& telemetry)
     : clock_(config.clock),
       default_metadata_(config.metadata),
@@ -133,7 +132,6 @@ ConfigManager::ConfigManager(const FinalizedTracerConfig& config,
       rules_(config.trace_sampler.rules),
       span_defaults_(std::make_shared<SpanDefaults>(config.defaults)),
       report_traces_(config.report_traces),
-      tracer_signature_(tracer_signature),
       telemetry_(telemetry) {}
 
 rc::Products ConfigManager::get_products() { return rc::product::APM_TRACING; }

src/datadog/config_manager.h

@@ -76,7 +76,6 @@ class ConfigManager : public remote_config::Listener {
   DynamicConfig<std::shared_ptr<const SpanDefaults>> span_defaults_;
   DynamicConfig<bool> report_traces_;
 
-  const TracerSignature& tracer_signature_;
   std::shared_ptr<TracerTelemetry> telemetry_;
 
  private:
@@ -86,7 +85,6 @@ class ConfigManager : public remote_config::Listener {
 
  public:
   ConfigManager(const FinalizedTracerConfig& config,
-                const TracerSignature& signature,
                 const std::shared_ptr<TracerTelemetry>& telemetry);
   ~ConfigManager() override{};
 

src/datadog/tracer.cpp

@@ -47,8 +47,8 @@ Tracer::Tracer(const FinalizedTracerConfig& config,
       tracer_telemetry_(std::make_shared<TracerTelemetry>(
           config.telemetry.enabled, config.clock, logger_, signature_,
           config.integration_name, config.integration_version)),
-      config_manager_(std::make_shared<ConfigManager>(config, signature_,
-                                                      tracer_telemetry_)),
+      config_manager_(
+          std::make_shared<ConfigManager>(config, tracer_telemetry_)),
       collector_(/* see constructor body */),
       span_sampler_(
           std::make_shared<SpanSampler>(config.span_sampler, config.clock)),

test/test_config_manager.cpp

@@ -29,8 +29,7 @@ CONFIG_MANAGER_TEST("remote configuration handling") {
   auto tracer_telemetry = std::make_shared<TracerTelemetry>(
       false, default_clock, nullptr, tracer_signature, "", "");
 
-  ConfigManager config_manager(*finalize_config(config), tracer_signature,
-                               tracer_telemetry);
+  ConfigManager config_manager(*finalize_config(config), tracer_telemetry);
 
   rc::Listener::Configuration config_update{/* id = */ "id",
                                             /* path = */ "",

test/test_datadog_agent.cpp

@@ -200,8 +200,7 @@ TEST_CASE("Remote Configuration", "[datadog_agent]") {
       finalized->telemetry.enabled, finalized->clock, finalized->logger,
       signature, "", "");
 
-  auto config_manager =
-      std::make_shared<ConfigManager>(*finalized, signature, telemetry);
+  auto config_manager = std::make_shared<ConfigManager>(*finalized, telemetry);
 
   const auto& agent_config =
       std::get<FinalizedDatadogAgentConfig>(finalized->collector);