cli: proper error for failed snap command
Metadata
Current evaluation
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).
Suggested action: —
No scores available.
Issue body
Return a proper error code when failing to snap
Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | 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). | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged CLI update that returns a proper error code when a snap command fails. Authored by maintainer sergiusens, the change was integrated directly into the codebase without review comments. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged fix updates the CLI to return a proper error code when a snap command fails, ensuring accurate failure reporting and improved debugging. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #3974 snap config: catch more errors when config cannot be retrieved | snapcraft | merged | Merged improvements to snap configuration error handling. Follows up on PR #3971 to catch additional retrieval failures reported by users. Approved by one reviewer, passed CI checks, and updated four files (+30/-10). | |
| #591 Improve error reporting | snapcraft | merged | 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. | |
| #497 fix(config): warn on snapctl errors | craft-application | merged | Merged a configuration fix that warns on snapctl errors. Approved by two reviewers, the change adds 32 lines across two files and addresses pending review feedback. | |
| #2299 docs: fix formatting issues | charmcraft | merged | Merged documentation formatting fixes resolving local build errors. Approved by two reviewers with minimal changes (+3/-1 across two files). Merged despite failing snap-build CI checks. | |
| #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. | |
| #3640 spread: update error when local snap is missing | snapcraft | merged | Merged a one-line change to improve error reporting when a local snap is missing during spread updates. Approved by a reviewer, passed CI, and integrated into the repository. | |
| #3971 snap config: do not crash when config cannot be retrieved | snapcraft | merged | Merged. Fixed a crash in get_snap_config() when snapd is unavailable by catching and logging errors. Approved by reviewer and passed CI checks. | |
| #3396 snap: address snapcraft warnings for app command | snapcraft | merged | Merged a single-line fix to address snapcraft warnings for the app command. Approved by one reviewer with zero unresolved comments and passing CI checks. | |
| #2609 schema: allow 'snapd' snap type | snapcraft | merged | Merged to allow the 'snapd' snap type in the schema. Approved by one reviewer, passed CI checks, and modified three files. | |
| #228 packages: run snap operations silently | craft-parts | merged | Merged to fix #218 by running snap subprocesses silently, preventing stdout leakage that conflicts with craft-cli during lifecycle execution. Approved by one reviewer, passed CI, and modified two files (+32/-15 lines). |