spread tests: do not attempt to remove snapd snap
Metadata
Current evaluation
Merged in commit b9290bb32; spread tests now skip removing the protected snapd snap, fixing LXD test failures.
Resolution: closed resolved
Reason: PR snapcraft#2937 was approved and merged as commit b9290bb32, resolving LXD spread-test errors caused by attempting to remove snapd.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Fixes spread test errors with LXD using snapd snap.
error: cannot remove "snapd": snap "snapd" is not removable:
remove all other snaps first
Signed-off-by: Chris Patterson <chris.patterson@canonical.com>
- [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `./runtests.sh static`?
- [ ] Have you successfully run `./runtests.sh tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged in commit b9290bb32; spread tests now skip removing the protected snapd snap, fixing LXD test failures. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in #2937 (commit b9290bb32): spread tests no longer remove the snapd snap, fixing LXD failures caused by snapd's protected status. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged a one-line fix to prevent spread tests from removing the snapd snap in LXD environments, resolving test failures caused by snapd's protected status. Approved by one reviewer with passing CI checks. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix for spread test failures in LXD. The change prevents cleanup routines from attempting to remove the snapd snap, resolving errors caused by its immutability when other snaps are present. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1133 test(spread): make LXD refresh in prepare more reliable | craft-application | closed | Closed unmerged; its LXD auto-refresh race fix was superseded by merged PR #1170 (commit fea3e835), which waits for active snap refreshes during spread prepare. | |
| #3957 tests: retry installing lxd snap | snapcraft | merged | PR #3957 merged as 57cd1d97b, adding snapd-testing-tools retries for LXD installation in spread tests to work around intermittent snapd bug 1870201. | |
| #534 test(spread): workaround for flaky test | fetch-service | merged | Merged PR #534 (commit 4f89bb7) worked around #533 by skipping the flaky snap names-query assertion while retaining the other spread-test checks. | |
| #521 test(spread): add an improved snap inspector test | fetch-service | merged | Merged in PR #521 (commit 06cd210), adding a more realistic snapd setup to the spread snap inspector test and exposing additional snapd network requests. | |
| #2983 tests: add LXD spread test | snapcraft | merged | Merged PR #2983 as 4b8522a14, adding spread coverage that builds the LXD package with LXD and destructive/host providers; installation checks were deferred. | |
| #302 test(spread): switch to testing in destructive mode | imagecraft | merged | Merged PR #302 as commit f6aae5e, switching spread tests to destructive mode while LXD lacks required loop-device support. | |
| #5845 test(spread): test building base and snapd snaps | snapcraft | merged | Merged as fb8c81e8f, adding manual Spread coverage that builds current base snaps and the snapd snap against mainline Snapcraft. | |
| #269 tests: add initial spread testing | fetch-service | merged | Merged PR #269 (commit 8fe0091) added initial Spread integration tests to fetch-service: spread.yaml plus snap-config, basic-inspection, update-config and upgrade-revert suites; all 10 lxd-vm tasks passed, resolving #226. | |
| #2631 spread tests: enable LXD build provider tests | snapcraft | merged | Merged PR #2631 enabled LXD build-provider spread tests and reused early-prepared LXD setup, reducing repeated setup and image-download overhead. | |
| #483 test(spread): run snap-preseed tests in lxd | imagecraft | merged | Merged PR #483 to update snap-preseed plugin tests to run in LXD across all systems, addressing CRAFT-5332. |