Skip to content
This repository was archived by the owner on Jun 26, 2026. It is now read-only.

Improve GPU driver reporting for Sentry and telemetry - #1594

Draft
benceruleanlu wants to merge 1 commit into
mainfrom
fix/sentry-gpu-driver-reporting
Draft

Improve GPU driver reporting for Sentry and telemetry#1594
benceruleanlu wants to merge 1 commit into
mainfrom
fix/sentry-gpu-driver-reporting

Conversation

@benceruleanlu

@benceruleanlu benceruleanlu commented Feb 7, 2026

Copy link
Copy Markdown
Member

Summary

  • add a shared GPU metadata collector used by both Sentry and telemetry
  • normalize driverVersion values and backfill missing NVIDIA driver versions with nvidia-smi query + fallback parse
  • initialize Sentry GPU context earlier in startup and make context setup idempotent
  • reuse the shared NVIDIA driver-version utility in InstallationManager to remove duplicate detection logic

Testing

  • yarn format
  • yarn lint
  • yarn typecheck
  • yarn vitest run tests/unit/services/gpuInfo.test.ts tests/unit/services/telemetry.test.ts tests/unit/install/installationManager.test.ts

┆Issue is synchronized with this Notion page by Unito

@coderabbitai

coderabbitai Bot commented Feb 7, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • 🔍 Trigger a full review
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/sentry-gpu-driver-reporting

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant