Revert "feat(tracing): Add propagate_traces deprecation warning (#3899)"

This reverts commit 8a70b76.

Author: cmanallen

sentry_sdk/integrations/celery/__init__.py

@@ -1,6 +1,4 @@
 import sys
-import warnings
-
 from collections.abc import Mapping
 from functools import wraps
 
@@ -70,11 +68,6 @@ def __init__(
         exclude_beat_tasks=None,
     ):
         # type: (bool, bool, Optional[List[str]]) -> None
-        warnings.warn(
-            "The `propagate_traces` parameter is deprecated. Please use `trace_propagation_targets` instead.",
-            DeprecationWarning,
-            stacklevel=2,
-        )
         self.propagate_traces = propagate_traces
         self.monitor_beat_tasks = monitor_beat_tasks
         self.exclude_beat_tasks = exclude_beat_tasks

sentry_sdk/scope.py

@@ -625,11 +625,6 @@ def iter_trace_propagation_headers(self, *args, **kwargs):
         """
         client = self.get_client()
         if not client.options.get("propagate_traces"):
-            warnings.warn(
-                "The `propagate_traces` parameter is deprecated. Please use `trace_propagation_targets` instead.",
-                DeprecationWarning,
-                stacklevel=2,
-            )
             return
 
         span = kwargs.pop("span", None)

tests/integrations/celery/test_celery.py

@@ -268,9 +268,7 @@ def dummy_task():
 
 
 def test_simple_no_propagation(capture_events, init_celery):
-    with pytest.warns(DeprecationWarning):
-        celery = init_celery(propagate_traces=False)
-
+    celery = init_celery(propagate_traces=False)
     events = capture_events()
 
     @celery.task(name="dummy_task")
@@ -534,10 +532,9 @@ def test_sentry_propagate_traces_override(init_celery):
     Test if the `sentry-propagate-traces` header given to `apply_async`
     overrides the `propagate_traces` parameter in the integration constructor.
     """
-    with pytest.warns(DeprecationWarning):
-        celery = init_celery(
-            propagate_traces=True, traces_sample_rate=1.0, release="abcdef"
-        )
+    celery = init_celery(
+        propagate_traces=True, traces_sample_rate=1.0, release="abcdef"
+    )
 
     @celery.task(name="dummy_task", bind=True)
     def dummy_task(self, message):

tests/tracing/test_integration_tests.py

@@ -147,20 +147,6 @@ def test_continue_from_headers(
     assert message_payload["message"] == "hello"
 
 
-@pytest.mark.parametrize("sample_rate", [0.0, 1.0])
-def test_propagate_traces_deprecation_warning(sentry_init, sample_rate):
-    sentry_init(traces_sample_rate=sample_rate, propagate_traces=False)
-
-    with start_transaction(name="hi"):
-        with start_span() as old_span:
-            with pytest.warns(DeprecationWarning):
-                dict(
-                    sentry_sdk.get_current_scope().iter_trace_propagation_headers(
-                        old_span
-                    )
-                )
-
-
 @pytest.mark.parametrize("sample_rate", [0.5, 1.0])
 def test_dynamic_sampling_head_sdk_creates_dsc(
     sentry_init, capture_envelopes, sample_rate, monkeypatch