newrelic_rpm 9.23.0 → 9.24.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c3bc35eb6cc82a0dd1ba947b3270d18b515da940533c11362ff531cfbeb05575
-  data.tar.gz: afd55578e5580b8bfb50280259c007c8c6e039a3d2029d9b8be2a2b2b4ea289c
+  metadata.gz: 2a513be9b409b6ae8c4e08416ded683ef6abeb372607baf25ba7aa10f7a916f5
+  data.tar.gz: 3604c7f4df3c33a1f61cd2b3b984dce971bac3b3b028862929a0c99c79931f4c
 SHA512:
-  metadata.gz: 75bd39b66cf822e975b2e704360eea2bc3ac44fbd2f63ed38b069a7115ae1a502a29c149adca19f3d057fe1332f3e5122afcdec4f42aec9182195d432db1f206
-  data.tar.gz: 44a1abc2d6f0ce8c28ac20e0912b21a1081baa474e455c561c3f4ace6ade983d2fd1922174ef3922fc51d99f7a771b0d13756fc10f14c8a4f420b5cb84ec4a20
+  metadata.gz: 5ecb0cd5c7b6476b008ac82be5bbda20b2c4e8b58465f457057a9357ac499f82b3779b9bbbcdaa2e5c2376db98e000957f653875bb9e98436f41b1a4f718dc42
+  data.tar.gz: 72e77abe6f9b63be2ae7af1bd99dcb285ec82d54725826e38c5c54df391c13bc983b7d50fe4b9e0795ef6b28f02c732d196db1ebe40f7e590d031f3d8394c3a6

data/CHANGELOG.md

@@ -1,5 +1,23 @@
 # New Relic Ruby Agent Release Notes
 
+## v9.24.0
+
+- **Feature: Deprecation reminder for SqlSampler#notice_sql API**
+
+  The `NewRelic::Agent::SqlSampler#notice_sql` method is deprecated and will be removed in a future major version. Instead, users should call `NewRelic::Agent::Datastores.notice_sql`. [PR#3345](https://github.com/newrelic/newrelic-ruby-agent/pull/3345)
+
+- **Feature: Deprecation notice for second and third arguments in Datastores.notice_sql API**
+
+  The second (`scoped_metric`) and third (`elapsed`) arguments in the `NewRelic::Agent::Datastores.notice_sql` method are deprecated. They have not been used by the method for some time. Instead, these values will be set based on the current segment when the API is called. [PR#3345](https://github.com/newrelic/newrelic-ruby-agent/pull/3345)
+
+- **Feature: Deprecation notice for second argument in Datastores.notice_statement API**
+
+  The second (`elapsed`) argument in the `NewRelic::Agent::Datastores.notice_statement` method is deprecated. It has not been used by the method for some time. Instead, this value will be set based on the current segment when the API is called. [PR#3346](https://github.com/newrelic/newrelic-ruby-agent/pull/3346)
+
+- **Feature: Deprecation notice for proc's second and third arguments in Datastores.wrap API**
+
+  The `NewRelic::Agent::Datastores.wrap` method is changing. In a future major version, proc will only accept a single argument, the result of the yield. The scoped metric name and elapsed arguments will be removed, as they are being removed from the `Datastores.notice_sql` method. The scoped metric name and elapsed values are derived from the current segment when the wrap yields. [PR#3346](https://github.com/newrelic/newrelic-ruby-agent/pull/3346)
+
 ## v9.23.0
 
 - **Feature: Add sidekiq.ignore_retry_errors configuration option**

data/lib/new_relic/agent/configuration/default_source.rb

@@ -4,6 +4,7 @@
 
 require 'forwardable'
 require_relative '../../constants'
+require_relative 'sampler_config_validator'
 
 module NewRelic
   module Agent
@@ -281,16 +282,6 @@ module NewRelic
           constants.compact!
           constants
         end
-
-        def self.enforce_fallback(allowed_values: nil, fallback: nil)
-          proc do |configured_value|
-            if allowed_values.any? { |v| v =~ /#{configured_value}/i }
-              configured_value
-            else
-              fallback
-            end
-          end
-        end
       end
 
       AUTOSTART_DENYLISTED_RAKE_TASKS = [
@@ -1463,19 +1454,68 @@ module NewRelic
           :allowed_from_server => true,
           :description => 'Distributed tracing lets you see the path that a request takes through your distributed system. Enabling distributed tracing changes the behavior of some New Relic features, so carefully consult the [transition guide](/docs/transition-guide-distributed-tracing) before you enable this feature.'
         },
+        :'distributed_tracing.sampler.root' => {
+          :default => 'default',
+          :public => false,
+          :type => String,
+          :allowed_from_server => false,
+          :allowlist => %w[default adaptive always_on always_off trace_id_ratio_based],
+          :transform => SamplerConfigValidator.validate_sampler_strategy_with_ratio(
+            :'distributed_tracing.sampler.root',
+            :'distributed_tracing.sampler.root.trace_id_ratio_based.ratio'
+          ),
+          :description => 'This setting controls the behavior of transaction sampling for transactions without a remote parent, traces that originate within this instance of the New Relic agent. Available values are `default`, `adaptive`, `always_on`, `always_off`, and `trace_id_ratio_based`. At this time `default` and `adaptive` are the same.'
+        },
         :'distributed_tracing.sampler.remote_parent_sampled' => {
           :default => 'default',
           :public => true,
           :type => String,
-          :allowed_from_server => true,
-          :description => 'This setting controls the behavior of transaction sampling when a remote parent is sampled and the trace flag is set in the traceparent. Available values are `default`, `always_on`, and `always_off`.'
+          :allowed_from_server => false,
+          :allowlist => %w[default adaptive always_on always_off trace_id_ratio_based],
+          :transform => SamplerConfigValidator.validate_sampler_strategy_with_ratio(
+            :'distributed_tracing.sampler.remote_parent_sampled',
+            :'distributed_tracing.sampler.remote_parent_sampled.trace_id_ratio_based.ratio'
+          ),
+          :description => 'This setting controls the behavior of transaction sampling when a remote parent is sampled. Available values are `default`, `always_on`, and `always_off`.'
         },
         :'distributed_tracing.sampler.remote_parent_not_sampled' => {
           :default => 'default',
           :public => true,
           :type => String,
-          :allowed_from_server => true,
-          :description => 'This setting controls the behavior of transaction sampling when a remote parent is not sampled and the trace flag is not set in the traceparent. Available values are `default`, `always_on`, and `always_off`.'
+          :allowed_from_server => false,
+          :allowlist => %w[default adaptive always_on always_off trace_id_ratio_based],
+          :transform => SamplerConfigValidator.validate_sampler_strategy_with_ratio(
+            :'distributed_tracing.sampler.remote_parent_not_sampled',
+            :'distributed_tracing.sampler.remote_parent_not_sampled.trace_id_ratio_based.ratio'
+          ),
+          :description => 'This setting controls the behavior of transaction sampling when a remote parent is not sampled. Available values are `default`, `always_on`, and `always_off`.'
+        },
+        :'distributed_tracing.sampler.root.trace_id_ratio_based.ratio' => {
+          :default => nil,
+          :public => false,
+          :type => Float,
+          :allow_nil => true,
+          :allowed_from_server => false,
+          :transform => SamplerConfigValidator.method(:validate_sampling_ratio),
+          :description => 'The ratio used for the trace_id_ratio_based sampling decision for the root sampler. This must be a float between 0.0 and 1.0. If you provide an invalid value, the sampler will not use the trace_id_ratio_based sampler and will return to the default behavior. If you do not provide a value, the sampler will not use the trace_id_ratio_based_sampler and fall back to the default sampler.'
+        },
+        :'distributed_tracing.sampler.remote_parent_sampled.trace_id_ratio_based.ratio' => {
+          :default => nil,
+          :public => false,
+          :type => Float,
+          :allow_nil => true,
+          :allowed_from_server => false,
+          :transform => SamplerConfigValidator.method(:validate_sampling_ratio),
+          :description => 'The ratio used for the trace_id_ratio_based sampling decision for the remote parent sampled sampler. This must be a float between 0.0 and 1.0. If you provide an invalid value, the sampler will not use the trace_id_ratio_based sampler and will return to the default behavior. If you do not provide a value, the sampler will not use the trace_id_ratio_based_sampler and fall back to the default sampler.'
+        },
+        :'distributed_tracing.sampler.remote_parent_not_sampled.trace_id_ratio_based.ratio' => {
+          :default => nil,
+          :public => false,
+          :type => Float,
+          :allow_nil => true,
+          :allowed_from_server => false,
+          :transform => SamplerConfigValidator.method(:validate_sampling_ratio),
+          :description => 'The ratio used for the trace_id_ratio_based sampling decision for the remote parent not sampled sampler. This must be a float between 0.0 and 1.0. If you provide an invalid value or do not provide a value, the sampler will not use the trace_id_ratio_based_sampler and fall back to the default sampler.'
         },
         # Elasticsearch
         :'elasticsearch.capture_cluster_name' => {
@@ -2516,8 +2556,14 @@ module NewRelic
           :allowed_from_server => true,
           :description => 'Number of seconds betwixt connections to the New Relic span event collection services.'
         },
-        # TODO: Sync with the other agents to see what the config should be named, how it should be enabled, how it should be described
         :'opentelemetry.enabled' => {
+          :default => false,
+          :public => false,
+          :type => Boolean,
+          :allowed_from_server => false,
+          :description => 'A global configuration option for disabling all OpenTelemetry signals sent through New Relic. If false, no OpenTelemetry signals will be sent to New Relic. If true, the signal-specific enabled config option (e.g. opentelemetry.traces.enabled) determines whether telemetry of that signal type will be reported to New Relic.'
+        },
+        :'opentelemetry.traces.enabled' => {
           :default => false,
           :public => false,
           :type => Boolean,

data/lib/new_relic/agent/configuration/sampler_config_validator.rb

@@ -0,0 +1,54 @@
+# This file is distributed under New Relic's license terms.
+# See https://github.com/newrelic/newrelic-ruby-agent/blob/main/LICENSE for complete details.
+# frozen_string_literal: true
+
+module NewRelic
+  module Agent
+    module Configuration
+      # Handles validation for the `distributed_tracing.sampler.*` configs
+      # Focuses on validating the trace id ratio based ratios
+      module SamplerConfigValidator
+        @sampler_strategy_warnings = {}
+
+        class << self
+          def validate_sampling_ratio(ratio)
+            return nil if ratio.nil?
+            return nil unless valid_ratio?(ratio)
+
+            ratio
+          end
+
+          def validate_sampler_strategy_with_ratio(strategy_key, ratio_key)
+            proc do |strategy|
+              next strategy unless strategy == 'trace_id_ratio_based'
+
+              ratio = NewRelic::Agent.config[ratio_key]
+
+              next strategy if valid_ratio?(ratio)
+
+              unless @sampler_strategy_warnings[strategy_key]
+                NewRelic::Agent.logger.warn(
+                  "Invalid or missing ratio for #{ratio_key} (value: #{ratio.inspect}). " \
+                  "Falling back to 'default' for #{strategy_key}."
+                )
+
+                @sampler_strategy_warnings[strategy_key] = true
+              end
+
+              'default'
+            end
+          end
+
+          def valid_ratio?(ratio)
+            ratio.is_a?(Float) && (0.0..1.0).cover?(ratio)
+          end
+
+          # used for testing
+          def reset_warnings!
+            @sampler_strategy_warnings = {}
+          end
+        end
+      end
+    end
+  end
+end

data/lib/new_relic/agent/datastores.rb

@@ -85,9 +85,11 @@ module NewRelic
       #   statement-level metrics (i.e. table or model name)
       #
       # @param [Proc,#call] callback proc or other callable to invoke after
-      #   running the datastore block. Receives three arguments: result of the
-      #   yield, the most specific (scoped) metric name, and elapsed time of the
-      #   call. An example use is attaching SQL to Transaction Traces at the end
+      #   running the datastore block. Receives three arguments:
+      #     * result of the yield
+      #     * (optional, deprecated) the most specific (scoped) metric name
+      #     * (optional, deprecated) elapsed time of the call
+      #   An example use is attaching SQL to Transaction Traces at the end
       #   of a wrapped datastore call.
       #
       #     callback = Proc.new do |result, metrics, elapsed|
@@ -107,6 +109,11 @@ module NewRelic
       # @api public
       #
       def self.wrap(product, operation, collection = nil, callback = nil)
+        NewRelic::Agent.logger.warn('The NewRelic::Agent::Datastores.wrap method is changing. ' \
+          'In a future major version, proc will only accept a single argument, the result of the yield. ' \
+          'The scoped metric name and elapsed arguments will be removed, as they are being removed from the ' \
+          'Datastores.notice_sql method. The scoped metric name and elapsed values are derived from the ' \
+          'current segment when the wrap yields.')
         NewRelic::Agent.record_api_supportability_metric(:wrap)
 
         return yield unless operation
@@ -145,11 +152,12 @@ module NewRelic
       #   some dialects of SQL (or non-SQL queries) are not guaranteed to be
       #   properly obfuscated by these routines!
       #
-      # @param [String] scoped_metric The most specific metric relating to this
-      #   query. Typically the result of
+      # @param [String] scoped_metric (optional, deprecated) The most specific
+      #   metric relating to this query. Typically the result of
       #   NewRelic::Agent::Datastores::MetricHelper#metrics_for
       #
-      # @param [Float] elapsed the elapsed time during query execution
+      # @param [Float] elapsed (optional, deprecated)  the elapsed time during
+      #   query execution
       #
       # @note THERE ARE SECURITY CONCERNS WHEN CAPTURING QUERY TEXT!
       #   New Relic's Transaction Tracing and Slow SQL features will
@@ -159,7 +167,11 @@ module NewRelic
       #
       # @api public
       #
-      def self.notice_sql(query, scoped_metric, elapsed)
+      def self.notice_sql(query, scoped_metric = '', elapsed = 0.0)
+        NewRelic::Agent.logger.warn('The NewRelic::Agent::Datastores.notice_sql method is changing. ' \
+          'In a future major version, the scoped_metric and elapsed arguments will be removed. ' \
+          'The scoped_metric and elapsed values are now based on the current segment when the notice_sql method ' \
+          'was called.')
         NewRelic::Agent.record_api_supportability_metric(:notice_sql)
 
         if (txn = Tracer.current_transaction) && (segment = txn.current_segment) && segment.respond_to?(:notice_sql)
@@ -180,7 +192,8 @@ module NewRelic
       #
       # @param [String] statement text of the statement to capture.
       #
-      # @param [Float] elapsed the elapsed time during query execution
+      # @param [Float] elapsed (optional, deprecated) the elapsed time during
+      #   query execution
       #
       # @note THERE ARE SECURITY CONCERNS WHEN CAPTURING STATEMENTS!
       #   This method will properly ignore statements when the user has turned
@@ -191,7 +204,12 @@ module NewRelic
       #
       # @api public
       #
-      def self.notice_statement(statement, elapsed)
+      def self.notice_statement(statement, elapsed = 0.0)
+        NewRelic::Agent.logger.warn('The ' \
+          'NewRelic::Agent::Datastores.notice_statement method is changing. ' \
+          'In a future major version, the elapsed argument will be removed. ' \
+          'The elapsed value is now based on the current segment when the ' \
+          'notice_statement method was called.')
         NewRelic::Agent.record_api_supportability_metric(:notice_statement)
 
         # Settings may change eventually, but for now we follow the same

data/lib/new_relic/agent/opentelemetry_bridge.rb

@@ -6,11 +6,14 @@ module NewRelic
   module Agent
     class OpenTelemetryBridge
       def initialize
-        # no-op without OpenTelemetry API & config
-        return unless defined?(OpenTelemetry) &&
-          NewRelic::Agent.config[:'opentelemetry.enabled']
-
-        OpenTelemetryBridge.install
+        # currently, we only have support for traces
+        # this method should change when we add support for metrics and logs.
+        if defined?(OpenTelemetry) && Agent.config[:'opentelemetry.enabled'] && Agent.config[:'opentelemetry.traces.enabled']
+          OpenTelemetryBridge.install
+          NewRelic::Agent.record_metric('Supportability/Tracing/Ruby/OpenTelemetryBridge/enabled', 0.0)
+        else
+          NewRelic::Agent.record_metric('Supportability/Tracing/Ruby/OpenTelemetryBridge/disabled', 0.0)
+        end
       end
 
       private

data/lib/new_relic/agent/sql_sampler.rb

@@ -140,6 +140,9 @@ module NewRelic
       # @deprecated Use {Datastores.notice_sql} instead.
       #
       def notice_sql(sql, metric_name, config, duration, state = nil, explainer = nil, binds = nil, name = nil) # THREAD_LOCAL_ACCESS sometimes
+        NewRelic::Agent.logger.warn(
+          'The SqlSampler#notice_sql method is deprecated. Please use Datastores.notice_sql instead.'
+        )
         state ||= Tracer.state
         data = state.sql_sampler_transaction_data
         return unless data

data/lib/new_relic/agent/transaction/distributed_tracing.rb

@@ -161,29 +161,21 @@ module NewRelic
           transaction.parent_span_id = payload.id
 
           unless payload.sampled.nil?
-            if payload.sampled == true
-              set_priority_and_sampled_newrelic_header(NewRelic::Agent.config[:'distributed_tracing.sampler.remote_parent_sampled'], payload)
-            elsif payload.sampled == false
-              set_priority_and_sampled_newrelic_header(NewRelic::Agent.config[:'distributed_tracing.sampler.remote_parent_not_sampled'], payload)
+            if payload.sampled
+              set_priority_and_sampled(
+                NewRelic::Agent.config[:'distributed_tracing.sampler.remote_parent_sampled'],
+                NewRelic::Agent.config[:'distributed_tracing.sampler.remote_parent_sampled.trace_id_ratio_based.ratio'],
+                payload
+              )
             else
-              transaction.sampled = payload.sampled
-              transaction.priority = payload.priority if payload.priority
+              set_priority_and_sampled(
+                NewRelic::Agent.config[:'distributed_tracing.sampler.remote_parent_not_sampled'],
+                NewRelic::Agent.config[:'distributed_tracing.sampler.remote_parent_not_sampled.trace_id_ratio_based.ratio'],
+                payload
+              )
             end
           end
         end
-
-        def set_priority_and_sampled_newrelic_header(config, payload)
-          if config == 'always_on'
-            transaction.sampled = true
-            transaction.priority = 2.0
-          elsif config == 'always_off'
-            transaction.sampled = false
-            transaction.priority = 0
-          else # case for 'default' value
-            transaction.sampled = payload.sampled
-            transaction.priority = payload.priority if payload.priority
-          end
-        end
       end
     end
   end

data/lib/new_relic/agent/transaction/trace_context.rb

@@ -154,9 +154,17 @@ module NewRelic
 
         def determine_sampling_decision(payload, trace_flags)
           if trace_flags == '01'
-            set_priority_and_sampled(NewRelic::Agent.config[:'distributed_tracing.sampler.remote_parent_sampled'], payload)
+            set_priority_and_sampled(
+              NewRelic::Agent.config[:'distributed_tracing.sampler.remote_parent_sampled'],
+              NewRelic::Agent.config[:'distributed_tracing.sampler.remote_parent_sampled.trace_id_ratio_based.ratio'],
+              payload
+            )
           elsif trace_flags == '00'
-            set_priority_and_sampled(NewRelic::Agent.config[:'distributed_tracing.sampler.remote_parent_not_sampled'], payload)
+            set_priority_and_sampled(
+              NewRelic::Agent.config[:'distributed_tracing.sampler.remote_parent_not_sampled'],
+              NewRelic::Agent.config[:'distributed_tracing.sampler.remote_parent_not_sampled.trace_id_ratio_based.ratio'],
+              payload
+            )
           else
             use_nr_tracestate_sampled(payload)
           end
@@ -171,14 +179,28 @@ module NewRelic
           end
         end
 
-        def set_priority_and_sampled(config, payload)
-          if config == 'always_on'
+        def set_priority_and_sampled(sampler, ratio, payload)
+          # Ruby evaluates case statements top to bottom. Listing the statements
+          # in the order of most likely to match keeps things performant.
+          # If we put more than one string in a condition, it'll check to see if
+          # the case matches any one of those conditions before moving on to the
+          # next one.
+          #
+          # Even though adaptive behaves the same as default, I hypothesize it
+          # will be used so infrequently, it should just be listed last.
+          case sampler
+          when 'default'
+            use_nr_tracestate_sampled(payload)
+          when 'always_on'
             transaction.sampled = true
             transaction.priority = 2.0
-          elsif config == 'always_off'
+          when 'always_off'
             transaction.sampled = false
             transaction.priority = 0
-          else # default
+          when 'trace_id_ratio_based'
+            transaction.sampled = transaction.trace_ratio_sampled?(ratio)
+            transaction.priority = transaction.default_priority
+          when 'adaptive'
             use_nr_tracestate_sampled(payload)
           end
         end

data/lib/new_relic/agent/transaction.rb

@@ -294,11 +294,41 @@ module NewRelic
         return false unless Agent.config[:'distributed_tracing.enabled']
 
         if @sampled.nil?
-          @sampled = NewRelic::Agent.instance.adaptive_sampler.sampled?
+          # Ruby evaluates case statements top to bottom. Listing the statements
+          # in the order of most likely to match keeps things performant.
+          # If we put more than one string in a condition, it'll check to see if
+          # the case matches any one of those conditions before moving on to the
+          # next one.
+          #
+          # Even though adaptive behaves the same as default, I hypothesize it
+          # will be used so infrequently, it should just be listed last.
+          @sampled = case NewRelic::Agent.config[:'distributed_tracing.sampler.root']
+          when 'default'
+            NewRelic::Agent.instance.adaptive_sampler.sampled?
+          when 'always_on'
+            true
+          when 'always_off'
+            false
+          when 'trace_id_ratio_based'
+            trace_ratio_sampled?(NewRelic::Agent.config[:'distributed_tracing.sampler.root.trace_id_ratio_based.ratio'])
+          when 'adaptive'
+            NewRelic::Agent.instance.adaptive_sampler.sampled?
+          end
         end
         @sampled
       end
 
+      def trace_ratio_sampled?(ratio)
+        # In the opentelemetry-sdk TraceIdRatioBased sampler, the algorithm
+        # looks like this:
+        # ratio == 1.0 || trace_id[8, 8].unpack1('Q>') < (ratio * (2**64 - 1)).ceil
+        # OTel stores their trace ids as binary strings
+        # ex. Array(trace_id).pack('H*')
+        # Since we don't store our trace ids as binary strings, this way is
+        # faster, but still gets the same result.
+        ratio == 1.0 || Integer(trace_id[16, 16], 16) < (ratio * (2**64 - 1)).ceil
+      end
+
       def trace_id
         @trace_id ||= NewRelic::Agent::GuidGenerator.generate_guid(32)
       end
@@ -307,8 +337,19 @@ module NewRelic
         @trace_id = value
       end
 
+      def default_priority
+        (sampled? ? rand + 1.0 : rand).round(NewRelic::PRIORITY_PRECISION)
+      end
+
       def priority
-        @priority ||= (sampled? ? rand + 1.0 : rand).round(NewRelic::PRIORITY_PRECISION)
+        @priority ||= case NewRelic::Agent.config[:'distributed_tracing.sampler.root']
+        when 'default', 'trace_id_ratio_based', 'adaptive'
+          default_priority
+        when 'always_on'
+          2.0
+        when 'always_off'
+          0
+        end
       end
 
       def referer

data/lib/new_relic/version.rb

@@ -6,7 +6,7 @@
 module NewRelic
   module VERSION # :nodoc:
     MAJOR = 9
-    MINOR = 23
+    MINOR = 24
     TINY = 0
 
     STRING = "#{MAJOR}.#{MINOR}.#{TINY}"

data/newrelic.yml

@@ -361,13 +361,12 @@ common: &default_settings
   # distributed_tracing.enabled: true
 
   # This setting controls the behavior of transaction sampling when a remote
-  # parent is not sampled and the trace flag is not set in the traceparent.
-  # Available values are default, always_on, and always_off.
+  # parent is not sampled. Available values are default, always_on, and
+  # always_off.
   # distributed_tracing.sampler.remote_parent_not_sampled: default
 
   # This setting controls the behavior of transaction sampling when a remote
-  # parent is sampled and the trace flag is set in the traceparent. Available
-  # values are default, always_on, and always_off.
+  # parent is sampled. Available values are default, always_on, and always_off.
   # distributed_tracing.sampler.remote_parent_sampled: default
 
   # If true, the agent captures the Elasticsearch cluster name in transaction

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: newrelic_rpm
 version: !ruby/object:Gem::Version
-  version: 9.23.0
+  version: 9.24.0
 platform: ruby
 authors:
 - Tanna McClure
@@ -301,6 +301,7 @@ files:
 - lib/new_relic/agent/configuration/manager.rb
 - lib/new_relic/agent/configuration/manual_source.rb
 - lib/new_relic/agent/configuration/mask_defaults.rb
+- lib/new_relic/agent/configuration/sampler_config_validator.rb
 - lib/new_relic/agent/configuration/security_policy_source.rb
 - lib/new_relic/agent/configuration/server_source.rb
 - lib/new_relic/agent/configuration/yaml_source.rb