← Back to issue list

Snaps of type os should not squash with -all-root

View original Github issue

Metadata

Project
snapcraft
Number
#391
Type
pull request
State
merged
Author
sergiusens
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #391 in commit 8887fe616 so os-type snaps skip -all-root during squashfs packaging, preserving user permissions.

Resolution: closed resolved

Reason: PR #391 was merged as commit 8887fe616, incorporating implementation commit 13a0d1cda. The change prevents mksquashfs from applying -all-root to os-type snaps.

Evaluated by: mr-cal-bot

No scores available.

Issue body

When creating os snaps we want to preserve user permissions. LP: #1557513 Signed-off-by: Sergio Schvezov sergio.schvezov@ubuntu.com

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #391 in commit 8887fe616 so os-type snaps skip -all-root during squashfs packaging, preserving user permissions.
stealth/space-bunny-alpha — closed resolved Merged PR #391 in commit 8887fe616: OS-type snaps now skip -all-root when building squashfs, preserving user permissions.
qwen/qwen3.6-35b-a3b — — Merged into master. Prevents OS-type snaps from using the -all-root squash option to preserve user permissions, resolving LP #1557513. Approved by kyrofa and integrated following successful coverage checks.
qwen3.6-35b-a3b-mtp-q6 — — Merged into ubuntu-core:master. Prevents OS-type snaps from squashing with -all-root to preserve user permissions, resolving LP #1557513. Approved by kyrofa.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1557513 please add option to override -all-root in "snapcraft snap" snapcraft (launchpad) closed Resolved by canonical/snapcraft#391 (commit 8887fe616, 'Snaps of type os should not squash with -all-root'): snaps of type 'os' now omit mksquashfs -all-root, preserving system-user ownership for livecd rootfs builds.
75%
#2090 internal: skip -all-root for base snaps snapcraft merged Merged #2090 as commit 7f8ebc425, omitting -all-root for base snaps to preserve file ownership and prevent core18 files such as etc/shadow from becoming root:root.
71%