Deprecate enable_tracing in favor of traces_sample_rate

Author: sl0thentr0py

CHANGELOG.md

@@ -5,6 +5,10 @@
 - Fix handling of cron with tz in Cron::Job ([#2530](https://github.com/getsentry/sentry-ruby/pull/2530))
 - Revert "[rails] support string errors in error reporter (#2464)" ([#2533](https://github.com/getsentry/sentry-ruby/pull/2533))
 
+### Miscellaneous
+
+- Deprecate `enable_tracing` in favor of `traces_sample_rate = 1.0` [#2535](https://github.com/getsentry/sentry-ruby/pull/2535)
+
 ## 5.22.3
 
 ### Bug Fixes

sentry-rails/lib/generators/sentry_generator.rb

@@ -16,7 +16,7 @@ def copy_initializer_file
       Sentry.init do |config|
         config.breadcrumbs_logger = [:active_support_logger]
         config.dsn = #{dsn}
-        config.enable_tracing = true
+        config.traces_sample_rate = 1.0
       end
     RUBY
   end

sentry-rails/spec/sentry/generator_spec.rb

@@ -55,7 +55,7 @@
       Sentry.init do |config|
         config.breadcrumbs_logger = [:active_support_logger]
         config.dsn = ENV['SENTRY_DSN']
-        config.enable_tracing = true
+        config.traces_sample_rate = 1.0
       end
     RUBY
   end
@@ -87,7 +87,7 @@
         Sentry.init do |config|
           config.breadcrumbs_logger = [:active_support_logger]
           config.dsn = 'foobarbaz'
-          config.enable_tracing = true
+          config.traces_sample_rate = 1.0
         end
       RUBY
     end

sentry-ruby/lib/sentry/configuration.rb

@@ -264,6 +264,7 @@ def capture_exception_frame_locals=(value)
 
     # Easier way to use performance tracing
     # If set to true, will set traces_sample_rate to 1.0
+    # @deprecated It will be removed in the next major release.
     # @return [Boolean, nil]
     attr_reader :enable_tracing
 
@@ -485,6 +486,12 @@ def instrumenter=(instrumenter)
     end
 
     def enable_tracing=(enable_tracing)
+      unless enable_tracing.nil?
+        log_warn <<~MSG
+          `enable_tracing` is now deprecated in favor of `traces_sample_rate = 1.0`.
+        MSG
+      end
+
       @enable_tracing = enable_tracing
       @traces_sample_rate ||= 1.0 if enable_tracing
     end

sentry-ruby/spec/sentry/configuration_spec.rb

@@ -122,6 +122,17 @@
 
           expect(subject.tracing_enabled?).to eq(false)
         end
+
+        it "prints deprecation message when being assigned" do
+          string_io = StringIO.new
+          subject.logger = Logger.new(string_io)
+
+          subject.enable_tracing = true
+
+          expect(string_io.string).to include(
+            "WARN -- sentry: `enable_tracing` is now deprecated in favor of `traces_sample_rate = 1.0`."
+          )
+        end
       end
     end
 

sentry-ruby/spec/sentry/metrics/aggregator_spec.rb

@@ -11,7 +11,7 @@
   before do
     perform_basic_setup do |config|
       config.metrics.enabled = true
-      config.enable_tracing = true
+      config.traces_sample_rate = 1.0
       config.release = 'test-release'
       config.environment = 'test'
       config.logger = Logger.new(string_io)
@@ -194,7 +194,7 @@
       before do
         perform_basic_setup do |config|
           config.metrics.enabled = true
-          config.enable_tracing = true
+          config.traces_sample_rate = 1.0
           config.release = 'test-release'
           config.environment = 'test'
           config.logger = Logger.new(string_io)

sentry-ruby/spec/sentry/metrics_spec.rb

@@ -116,7 +116,7 @@
 
       before do
         perform_basic_setup do |config|
-          config.enable_tracing = true
+          config.traces_sample_rate = 1.0
           config.metrics.enabled = true
         end