← Back to issue list

Remove Sentry and fallback error reporting (#5582)

View original Github issue

Metadata

Project
snapcraft
Number
#5592
Type
pull request
State
merged
Author
ahkazak23
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #5592 as commit 0a120c694, removing Snapcraft Legacy's Sentry/Raven reporting, error-data prompts, and Launchpad fallback; closed #5582.

Resolution: closed resolved

Reason: PR #5592 was approved and merged as commit 0a120c694. The change removed Sentry/Raven error reporting and the fallback Launchpad prompt, resolving issue #5582.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [x] Have you followed the [guidelines for contributing](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md)? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you successfully run `make lint`? - [x] Have you successfully run `make test`? --- Closes #5582 This PR removes all Sentry-related error reporting logic and the fallback Launchpad bug prompt from the Snapcraft CLI: ### 🔧 Changes - Deleted Raven/Sentry imports and submission functions - Removed interactive “send error data?” prompts - Removed Launchpad fallback message - Stripped out related config methods - Cleaned up and reformatted via pre-commit (ruff) ### ✅ Testing - Ran `make lint` / `pre-commit` hooks (all passed) - Ran `make test` / `pytest` (all tests pass) - Verified no `sentry` or `raven` references remain using `grep`

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #5592 as commit 0a120c694, removing Snapcraft Legacy's Sentry/Raven reporting, error-data prompts, and Launchpad fallback; closed #5582.
stealth/space-bunny-alpha — closed resolved Merged PR #5592 removed Sentry/Raven reporting, error-data prompts, and Launchpad fallback reporting from Snapcraft Legacy; it closes #5582 via commit 0a120c694.
stealth/space-bunny-alpha — closed resolved Merged PR #5592 as commit 0a120c694, removing Sentry/Raven reporting, interactive error-data prompts, and the Launchpad fallback from Snapcraft; closes issue #5582.
qwen/qwen3.6-35b-a3b — — Merged. Removes Sentry and Raven error reporting, interactive prompts, and Launchpad fallback from the Snapcraft CLI. Deletes imports, strips config methods, and cleans up code. Approved by reviewers and passed CI tests prior to merge.
qwen3.6-35b-a3b-mtp-q6 — — Merged #5582 removes Sentry error reporting and the Launchpad fallback prompt from the Snapcraft CLI. The update deletes related imports, functions, and config methods, cleans up code, and passes all lint and test checks.

Update history

No update history recorded yet.

Related work

Related issues

Issue Project State Summary Similarity
#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.
92%
#1961 Sentry snapcraft merged Merged PR #1961 as commit 364596087, adding confirmation-gated Sentry reports for snap builds and closing #1918.
76%
#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.
74%
#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.
73%
#1792182 Don't prompt to report Sentry errors unless stdin is a tty snapcraft (launchpad) closed Obsolete after canonical/snapcraft#5592 (commit 0a120c694) removed Sentry reporting from Snapcraft Legacy, eliminating the non-TTY reporting prompt.
71%
#1791968 Snapcraft lies when submission to sentry fails snapcraft (launchpad) closed Snapcraft now handles Sentry submission failures gracefully instead of falsely reporting success. Fix merged in canonical/snapcraft#2271 via commit 859c111e5.
70%