feat(attributes): Add runtime context attributes#383
Conversation
Add top-level runtime.* attributes mapping to the Sentry runtime context: runtime.name, runtime.version, runtime.build, and runtime.raw_description. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Semver Impact of This PR🟡 Minor (new features) 📋 Changelog PreviewThis is how your changes will appear in the changelog. New Features ✨Attributes
Bug Fixes 🐛Attributes
Other
Internal Changes 🔧
🤖 This preview updates automatically when you update the PR. |
buenaflor
left a comment
There was a problem hiding this comment.
are these the runtime attributes that we send in mobile today?
There was a problem hiding this comment.
I think we should add these since we already send them but deprecate them in favour of something else
it'd confusing looking at this + process.runtime
additionally I saw process.runtime.name is supposed to be the same as this runtime.name
https://getsentry.github.io/sentry-conventions/attributes/process/#process-runtime-name
|
Yeah it's confusing, as far as I can see runtime exists for a while now (e.g. sentry-java backend or cocoa).
|
yea, though migrations like backfilling and coalescing are currently limited to EAP afaict we don't need to migrate SDKs now but I guess it does make sense to at least document it as a deprecated attribute e.g |
Summary
runtime.*attributes that map to the existing Sentry runtime context fieldsruntime.name,runtime.version,runtime.build,runtime.raw_descriptionprocess.runtime.*attributes, soruntime.name,runtime.version, andruntime.raw_descriptionare marked as deprecated with_status: backfill, pointing toprocess.runtime.name,process.runtime.version, andprocess.runtime.descriptionrespectivelyruntime.buildhas no OTel equivalent and remains non-deprecatedTest plan
yarn testpasses (5290 JS + 3 Python tests)yarn run generateran successfully🤖 Generated with Claude Code