Sentry
Metadata
Current evaluation
Merged PR #1961 as commit 364596087, adding confirmation-gated Sentry reports for snap builds and closing #1918.
Resolution: closed resolved
Reason: The approved PR was merged as commit 364596087. It enabled traceback reporting for snap-packaged Snapcraft with a y/n prompt and explicitly closed #1918.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Adds the ability to send reports, asking for confirmation before doing
so. This feature is only enabled for the deb as raven is not provided in
xenial and most of the user base will be using the snap.
Closes: #1918
- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [x] Hae you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)?
- [x] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)?
- [x] Have you successfully run `./runtests.sh static`?
- [x] Have you successfully run `./runtests.sh unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #1961 as commit 364596087, adding confirmation-gated Sentry reports for snap builds and closing #1918. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #1961 as commit 364596087, adding confirmed Sentry error reports for the snap and closing #1918. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged. Adds Sentry error reporting with user confirmation prompts, restricted to snap due to missing raven in xenial. Reviewer requested UX improvements for confirmation feedback, Ctrl+C handling, and debug logging scoping. Closes #1918. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged pull request adding Sentry error reporting with user confirmation prompts. Clarified as snap-only due to dependency constraints. Reviewer noted minor UX and logging improvements which were acknowledged. Closes #1918. |
Update history
No update history recorded yet.
Related work
-
Related To:
snapcraft#1918
(confidence 100%)
The request for confirmation before sending error reports; PR #1961 states that it closes this issue.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2069 Reports | snapcraft | merged | Merged PR #1961 via commit 364596087, adding Sentry traceback reporting with user confirmation prompts and closing issue #1918. | |
| #5592 Remove Sentry and fallback error reporting (#5582) | snapcraft | merged | Merged PR #5592 as commit 0a120c694, removing Snapcraft Legacy's Sentry/Raven reporting, error-data prompts, and Launchpad fallback; closed #5582. | |
| #2214 sentry: support disabling error reporting | snapcraft | merged | Merged PR #2214 (commit 1c93ab107), restoring the ability to disable Sentry after regression #2154 caused tests to hang. | |
| #2187 errors: enable sentry by default | snapcraft | merged | Merged PR #2187 in commit f77a6c10a, enabling Sentry error reporting by default and adding SNAPCRAFT_ENABLE_SILENT_REPORTS for prompt-free reports. | |
| #5582 Remove Sentry reporting | snapcraft | closed | Merged PR #5592 (commit 0a120c694) removed Sentry/Raven reporting, error-data prompts, and the Launchpad fallback; Snapcraft now shows errors without requesting submissions. | |
| #1947 errors: add ability to send stack traces to sentry | snapcraft | closed | PR #1947 was closed unmerged and superseded by #1961, merged as commit 364596087 with confirmation-gated Sentry traceback reporting for #1918. |