sanity checks: run properly on build VMs
Metadata
Current evaluation
Merged snapcraft#2279 in commit 6f4ca585f, making snap-directory sanity checks project-relative instead of CWD-relative so they work on build VMs (LP #1792962).
Resolution: closed resolved
Reason: PR #2279 was merged as commit 6f4ca585f. It fixed LP #1792962 by resolving the snap-directory check against the project directory rather than the current working directory.
Evaluated by: mr-cal-bot
No scores available.
Issue body
The snapcraft CLI runs sanity checks. The check of the snap/ directory is currently relative to cwd, which breaks in build VMs. This PR fixes [LP: #1792962](https://bugs.launchpad.net/snapcraft/+bug/1792962) by making it relative to the project directory instead.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged snapcraft#2279 in commit 6f4ca585f, making snap-directory sanity checks project-relative instead of CWD-relative so they work on build VMs (LP #1792962). | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #2279 (commit 6f4ca585f), making snap-directory sanity checks project-relative instead of cwd-relative so they work on build VMs (LP #1792962). | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in canonical/snapcraft#2279 as 6f4ca585f, making sanity checks project-relative so they work in build VMs and resolving LP: #1792962. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged. Updated snap directory sanity checks from cwd-relative to project-relative paths to resolve build VM failures. Resolves LP: #1792962. Approved by one reviewer with 14 additions and 7 deletions across two files. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged fix for sanity checks failing in build VMs. The snap/ directory validation previously used a path relative to the current working directory. Changed to project-relative paths, resolving LP: #1792962. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1792962 Sanity checks don't work in build VMs | snapcraft (launchpad) | closed | Sanity checks were fixed for build VMs by snapcore/snapcraft#2279, merged as 6f4ca585f; checks now resolve snap/ relative to the project rather than CWD. | |
| #2268 sanity checks: allow snap/local dir | snapcraft | merged | Merged PR #2268 via commit 10bdb0473 allowed snapcraft-specific files under snap/local/ and added tests, resolving LP #1792203. | |
| #2919 sanity_checks: fix sanity check on Windows | snapcraft | merged | Merged PR #2919 fixed Windows sanity checks for expected snap directories by allowing backslash paths; landed as commit cc40d5f68. |