Skip to content

chore(langchain): Remove transaction-based tracing - #7410

Merged
alexander-alderman-webb merged 6 commits into
major/3.0from
webb/langchain/remove-transactions
Sep 7, 2026
Merged

alexander-alderman-webb merged 6 commits into
major/3.0from
webb/langchain/remove-transactions

Conversation

@alexander-alderman-webb

@alexander-alderman-webb alexander-alderman-webb commented Sep 7, 2026 •

Copy link
Copy Markdown
Collaborator

Description

Remove branches for the static trace lifecycle.
Also remove manual tracing API calls in tests, as the outer transaction is no longer needed.

Remove test_langchain_embeddings_multiple_calls() and test_langchain_embeddings_span_hierarchy() as the tests were coupled to the static lifecycle.

Issues

Closes #7097

Reminders

@alexander-alderman-webb
alexander-alderman-webb marked this pull request as ready for review September 7, 2026 11:35
@alexander-alderman-webb
alexander-alderman-webb requested a review from a team as a code owner September 7, 2026 11:35
@github-actions

github-actions Bot commented Sep 7, 2026 •

Copy link
Copy Markdown
Contributor

Codecov Results 📊

✅ 74670 passed | ⏭️ 3429 skipped | Total: 78099 | Pass Rate: 95.61% | Execution Time: 233m 50s

📊 Comparison with Base Branch

Metric Change
Total Tests 📉 -27302
Passed Tests 📉 -24252
Failed Tests —
Skipped Tests 📉 -3050

All tests are passing successfully.

✅ Patch coverage is 98.53%. Project has 2272 uncovered lines.
✅ Project coverage is 90.13%. Comparing base (7a758b9) to head (4b19cc2).

Files with missing lines (1)
File Patch % Lines
sentry_sdk/integrations/langchain.py 98.53% ⚠️ 1 Missing and 3 partials
Coverage diff
@@            Coverage Diff             @@
##        master       #PR       +/-##
==========================================
+ Coverage    89.92%    90.13%    +0.21%
==========================================
  Files          193       185        -8
  Lines        24908     23026     -1882
  Branches      8986      8414      -572
==========================================
+ Hits         22398     20754     -1644
- Misses        2510      2272      -238
- Partials      1430      1352       -78

Generated by Codecov Action

@alexander-alderman-webb
alexander-alderman-webb merged commit 7e83269 into major/3.0 Sep 7, 2026
147 checks passed
@alexander-alderman-webb
alexander-alderman-webb deleted the webb/langchain/remove-transactions branch September 7, 2026 12:29
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.

2 participants