Improve error reporting
Metadata
Current evaluation
Merged PR #591 as commit b9258ea73, making errors log before exit status 1 and adding a clear error for uploads of nonexistent snaps.
Resolution: closed resolved
Reason: PR #591 was merged into Snapcraft as commit b9258ea73. It updated error exits and tests and added specific reporting for nonexistent snap uploads, resolving LP #1588023.
Evaluated by: mr-cal-bot
No scores available.
Issue body
This changes the way we exit on error to first log an error and then
exit with a status code of 1. The tests are updated for this new
behaviour.
A proper error when trying to upload an unexisting snap is also added.
LP: #1588023
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #591 as commit b9258ea73, making errors log before exit status 1 and adding a clear error for uploads of nonexistent snaps. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged error-reporting improvements: errors are logged before exiting with status 1, and nonexistent-snap uploads now produce a clear error. Resolves LP #1588023. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged after improving error reporting to log errors before exiting with status 1. Tests updated and specific handling added for uploading non-existent snaps. Resolved following maintainer review and commit message adjustment. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged after retests and commit adjustments. Improves error reporting by logging errors before exiting with status 1, updates tests, and adds proper error handling for uploading non-existent snaps. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #401 fix: add snap commands logging | craft-providers | merged | Merged PR #401 added Snap command logging to expose command output and errors for debugging, fixing #396; landed as commit d7aa4dc. | |
| #5188 fix: improve error message from snapd pack | snapcraft | merged | Merged PR #5188 in commit a9fc44093, improving snapd pack failures by surfacing snapd's error output and fixing snapcraft#4499. | |
| #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. |