Improve error reporting
Metadata
Current evaluation
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.
Suggested action: —
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 |
|---|---|---|---|---|
| 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged after maintainer approval. Updates error handling to log messages before exiting with status 1, updates tests, and adds specific error reporting for uploading non-existent snaps. Changes were squashed and integrated. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1325 cli: proper error for failed snap command | snapcraft | merged | Merged: returns a proper error code when a snap command fails. Approved by one reviewer with no unresolved comments. Changes span two files (+15/-2). | |
| #1815 repo: handle invalid snaps | snapcraft | merged | Merged changes handling invalid snaps in build-snaps with a clear error message. Adds retries to information fetching for improved network failure debugging. Approved after resolving a minor CI typo. | |
| #4876 Improve error reporting for snap builds | snapcraft | closed | Contributor jawadsalwati prepared a fix to clarify the snapcraft base validation error and updated unit tests. The issue was closed after the pull request was merged, improving error messaging for kernel and base types. |