← Back to issue list

errors: enable sentry by default

View original Github issue

Metadata

Project
snapcraft
Number
#2187
Type
pull request
State
merged
Author
sergiusens
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #2187 in commit f77a6c10a, enabling Sentry error reporting by default and adding SNAPCRAFT_ENABLE_SILENT_REPORTS for prompt-free reports.

Resolution: closed resolved

Reason: PR #2187 was merged into master as commit f77a6c10a, completing the Sentry reporting and anonymity improvements.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Final touches to polish the experience, more ensurance of anonymity and adding a new option to send without prompting (SNAPCRAFT_ENABLE_SILENT_REPORTS). Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com> - [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`? - [ ] Have you successfully run `./runtests.sh unit`? ----- This is how it looks ``` (snapcraft) ubuntu@snapcraft-xenial-dev:~/source/make-hello$ snapcraft Pulling make-project Sorry, Snapcraft ran into an error when trying to running through its lifecycle that generated a trace that has been put in '/tmp/tmpucwb0wnf/trace.txt'. You can anonymously report this issue to the snapcraft developers. No other data than this traceback, this host's machine-id and the version of snapcraft in use will be sent. Would you like send this error data? (Yes/No/Always) [no]: n (snapcraft) ubuntu@snapcraft-xenial-dev:~/source/make-hello$ snapcraft Pulling make-project Sorry, Snapcraft ran into an error when trying to running through its lifecycle that generated a trace that has been put in '/tmp/tmpe0clvpg2/trace.txt'. You can anonymously report this issue to the snapcraft developers. No other data than this traceback, this host's machine-id and the version of snapcraft in use will be sent. Would you like send this error data? (Yes/No/Always) [no]: y Thank you, sent. (snapcraft) ubuntu@snapcraft-xenial-dev:~/source/make-hello$ SNAPCRAFT_ENABLE_SILENT_REPORT=y snapcraft Pulling make-project Sorry, Snapcraft ran into an error when trying to running through its lifecycle that generated a trace that has been put in '/tmp/tmpft1qi_ni/trace.txt'. Sending an error report because SNAPCRAFT_ENABLE_SILENT_REPORT is set. Thank you, sent. (snapcraft) ubuntu@snapcraft-xenial-dev:~/source/make-hello$ snapcraft Pulling make-project Sorry, Snapcraft ran into an error when trying to running through its lifecycle that generated a trace that has been put in '/tmp/tmpmp3dfft7/trace.txt'. You can anonymously report this issue to the snapcraft developers. No other data than this traceback, this host's machine-id and the version of snapcraft in use will be sent. Would you like send this error data? (Yes/No/Always) [no]: always Saving choice to always send data to Sentry Thank you, sent. (snapcraft) ubuntu@snapcraft-xenial-dev:~/source/make-hello$ snapcraft Pulling make-project Sorry, Snapcraft ran into an error when trying to running through its lifecycle that generated a trace that has been put in '/tmp/tmppvenll25/trace.txt'. Sending an error report because ALWAYS was selected in a past prompt. Thank you, sent. ```

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #2187 in commit f77a6c10a, enabling Sentry error reporting by default and adding SNAPCRAFT_ENABLE_SILENT_REPORTS for prompt-free reports.
stealth/space-bunny-alpha — closed resolved Merged in snapcore/snapcraft#2187 via commit f77a6c10a, enabling Sentry error reporting by default and adding SNAPCRAFT_ENABLE_SILENT_REPORTS for prompt-free reports.
qwen/qwen3.6-35b-a3b — — Merged to master, enabling Sentry error reporting by default in Snapcraft. Introduces SNAPCRAFT_ENABLE_SILENT_REPORTS for silent submissions. Approved by reviewers and integrated despite a minor 0.02% coverage drop.
qwen3.6-35b-a3b-mtp-q6 — — Merged into master. Enables Sentry error reporting by default and introduces the SNAPCRAFT_ENABLE_SILENT_REPORTS environment variable for silent submission. Codecov noted a minor 0.02% coverage decrease upon merge.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1785231 enable error reporting by default snapcraft (launchpad) closed Enabled Sentry error reporting by default in snapcore/snapcraft#2187 (commit f77a6c10a), with prompt-free reports controlled by SNAPCRAFT_ENABLE_SILENT_REPORTS.
86%
#2101 errors: implement an always option to send to sentry snapcraft merged Merged PR #2101 in commit 5fd2b3220, adding an always option to send Snapcraft error reports to Sentry.
78%
#1961 Sentry snapcraft merged Merged PR #1961 as commit 364596087, adding confirmation-gated Sentry reports for snap builds and closing #1918.
74%
#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.
74%
#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.
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.
72%
#1767151 Setting SNAPCRAFT_ENABLE_SENTRY= causes stacktrace (empty string value not truthy) snapcraft (launchpad) closed Fixed by removing the legacy Sentry error-reporting stack entirely; SNAPCRAFT_ENABLE_SENTRY is no longer parsed in snapcraft.
72%
#2602 errors: refactor exception/error handling snapcraft merged Merged PR #2602 (commit 7d0391dc5, 'errors: refactor exception/error handling') reworked snapcraft/_errors.py: always print exception messages, conditionally write tracebacks to a temp file, gate bug reporting on SNAPCRAFT_ENABLE_SILENT_REPORT, and honor error exit codes.
70%