Skip to content

fix: Change type of sentry.dsc.project_id to string#400

Open
elramen wants to merge 1 commit into
mainfrom
elramen-dsc
Open

fix: Change type of sentry.dsc.project_id to string#400
elramen wants to merge 1 commit into
mainfrom
elramen-dsc

Conversation

@elramen
Copy link
Copy Markdown
Member

@elramen elramen commented May 21, 2026

Description

Change the type of sentry.dsc.project_id from integer to string as there is no guarantee that a project id will fit in an i64 (u64 isn't supported by EAP). This prevents issues if we were to change the project id generation logic in the future.

Fixes INGEST-927

PR Checklist

  • I have run yarn test and verified that the tests pass.
  • I have run yarn generate to generate and format code and docs.

@elramen elramen requested review from a team, Lms24, cleptric, mjq and nsdeschenes as code owners May 21, 2026 12:00
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 21, 2026

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 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 additional_context to gen_ai cost and usage attributes by matejminar in #397
  • (schema) Add additional_context field to attribute definitions by matejminar in #393

Bug Fixes 🐛

  • Change type of sentry.dsc.project_id to string by elramen in #400

🤖 This preview updates automatically when you update the PR.

@linear-code
Copy link
Copy Markdown

linear-code Bot commented May 21, 2026

INGEST-927

Copy link
Copy Markdown
Member

@Lms24 Lms24 left a comment

Choose a reason for hiding this comment

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

thanks!

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.

3 participants