snap step prints too much information about mksquashfs
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Snapcraft |
Fix Released
|
Wishlist
|
Sergio Schvezov | ||
| snapcraft (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
| Xenial |
Fix Released
|
Undecided
|
Unassigned | ||
| Yakkety |
Fix Released
|
Undecided
|
Unassigned | ||
Bug Description
The snapcraft build finishes with something like this:
Snapping kpcli-elopio_
Parallel mksquashfs: Using 10 processors
Creating 4.0 filesystem on kpcli-elopio_
[======
Exportable Squashfs 4.0 filesystem, xz compressed, data block size 131072
compressed data, compressed metadata, compressed fragments, no xattrs
duplicates are removed
Filesystem size 12193.21 Kbytes (11.91 Mbytes)
23.13% of uncompressed filesystem size (52710.07 Kbytes)
Inode table size 25248 bytes (24.66 Kbytes)
23.25% of uncompressed inode table size (108609 bytes)
Directory table size 28830 bytes (28.15 Kbytes)
42.88% of uncompressed directory table size (67241 bytes)
Number of duplicate files found 481
Number of inodes 3347
Number of files 2752
Number of fragments 168
Number of symbolic links 40
Number of device nodes 0
Number of fifo nodes 0
Number of socket nodes 0
Number of directories 555
Number of ids (unique uids + gids) 1
Number of uids 1
root (0)
Number of gids 1
root (0)
Snapped kpcli-elopio_
Everything between the progress bar and the Snapped message seems irrelevant to me.
[Impact]
* The output of every snapcraft build is too noise and not really understandable.
* A cleaner output makes the experience better, and helps to find real errors.
[Test Case]
* Run snapcraft with and without debug to generate a snap.
* Check that a progress animations is shown.
* Check that the output is short and to the point without the flag, and that it's the same as before with the flag.
[Regression Potential]
* This touches the final step of the snap generation and could break it. We are safe against this regression because we have a suite that generates, installs and executes snaps.
| Changed in snapcraft: | |
| assignee: | nobody → Sergio Schvezov (sergiusens) |
| importance: | Undecided → Wishlist |
| status: | New → Triaged |
| milestone: | none → 2.10 |
| Mark Shuttleworth (sabdfl) wrote | #1 |
| Gustavo Niemeyer (niemeyer) wrote | #2 |
| Kyle Fazzari (kyrofa) wrote | #3 |
| Sergio Schvezov (sergiusens) wrote | #4 |
| Changed in snapcraft: | |
| status: | Triaged → In Progress |
| Changed in snapcraft: | |
| status: | In Progress → Fix Committed |
| description: | updated |
| summary: |
- snap step prints too much information abut mksquashfs + snap step prints too much information about mksquashfs |
| Changed in snapcraft (Ubuntu Xenial): | |
| milestone: | none → xenial-updates |
| Launchpad Janitor (janitor) wrote | #5 |
| Changed in snapcraft (Ubuntu Yakkety): | |
| status: | New → Fix Released |
| Chris J Arges (arges) wrote Please test proposed package | #6 |
| Changed in snapcraft (Ubuntu Xenial): | |
| status: | New → Fix Committed |
| tags: | added: verification-needed |
| Changed in snapcraft: | |
| status: | Fix Committed → Fix Released |
| Leo Arias (elopio) wrote | #7 |
| tags: |
added: verification-done removed: verification-needed |
| Launchpad Janitor (janitor) wrote | #8 |
| Changed in snapcraft (Ubuntu Xenial): | |
| status: | Fix Committed → Fix Released |
Agree, let's suppress the squashfs output unless there is an error.