sentry: support disabling error reporting
Metadata
Current evaluation
Merged PR #2214 (commit 1c93ab107), restoring the ability to disable Sentry after regression #2154 caused tests to hang.
Resolution: closed resolved
Reason: PR #2214 was merged as commit 1c93ab107, restoring the Sentry opt-out and resolving the regression identified in #2154.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [x] Have 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 tests/unit`?
-----
We recently regressed on #2154 by removing the ability to disable Sentry, which can cause tests to hang. Add this ability back.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #2214 (commit 1c93ab107), restoring the ability to disable Sentry after regression #2154 caused tests to hang. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in #2214 (commit 1c93ab107), restoring the ability to disable Sentry error reporting after a regression that could cause tests to hang. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged into master to restore the ability to disable Sentry error reporting, fixing a regression that caused tests to hang. Approved, passed CI, and achieved 100% diff coverage. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into master. Restores ability to disable Sentry error reporting, fixing a regression that caused tests to hang. Achieved 100% diff coverage and integrated into main branch. |
Update history
No update history recorded yet.
Related work
-
Related To:
snapcraft#2154
(confidence 95%)
Earlier change disabled Sentry during tests to prevent hangs.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2154 tests: disable sentry | snapcraft | merged | Merged in snapcraft#2154 (commit aa9fdafc9), disabling Sentry during tests so test errors no longer hang waiting for user input. | |
| #1961 Sentry | snapcraft | merged | Merged PR #1961 as commit 364596087, adding confirmation-gated Sentry reports for snap builds and closing #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. | |
| #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. |