Skip to content

feat(descriptions): Add descriptions for AWS SDK spans by service, operation and resource - #717

Merged
pabloDeputter merged 5 commits into
mainfrom
pablo/aws-sdk-span-descriptions
Oct 23, 2026
Merged

pabloDeputter merged 5 commits into
mainfrom
pablo/aws-sdk-span-descriptions

Conversation

@pabloDeputter

Copy link
Copy Markdown
Member

Description

Adds description templates so it follows the same format like the object.* descriptions in #707:
<Service>.<Operation> <resource>.

op before (name fallback) after
rpc none (Lambda.Invoke) Lambda.Invoke billing, SecretsManager.GetSecretValue db-password, SQS.ChangeMessageVisibility orders; S3.ListBuckets (operations with no resource)
db (DynamoDB, no db.query.text) none (GetItem orders) DynamoDB.GetItem orders, DynamoDB.Query orders.by-status, DynamoDB.BatchGetItem customers, orders, DynamoDB.ListTables
queue.* (SQS, SNS) orders SQS.SendMessageBatch orders, SNS.Publish alerts

Only AWS SDK spans have rpc.service and rpc.method, so other rpc, db and messaging spans are unaffected; db spans that contain the db.query.text attribute keep it as as their description.

@vercel

vercel Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
sentry-conventions Ready Ready Preview Oct 23, 2026 11:18am UTC

Request Review

@github-actions

github-actions Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

Semver Impact of This PR

🟡 Minor (new features)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


New Features ✨

  • (attributes) Add AWS Lambda, Step Functions and Secrets Manager name attributes by pabloDeputter in #716
  • (descriptions) Add descriptions for AWS SDK spans by service, operation and resource by pabloDeputter in #717
  • (names) Add span name and description rules for object.* ops by pabloDeputter in #707

Internal Changes 🔧

  • (attributes) Use JSON for aws.s3.delete & document multi-collection db.collection.name by pabloDeputter in #711

🤖 This preview updates automatically when you update the PR.

@cursor cursor Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Stale Bugbot comment from a previous run.

Comment thread model/description/rpc.json Outdated
Comment thread model/description/db.json

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit fea8924. Configure here.

Comment thread model/description/rpc.json Outdated
Base automatically changed from pablo/aws-resource-name-attributes to main October 23, 2026 11:14
pabloDeputter added a commit that referenced this pull request Oct 23, 2026
…name attributes (#716)

## Description
Adds the following attributes (non-OTel):
- `aws.lambda.target_function.name`: the function a Lambda API call acts on (`GetFunction`, `UpdateFunctionCode`, …)
- `aws.step_functions.state_machine.name`
- `aws.step_functions.activity.name`
- `aws.secretsmanager.secret.name`: from the `Name` Secrets Manager returns

## Why
Next PR (#717)  in stack adds descriptions for AWS SDK spans by service, operation and resource, e.g. `SecretsManager.GetSecretValue db-password`. Other services already have a short name for this (`aws.s3.bucket`, `aws.kinesis.stream_name`, `faas.invoked_name`). These only had the full ARN, e.g. `arn:aws:secretsmanager:us-east-1:123456789012:secret:db-password-AbCdEf`, which is too long and already in the respective ARN attribute.

All of these resource names are already part of the ARNs we collect, so no new data is sent.
@pabloDeputter
pabloDeputter enabled stacked auto-merge (squash) October 23, 2026 11:19
@pabloDeputter
pabloDeputter merged commit 16432b6 into main Oct 23, 2026
18 checks passed
@pabloDeputter
pabloDeputter deleted the pablo/aws-sdk-span-descriptions branch October 23, 2026 11:21

This branch was successfully deployed

1 active deployment
Preview — 8d223ce5 Deployed Oct 23, 2026 by vercel[bot]
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