lxd: always remove existing device for project folder
Metadata
Current evaluation
Merged as a follow-up to #1483, ensuring existing devices are always removed for the LXD project folder. Approved by two reviewers with zero unresolved comments. The change modifies three files, adds 64 lines, removes one, and passed all CI checks.
Suggested action: —
No scores available.
Issue body
As a follow-up on #1483 this change removes an existing device for the project folder.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged as a follow-up to #1483, ensuring existing devices are always removed for the LXD project folder. Approved by two reviewers with zero unresolved comments. The change modifies three files, adds 64 lines, removes one, and passed all CI checks. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix ensuring existing LXD devices for the project folder are always removed. This addresses a follow-up to #1483. Test mocking discussions occurred but did not impact the merge. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a follow-up to #1483 ensuring existing devices for the LXD project folder are always removed. Updates device handling logic and adjusts test mocking for container state verification. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1403 lxd: Only remove container if one exists | snapcraft | merged | Merged following two reviewer approvals and passing CI. The update resolves duplicate exceptions during failed LXD container creation by adding a conditional check before attempting container removal. | |
| #1762 lxd: delete container only if parts is empty | snapcraft | merged | Merged fix resolving bug 1734145 by updating LXD container cleanup logic to only delete containers when parts are empty. Approved by one reviewer, passed CI checks, and modified two files. | |
| #1516 lxd: LXD not installed when using remote | snapcraft | merged | Merged fix for a bug where lxc installation was not correctly verified when using a LXD remote. Also corrected incomplete test scenario usage. Approved by two reviewers, modified two files. | |
| #1434 lxd: clean with no parts should only delete | snapcraft | merged | Merged. Updates snapcraft LXD handling so running clean without arguments only deletes container resources instead of executing commands inside them. Approved by one reviewer and merged. | |
| #2089 many: remove support for remote lxd per project containers | snapcraft | merged | Merged removal of remote LXD per-project container support due to poor user experience, recommending cleanbuild instead. Environment detection was future-proofed. Approved, passed CI, and modified 13 files. | |
| #1483 lxd: path cannot have extra forward slashes | snapcraft | merged | Merged fix removing leading forward slashes from LXD device mount paths, resolving internal errors during device removal. The change prevents LXD from claiming unmounted devices exist, though existing containers require manual configuration updates. | |
| #1677 ci: don't install lxd | craft-parts | merged | Merged a CI configuration update removing the unused LXD installation. Approved by three reviewers, the change was integrated despite unrelated test failures stemming from a Launchpad issue. | |
| #817 lxd: use built-in image streams. | snapcraft | merged | Merged. Replaced custom remote setup for cleanbuilds with LXD built-in image streams, streamlining configuration. Approved by one reviewer with zero unresolved comments. The two-file change (+4/-13 lines) resolves LP #1625787. | |
| #1742 lxd: always remove tmp_dir after execution | snapcraft | merged | Merged fix ensuring the temporary directory is always removed after LXD container builds, preventing disk space waste on errors. Approved and passed CI. | |
| #3112 spread: remove dead code for lxd setup and add debug prints | snapcraft | merged | Merged removal of redundant elif dead code in LXD setup logic and addition of debug prints for LXD snap install failures. Approved by one reviewer, passed CI checks, and integrated. |