chore(tracing): remove deprecated properties and parameters (#12199)

- Introduces the following breaking changes to tracing components: - Removes deprecated parameters from ``Tracer.configure(...)`` method and removes the ``Tracer.sampler`` attribute. - Drops support for multiple tracer instances, ``ddtrace.trace.Tracer`` can not be reinitialized. - Removes the deprecated ``Span.sampled`` property - Drops support for configuring sampling rules using functions and regex in the ``ddtrace.tracer.sampler.rules[].choose_matcher(...)`` method and removes the ``timestamp_ns`` parameter from ``ddtrace.internal.rate_limiter.RateLimiter.is_allowed()``. - Drops support for configuring ``DD_TRACE_METHODS`` with the '[]' notation. Ensure DD_TRACE_METHODS use the ':' notation instead". - Removes the deprecated ``ddtracer`` parameter from ``ddtrace.opentracer.tracer.Tracer()``. ## Checklist - [x] PR author has checked that all the criteria below are met - The PR description includes an overview of the change - The PR description articulates the motivation for the change - The change includes tests OR the PR description describes a testing strategy - The PR description notes risks associated with the change, if any - Newly-added code is easy to change - The change follows the [library release note guidelines](https://ddtrace.readthedocs.io/en/stable/releasenotes.html) - The change includes or references documentation updates if necessary - Backport labels are set (if [applicable](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting)) ## Reviewer Checklist - [x] Reviewer has checked that all the criteria below are met - Title is accurate - All changes are related to the pull request's stated goal - Avoids breaking [API](https://ddtrace.readthedocs.io/en/stable/versioning.html#interfaces) changes - Testing strategy adequately addresses listed risks - Newly-added code is easy to change - Release note makes sense to a user of the library - If necessary, author has acknowledged and discussed the performance implications of this PR as reported in the benchmarks PR comment - Backport labels are set in a manner that is consistent with the [release branch maintenance policy](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting)
DataDog · Feb 4, 2025 · 1245846 · 1245846
1 parent 3ac7025
commit 1245846
Show file tree

Hide file tree

Showing 14 changed files with 53 additions and 463 deletions.
diff --git a/benchmarks/bm/utils.py b/benchmarks/bm/utils.py
@@ -65,7 +65,7 @@ def process_trace(self, trace):
 
 
 def drop_traces(tracer):
-    tracer.configure(settings={"FILTERS": [_DropTraces()]})
+    tracer.configure(trace_processors=[_DropTraces()])
 
 
 def drop_telemetry_events():

diff --git a/benchmarks/rate_limiter/scenario.py b/benchmarks/rate_limiter/scenario.py
@@ -23,8 +23,8 @@ def _(loops):
             windows = [start + (i * self.time_window) for i in range(self.num_windows)]
             per_window = math.floor(loops / self.num_windows)
 
-            for window in windows:
+            for _ in windows:
                 for _ in range(per_window):
-                    rate_limiter.is_allowed(window)
+                    rate_limiter.is_allowed()
 
         yield _
diff --git a/ddtrace/_trace/sampling_rule.py b/ddtrace/_trace/sampling_rule.py
@@ -8,8 +8,6 @@
 from ddtrace.internal.glob_matching import GlobMatcher
 from ddtrace.internal.logger import get_logger
 from ddtrace.internal.utils.cache import cachedmethod
-from ddtrace.internal.utils.deprecations import DDTraceDeprecationWarning
-from ddtrace.vendor.debtcollector import deprecate
 
 
 if TYPE_CHECKING:  # pragma: no cover
@@ -210,14 +208,12 @@ def choose_matcher(self, prop):
         # We currently support the ability to pass in a function, a regular expression, or a string
         # If a string is passed in we create a GlobMatcher to handle the matching
         if callable(prop) or isinstance(prop, pattern_type):
-            # deprecated: passing a function or a regular expression'
-            deprecate(
-                "Using methods or regular expressions for SamplingRule matching is deprecated. ",
-                message="Please move to passing in a string for Glob matching.",
-                removal_version="3.0.0",
-                category=DDTraceDeprecationWarning,
+            log.error(
+                "Using methods or regular expressions for SamplingRule matching is not supported: %s ."
+                "Please move to passing in a string for Glob matching.",
+                str(prop),
             )
-            return prop
+            return "None"
         # Name and Resource will never be None, but service can be, since we str()
         #  whatever we pass into the GlobMatcher, we can just use its matching
         elif prop is None:

diff --git a/ddtrace/_trace/span.py b/ddtrace/_trace/span.py
@@ -52,8 +52,6 @@
 from ddtrace.internal.logger import get_logger
 from ddtrace.internal.sampling import SamplingMechanism
 from ddtrace.internal.sampling import set_sampling_decision_maker
-from ddtrace.internal.utils.deprecations import DDTraceDeprecationWarning
-from ddtrace.vendor.debtcollector import deprecate
 
 
 _NUMERIC_TAGS = (_ANALYTICS_SAMPLE_RATE_KEY,)
@@ -279,29 +277,6 @@ def duration(self) -> Optional[float]:
     def duration(self, value: float) -> None:
         self.duration_ns = int(value * 1e9)
 
-    @property
-    def sampled(self) -> Optional[bool]:
-        deprecate(
-            "span.sampled is deprecated and will be removed in a future version of the tracer.",
-            message="""span.sampled references the state of span.context.sampling_priority.
-            Please use span.context.sampling_priority instead to check if a span is sampled.""",
-            category=DDTraceDeprecationWarning,
-        )
-        if self.context.sampling_priority is None:
-            # this maintains original span.sampled behavior, where all spans would start
-            # with span.sampled = True until sampling runs
-            return True
-        return self.context.sampling_priority > 0
-
-    @sampled.setter
-    def sampled(self, value: bool) -> None:
-        deprecate(
-            "span.sampled is deprecated and will be removed in a future version of the tracer.",
-            message="""span.sampled has a no-op setter.
-            Please use span.set_tag('manual.keep'/'manual.drop') to keep or drop spans.""",
-            category=DDTraceDeprecationWarning,
-        )
-
     def finish(self, finish_time: Optional[float] = None) -> None:
         """Mark the end time of the span and submit it to the tracer.
         If the span has already been finished don't do anything.

diff --git a/ddtrace/_trace/tracer.py b/ddtrace/_trace/tracer.py
@@ -24,6 +24,7 @@
 from ddtrace._trace.processor import TraceProcessor
 from ddtrace._trace.processor import TraceSamplingProcessor
 from ddtrace._trace.processor import TraceTagsProcessor
+from ddtrace._trace.provider import BaseContextProvider
 from ddtrace._trace.provider import DefaultContextProvider
 from ddtrace._trace.sampler import BasePrioritySampler
 from ddtrace._trace.sampler import BaseSampler
@@ -200,7 +201,7 @@ def __init__(
         self,
         url: Optional[str] = None,
         dogstatsd_url: Optional[str] = None,
-        context_provider: Optional[DefaultContextProvider] = None,
+        context_provider: Optional[BaseContextProvider] = None,
     ) -> None:
         """
         Create a new ``Tracer`` instance. A global tracer is already initialized
@@ -328,28 +329,6 @@ def sample(self, span):
         else:
             log.error("No sampler available to sample span")
 
-    @property
-    def sampler(self):
-        deprecate(
-            "tracer.sampler is deprecated and will be removed.",
-            message="To manually sample call tracer.sample(span) instead.",
-            category=DDTraceDeprecationWarning,
-        )
-        return self._sampler
-
-    @sampler.setter
-    def sampler(self, value):
-        deprecate(
-            "Setting a custom sampler is deprecated and will be removed.",
-            message="""Please use DD_TRACE_SAMPLING_RULES to configure the sampler instead:
-    https://ddtrace.readthedocs.io/en/stable/configuration.html#DD_TRACE_SAMPLING_RULES""",
-            category=DDTraceDeprecationWarning,
-        )
-        if asm_config._apm_opt_out:
-            log.warning("Cannot set a custom sampler with Standalone ASM mode")
-            return
-        self._sampler = value
-
     def on_start_span(self, func: Callable) -> Callable:
         """Register a function to execute when a span start.
 
@@ -441,21 +420,7 @@ def get_log_correlation_context(self, active: Optional[Union[Context, Span]] = N
 
     def configure(
         self,
-        enabled: Optional[bool] = None,
-        hostname: Optional[str] = None,
-        port: Optional[int] = None,
-        uds_path: Optional[str] = None,
-        https: Optional[bool] = None,
-        sampler: Optional[BaseSampler] = None,
-        context_provider: Optional[DefaultContextProvider] = None,
-        wrap_executor: Optional[Callable] = None,
-        priority_sampling: Optional[bool] = None,
-        settings: Optional[Dict[str, Any]] = None,
-        dogstatsd_url: Optional[str] = None,
-        writer: Optional[TraceWriter] = None,
-        partial_flush_enabled: Optional[bool] = None,
-        partial_flush_min_spans: Optional[int] = None,
-        api_version: Optional[str] = None,
+        context_provider: Optional[BaseContextProvider] = None,
         compute_stats_enabled: Optional[bool] = None,
         appsec_enabled: Optional[bool] = None,
         iast_enabled: Optional[bool] = None,
@@ -472,58 +437,14 @@ def configure(
         :param bool appsec_standalone_enabled: When tracing is disabled ensures ASM support is still enabled.
         :param List[TraceProcessor] trace_processors: This parameter sets TraceProcessor (ex: TraceFilters).
            Trace processors are used to modify and filter traces based on certain criteria.
-
-        :param bool enabled: If True, finished traces will be submitted to the API, else they'll be dropped.
-            This parameter is deprecated and will be removed.
-        :param str hostname: Hostname running the Trace Agent. This parameter is deprecated and will be removed.
-        :param int port: Port of the Trace Agent. This parameter is deprecated and will be removed.
-        :param str uds_path: The Unix Domain Socket path of the agent. This parameter is deprecated and will be removed.
-        :param bool https: Whether to use HTTPS or HTTP. This parameter is deprecated and will be removed.
-        :param object sampler: A custom Sampler instance, locally deciding to totally drop the trace or not.
-            This parameter is deprecated and will be removed.
-        :param object wrap_executor: callable that is used when a function is decorated with
-            ``Tracer.wrap()``. This is an advanced option that usually doesn't need to be changed
-            from the default value. This parameter is deprecated and will be removed.
-        :param priority_sampling: This parameter is deprecated and will be removed in a future version.
-        :param bool settings: This parameter is deprecated and will be removed.
-        :param str dogstatsd_url: URL for UDP or Unix socket connection to DogStatsD
-            This parameter is deprecated and will be removed.
-        :param TraceWriter writer: This parameter is deprecated and will be removed.
-        :param bool partial_flush_enabled: This parameter is deprecated and will be removed.
-        :param bool partial_flush_min_spans: This parameter is deprecated and will be removed.
-        :param str api_version: This parameter is deprecated and will be removed.
-        :param bool compute_stats_enabled: This parameter is deprecated and will be removed.
         """
-        if settings is not None:
-            deprecate(
-                "Support for ``tracer.configure(...)`` with the settings parameter is deprecated",
-                message="Please use the trace_processors parameter instead of settings['FILTERS'].",
-                version="3.0.0",
-                category=DDTraceDeprecationWarning,
-            )
-            trace_processors = (trace_processors or []) + (settings.get("FILTERS") or [])
-
         return self._configure(
-            enabled,
-            hostname,
-            port,
-            uds_path,
-            https,
-            sampler,
-            context_provider,
-            wrap_executor,
-            priority_sampling,
-            trace_processors,
-            dogstatsd_url,
-            writer,
-            partial_flush_enabled,
-            partial_flush_min_spans,
-            api_version,
-            compute_stats_enabled,
-            appsec_enabled,
-            iast_enabled,
-            appsec_standalone_enabled,
-            True,
+            context_provider=context_provider,
+            trace_processors=trace_processors,
+            compute_stats_enabled=compute_stats_enabled,
+            appsec_enabled=appsec_enabled,
+            iast_enabled=iast_enabled,
+            appsec_standalone_enabled=appsec_standalone_enabled,
         )
 
     def _configure(
@@ -534,7 +455,7 @@ def _configure(
         uds_path: Optional[str] = None,
         https: Optional[bool] = None,
         sampler: Optional[BaseSampler] = None,
-        context_provider: Optional[DefaultContextProvider] = None,
+        context_provider: Optional[BaseContextProvider] = None,
         wrap_executor: Optional[Callable] = None,
         priority_sampling: Optional[bool] = None,
         trace_processors: Optional[List[TraceProcessor]] = None,
@@ -547,48 +468,18 @@ def _configure(
         appsec_enabled: Optional[bool] = None,
         iast_enabled: Optional[bool] = None,
         appsec_standalone_enabled: Optional[bool] = None,
-        log_deprecations: bool = False,
     ) -> None:
         if enabled is not None:
             self.enabled = enabled
-            if log_deprecations:
-                deprecate(
-                    "Enabling/Disabling tracing after application start is deprecated",
-                    message="Please use DD_TRACE_ENABLED instead.",
-                    version="3.0.0",
-                    category=DDTraceDeprecationWarning,
-                )
-
-        if priority_sampling is not None and log_deprecations:
-            deprecate(
-                "Disabling priority sampling is deprecated",
-                message="Calling `tracer.configure(priority_sampling=....) has no effect",
-                version="3.0.0",
-                category=DDTraceDeprecationWarning,
-            )
 
         if trace_processors is not None:
             self._user_trace_processors = trace_processors
 
         if partial_flush_enabled is not None:
             self._partial_flush_enabled = partial_flush_enabled
-            if log_deprecations:
-                deprecate(
-                    "Configuring partial flushing after application start is deprecated",
-                    message="Please use DD_TRACE_PARTIAL_FLUSH_ENABLED to enable/disable the partial flushing instead.",
-                    version="3.0.0",
-                    category=DDTraceDeprecationWarning,
-                )
 
         if partial_flush_min_spans is not None:
             self._partial_flush_min_spans = partial_flush_min_spans
-            if log_deprecations:
-                deprecate(
-                    "Configuring partial flushing after application start is deprecated",
-                    message="Please use DD_TRACE_PARTIAL_FLUSH_MIN_SPANS to set the flushing threshold instead.",
-                    version="3.0.0",
-                    category=DDTraceDeprecationWarning,
-                )
 
         if appsec_enabled is not None:
             asm_config._asm_enabled = appsec_enabled
@@ -620,33 +511,11 @@ def _configure(
         if sampler is not None:
             self._sampler = sampler
             self._user_sampler = self._sampler
-            if log_deprecations:
-                deprecate(
-                    "Configuring custom samplers is deprecated",
-                    message="Please use DD_TRACE_SAMPLING_RULES to configure the sample rates instead",
-                    category=DDTraceDeprecationWarning,
-                    removal_version="3.0.0",
-                )
 
         if dogstatsd_url is not None:
-            if log_deprecations:
-                deprecate(
-                    "Configuring dogstatsd_url after application start is deprecated",
-                    message="Please use DD_DOGSTATSD_URL instead.",
-                    version="3.0.0",
-                    category=DDTraceDeprecationWarning,
-                )
             self._dogstatsd_url = dogstatsd_url
 
         if any(x is not None for x in [hostname, port, uds_path, https]):
-            if log_deprecations:
-                deprecate(
-                    "Configuring tracer agent connection after application start is deprecated",
-                    message="Please use DD_TRACE_AGENT_URL instead.",
-                    version="3.0.0",
-                    category=DDTraceDeprecationWarning,
-                )
-
             # If any of the parts of the URL have updated, merge them with
             # the previous writer values.
             prev_url_parsed = compat.parse.urlparse(self._agent_url)
@@ -670,13 +539,6 @@ def _configure(
             new_url = None
 
         if compute_stats_enabled is not None:
-            if log_deprecations:
-                deprecate(
-                    "Configuring tracer stats computation after application start is deprecated",
-                    message="Please use DD_TRACE_STATS_COMPUTATION_ENABLED instead.",
-                    version="3.0.0",
-                    category=DDTraceDeprecationWarning,
-                )
             self._compute_stats = compute_stats_enabled
 
         try:
@@ -685,14 +547,6 @@ def _configure(
             # It's possible the writer never got started
             pass
 
-        if api_version is not None and log_deprecations:
-            deprecate(
-                "Configuring Tracer API version after application start is deprecated",
-                message="Please use DD_TRACE_API_VERSION instead.",
-                version="3.0.0",
-                category=DDTraceDeprecationWarning,
-            )
-
         if writer is not None:
             self._writer = writer
         elif any(x is not None for x in [new_url, api_version, sampler, dogstatsd_url, appsec_enabled]):
@@ -754,12 +608,6 @@ def _configure(
 
         if wrap_executor is not None:
             self._wrap_executor = wrap_executor
-            if log_deprecations:
-                deprecate(
-                    "Support for tracer.configure(...) with the wrap_executor parameter is deprecated",
-                    version="3.0.0",
-                    category=DDTraceDeprecationWarning,
-                )
 
         self._generate_diagnostic_logs()
 
@@ -1344,7 +1192,7 @@ def _handle_sampler_update(self, cfg: Config) -> None:
             and self._user_sampler
         ):
             # if we get empty configs from rc for both sample rate and rules, we should revert to the user sampler
-            self.sampler = self._user_sampler
+            self._sampler = self._user_sampler
             return
 
         if cfg._get_source("_trace_sample_rate") != "remote_config" and self._user_sampler: