Push output needs love
Metadata
Current evaluation
Push output cleanup landed in snapcraft PR #2469 (commit 194589f0e): duplicate/full-path messages and the "cached source snap" line dropped to debug, filenames made consistent and shortened.
Resolution: closed resolved
Reason: cmatsuoka closed the issue after PR https://github.com/snapcore/snapcraft/pull/2469 was merged as commit 194589f0e ("cli: clean up snapcraft push output"), which removed the redundant "Preparing to push" and "Found cached source snap" lines from normal output and moved them to debug level.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Actual output just now:
1 | Pushing blender_2.80-f021536babb_amd64.snap
2 | After pushing, an attempt will be made to release to 'edge'
3 | Preparing to push '/home/niemeyer/src/snaps/blender/snaps/google:ubuntu-16.04-64:tasks/build:edge/blender_2.80-f021536babb_amd64.snap' to the store.
4 | Found cached source snap /home/niemeyer/.cache/snapcraft/projects/blender/snap_hashes/amd64/d7ad5854c6fb52acf45ede52159cd6a35775451739644f20a14c1ae087f423b6c1bc4beac4b7524667310b9115bead61.
5 | Generating xdelta3 delta for blender_2.80-f021536babb_amd64.snap.
6 | Pushing delta /home/niemeyer/src/snaps/blender/snaps/google:ubuntu-16.04-64:tasks/build:edge/blender_2.80-f021536babb_amd64.snap.xdelta3.
7 | Pushing blender_2.80-f021536babb_amd64.snap.xdelta3 [===============================================================================================================] 100%
Some comments (labeled to make discussion easier):
A. Lines are colored or not, with no clear reason (lines 1, 2, 7 are gray, rest is green). Everything there may be just standard gray.
B. Lines are very long due to the paths, which are uninteresting. I'd suggest presenting just the filenames.
C. Line 4 talks about a "source snap". Source code? Source for the delta? Unclear.. I suggest dropping that line altogether. The fact a delta is generated later means a delta source was found.
D. Some paths are quoted, others are not. Consistency says either all, or none.
E. Lines 6 and 7 say the same thing. Line 6 may be dropped.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Push output cleanup landed in snapcraft PR #2469 (commit 194589f0e): duplicate/full-path messages and the "cached source snap" line dropped to debug, filenames made consistent and shortened. | |
| stealth/space-bunny-alpha | — | closed resolved | Push output cleanup landed in snapcraft PR #2469 (commit 194589f0e): verbose push messages dropped to debug level, filenames shown instead of full paths, quoting made consistent, duplicate 'Pushing' lines removed. Colorization left for a global discussion. | |
| stealth/space-bunny-alpha | — | closed resolved | Resolved by merged PR #1240 (commit 0d7468013), which simplified push output using basenames and removed duplicate and trailing spinner output. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed without resolution or comments, likely abandoned. The report highlighted inconsistent formatting, overly long paths, and unclear terminology in push command output. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed without discussion. The feedback on CLI output formatting was likely addressed in later releases or superseded, as no comments or explicit resolution were recorded. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed without resolution. The report flagged formatting inconsistencies, verbose paths, and redundant lines in snap push output, but no changes were implemented or linked. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Closed with no comments. The push output formatting improvements were not implemented, leaving the issue abandoned. |
Update history
| Date | Change |
|---|---|
| created |
Related work
-
Likely Fixed By:
snapcraft#2469
(confidence 98%)
Merged PR referenced in the final maintainer comment; commit 194589f0e.
-
Related To:
snapcraft#1240
(confidence 40%)
Earlier commit 0d7468013 "cli: improve push output" addressing similar concerns.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. | |
| #1625784 Progress bar artifacts on output | snapcraft (launchpad) | closed | Fixed push-output artifacts by merging canonical/snapcraft#1240 in commit 0d7468013, removing duplicate messages and the trailing spinner. | |
| #1240 cli: improve push output. | snapcraft | merged | Merged #1240 via commit 0d7468013, improving push output with file basenames, removing duplicate messages and a trailing spinner, and renaming Upload to Push. | |
| #1834319 Cleaning up output | snapcraft (launchpad) | closed | Noisy duplicate Snapcraft output was cleaned up by merged PR canonical/snapcraft#2643 (cd38acef9), with the fix released in Snapcraft 3.7.1. |