fix: improve error message from snapd pack
Metadata
Current evaluation
Merged PR #5188 in commit a9fc44093, improving snapd pack failures by surfacing snapd's error output and fixing snapcraft#4499.
Resolution: closed resolved
Reason: The approved change was merged as commit a9fc44093. It reports snapd's error line when packing fails, resolving the missing-error problem tracked in snapcraft#4499.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [ ] Have you followed the [guidelines for contributing](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md)?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `tox run -m lint`?
- [ ] Have you successfully run `tox run -e test-py310`? (supported versions: `py39`, `py310`, `py311`, `py312`)
-----
Fixes #4499
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #5188 in commit a9fc44093, improving snapd pack failures by surfacing snapd's error output and fixing snapcraft#4499. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #5188 (commit a9fc44093) improved snapd pack failure output, surfacing snapd's stderr error and resolving hidden failure messages in #4499. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #5188 (commit a9fc44093) improved snapd pack failure messages, fixing issue #4499. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged fix improving snapd pack error messages, resolving issue #4499. Approved by three reviewers, the update modifies seven files with 122 additions and 9 deletions to enhance error reporting. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix improving the snapd pack error message, resolving issue #4499. Authored by maintainer sergiusens, the change was successfully integrated into the codebase. |
Update history
No update history recorded yet.
Related work
-
Related To:
snapcraft#4499
(confidence 100%)
The issue was explicitly identified as fixed by PR #5188.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #4413 Error message on snap pack is not nice | snapcraft | closed | Fixed by merged PR #4424 (commit ec09cc07b): pack failures now show concise snapd errors while command details remain in execution logs; backported to 7.x and 8.x. | |
| #4548 Make `snap pack ...` errors also return the problem instead of just the error code | snapcraft | closed | Closed by mr-cal as a duplicate of #4499 (tracked as CRAFT-2559); that issue was fixed by merged PR #5188 (commit a9fc44093), which surfaces snapd's error output when `snap pack` fails. | |
| #4499 Failures when running `snap pack` don't get sent to the user. | snapcraft | closed | Fixed by merged PR #5188 (commit a9fc44093), which surfaces snapd's error output when snap pack fails instead of showing only the command and exit status. | |
| #4870 snapcraft pack error obtuse on snapd errors | snapcraft | closed | Closed as a duplicate of #4548, which was tracked by #4499; hidden snapd pack diagnostics were fixed by merged PR #5188 (a9fc44093). | |
| #4923 Increase verbosity of `snapcraft`'s missing files error | snapcraft | closed | Closed as a duplicate of #4499; suppressed `snap pack` errors were fixed by merged PR #5188 (commit a9fc44093), which surfaced snapd's stderr. | |
| #1627865 Snapcraft errors when releasing a snap | snapcraft (launchpad) | closed | Resolved by improving snapcraft's error handling to check HTTP status codes before JSON decoding, alongside a fix for the underlying store race condition (bug #1610948). | |
| #113 fix: #112 and help messages | rockcraft | merged | Merged in PR #113 (commit 116f6f41), moving init out of lifecycle commands and correcting help text inherited from snapcraft; closes #112. | |
| #2492 store: handle invalid snap file errors | snapcraft | merged | PR #2492 (cmatsuoka) merged into master as commit 9e83f4575, adding error handling for corrupt/invalid snap files when extracting package data for sign or push, fixing SNAPCRAFT-GE. | |
| #2469 cli: clean up snapcraft push output | snapcraft | merged | Merged PR #2469 as commit 194589f0e, reducing snapcraft push noise, moving extra messages to debug level, and standardizing filename output. | |
| #5694 fix(backport): remove unnecessary snapd_snap flag (#5689) | snapcraft | merged | Merged backport #5694 (cb5283b6a) removed obsolete snapd_snap handling for Snapcraft 7.5, fixing core20 repacking failure #5316; original fix was #5689. |