Skip to content

ref(boto3): Use rpc as the default span op - #8035

Open
pabloDeputter wants to merge 2 commits into
major/3.0from
pablo/boto3-default-rpc-op
Open

pabloDeputter wants to merge 2 commits into
major/3.0from
pablo/boto3-default-rpc-op

Conversation

@pabloDeputter

Copy link
Copy Markdown
Member

Description

boto3 spans now use sentry.op = rpc instead of http.client. Service extensions can (and will in the future) provide a more specific op with get_span_op(), e.g. db for DynamoDB.

(todo: add to migration guide?)

Why

  • boto3 span represents an AWS api call, not an HTTP request; the HTTP request is already captured as the as the http.client child
  • JS already also defaults AWS SDK spans to rpc

It also lets these spans use new name and description rules in sentry-conventions, e.g. (getsentry/sentry-conventions#714, #717), e.g. Lambda.Invoke billing instead of POST https://lambda.…

@pabloDeputter
pabloDeputter marked this pull request as ready for review October 14, 2026 16:50
@pabloDeputter
pabloDeputter requested a review from a team as a code owner October 14, 2026 16:50
@github-actions

Copy link
Copy Markdown
Contributor

Coverage Results 📊

✅ 56199 passed | ⏭️ 2872 skipped | Total: 59071 | Pass Rate: 95.14% | Execution Time: 152m 41s

📊 Comparison with Base Branch

Metric Change
Total Tests 📉 -81871
Passed Tests 📉 -77467
Failed Tests —
Skipped Tests 📉 -4404

All tests are passing successfully.

✅ Patch coverage is 100.00% (2 of 2 changed executable lines covered; target 80%).
Project statement coverage is 90.04% (down 0.22 percentage points from base (d22300b) to head (72498a1)).

Changed files with executable lines (2)
File Patch coverage Changed executable lines
sentry_sdk/consts.py 100.00% 1/1 covered
sentry_sdk/integrations/boto3/_instrumentation.py 100.00% 1/1 covered
Coverage diff
@@            Coverage Diff             @@
##        master     #8035       +/-##
==========================================
- Coverage    90.26%    90.04%    -0.22%
==========================================
  Files          202       193        -9
  Tracked lines     26660     20939     -5721
  Branches      9926      7208     -2718
==========================================
- Hits         24063     18853     -5210
+ Misses        2597      2086      -511
+ Partials      1510      1234      -276

Generated by Coverage Action

@pabloDeputter
pabloDeputter added this pull request to stack #8038 October 14, 2026 20:14
@pabloDeputter
pabloDeputter removed this pull request from stack #8038 October 14, 2026 20:15

This branch has not been deployed

No deployments
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