Remove duplicate query logging that breaks Django's assertNumQueries

The SqlCommenter _QueryWrapper manually appends the SQL string to
queries_log when a CursorDebugWrapper is active. However,
CursorDebugWrapper.debug_sql already logs the query as a dict with
"sql" and "time" keys. This causes two problems:

1. Queries are double-counted in queries_log
2. The raw string entries cause a TypeError in Django's
   assertNumQueries, which expects dict entries with a "sql" key

Remove the manual append and let Django handle query logging.

Author: federicobond

CHANGELOG.md

@@ -24,6 +24,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Fixed
 
+- `opentelemetry-instrumentation-django`: Remove duplicate query logging in SQLCommenter middleware that broke Django's `assertNumQueries`
+  ([#4367](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/XXXX))
 - `opentelemetry-docker-tests`: Replace deprecated `SpanAttributes` from `opentelemetry.semconv.trace` with `opentelemetry.semconv._incubating.attributes`
  ([#4339](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/4339))
 - `opentelemetry-instrumentation-confluent-kafka`: Skip `recv` span creation when `poll()` returns no message or `consume()` returns an empty list, avoiding empty spans on idle polls

instrumentation/opentelemetry-instrumentation-django/src/opentelemetry/instrumentation/django/middleware/sqlcommenter_middleware.py

@@ -18,7 +18,6 @@
 # pylint: disable=no-name-in-module
 from django import conf, get_version
 from django.db import connections
-from django.db.backends.utils import CursorDebugWrapper
 
 from opentelemetry.instrumentation.sqlcommenter_utils import _add_sql_comment
 from opentelemetry.instrumentation.utils import _get_opentelemetry_values
@@ -122,8 +121,4 @@ def __call__(self, execute: Type[T], sql, params, many, context) -> T:
         #  * https://github.com/basecamp/marginalia/issues/61
         #  * https://github.com/basecamp/marginalia/pull/80
 
-        # Add the query to the query log if debugging.
-        if isinstance(context["cursor"], CursorDebugWrapper):
-            context["connection"].queries_log.append(sql)
-
         return execute(sql, params, many, context)