Skip to content

chore(asyncpg): Remove transaction-based tracing - #7192

Draft
sentrivana wants to merge 4 commits into
major/3.0from
ivana/major/remove-transactions-asyncpg
Draft

chore(asyncpg): Remove transaction-based tracing#7192
sentrivana wants to merge 4 commits into
major/3.0from
ivana/major/remove-transactions-asyncpg

Conversation

@sentrivana

@sentrivana sentrivana commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Use Hide whitespace ✨

Closes PY-2681
Closes #7077

@linear-code

linear-code Bot commented Aug 13, 2026

Copy link
Copy Markdown

PY-2681

@github-actions

github-actions Bot commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊

106577 passed | ❌ 1 failed | ⏭️ 5461 skipped | Total: 112039 | Pass Rate: 95.12% | Execution Time: 386m 18s

📊 Comparison with Base Branch

Metric Change
Total Tests 📉 -7183
Passed Tests 📉 -5886
Failed Tests 📈 +1
Skipped Tests 📉 -1298

➕ New Tests (1)

View new tests
  • test_span_streaming_quart_auth_user_id[42-True]
    • File: tests.integrations.quart.test_quart
    • Status: ❌ Failing

❌ Failed Tests

test_span_streaming_quart_auth_user_id[42-True]

File: tests.integrations.quart.test_quart
Suite: py3.13-quart-v0.21.0
Error: KeyError: 'user.id'

Stack Trace
tests/integrations/quart/test_quart.py:1039: in test_span_streaming_quart_auth_user_id
    assert segment["attributes"]["user.id"] == user_id
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
E   KeyError: 'user.id'

✅ Patch coverage is 84.62%. Project has 2262 uncovered lines.
✅ Project coverage is 90.53%. Comparing base (base) to head (head).

Files with missing lines (1)
File Patch % Lines
sentry_sdk/integrations/asyncpg.py 84.62% ⚠️ 4 Missing and 4 partials
Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    90.23%    90.53%     +0.3%
==========================================
  Files          193       185        -8
  Lines        25361     23889     -1472
  Branches      9302      8792      -510
==========================================
+ Hits         22883     21627     -1256
- Misses        2478      2262      -216
- Partials      1432      1373       -59

Generated by Codecov Action

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant