fix(i/snap): don't reject non-snap squashfs files
Metadata
Current evaluation
Merged PR #544 as commit e8bdea8, updating snap inspection to ignore non-snap SquashFS files instead of rejecting them; all CI checks passed.
Resolution: closed resolved
Reason: PR #544 was approved and merged into fetch-service as commit e8bdea8. The change fixes the snap inspector’s handling of non-snap SquashFS files.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Fix the snap inspector to ignore non-snap squashfs files instead of
rejecting them.
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #544 as commit e8bdea8, updating snap inspection to ignore non-snap SquashFS files instead of rejecting them; all CI checks passed. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in canonical/fetch-service#544 (commit e8bdea8): snap inspection now ignores non-snap squashfs files instead of rejecting them. All CI checks passed. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged a fix to the snap inspector that ignores non-snap squashfs files instead of rejecting them. Approved by two reviewers, passed all CI checks, and applied with 16 additions and 4 deletions across two files. |
Update history
| Date | Change |
|---|---|
| created |
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #178 fix(inspectors/snap): minor inspector fixes and more test coverage | fetch-service | merged | Merged in PR #178 as commit 521b263, adding Snap inspector bounds checks, propagating SquashFS reader-creation errors, and broader tests. | |
| #334 More attributes for the squashfs'ed snap | snapcraft | merged | Merged PR #334 via e93fcc0a8, adding mksquashfs -all-root and -no-xattrs to improve snap security and reproducibility. |