feat: Remove background_task and hide get_current_transaction.

background_task is seemingly unusued, so it has been removed.

Also add comments explaining get_current_transaction is
internal-only and will not be exposed via API. Remove most
public references to it.

CHANGELOG.rst

@@ -11,6 +11,13 @@ Change Log
 
 .. There should always be an "Unreleased" section for changes pending release.
 
+Unreleased
+----------
+Changed
+~~~~~~~
+* Remove unused ``background_task`` monitoring function.
+* Remove ``get_current_transaction`` from the public API.
+
 [6.1.0] - 2024-10-15
 ---------------------
 Changed

edx_django_utils/monitoring/README.rst

@@ -36,18 +36,14 @@ Feature support matrix for built-in telemetry backends:
      - ✅
      - ❌
      - ✅
-   * - Retrieve and manipulate spans (``get_current_transaction``, ``ignore_transaction``)
+   * - Manipulate spans (``ignore_transaction``)
      - ✅
      - ❌
      - ❌
    * - Record exceptions (``record_exception``)
      - ✅
      - ✅
      - ✅
-   * - Instrument non-web tasks (``background_task``)
-     - ✅
-     - ❌
-     - ❌
 
 Additional requirements for using these backends:
 

edx_django_utils/monitoring/__init__.py

@@ -18,10 +18,9 @@
     MonitoringMemoryMiddleware,
     MonitoringSupportMiddleware
 )
-from .internal.transactions import get_current_transaction, ignore_transaction
+from .internal.transactions import ignore_transaction
 from .internal.utils import (
     accumulate,
-    background_task,
     function_trace,
     increment,
     record_exception,

edx_django_utils/monitoring/internal/transactions.py

@@ -53,7 +53,9 @@ def name(self):
 
 def get_current_transaction():
     """
-    Returns the current transaction.
+    Returns the current transaction. This is only used internally and won't
+    be ported over to the backends framework, because transactions will be
+    very different based on the backend.
     """
     current_transaction = None
     if newrelic: