lxd: LXD not installed when using remote
Metadata
Current evaluation
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.
Suggested action: —
No scores available.
Issue body
Whilst looking to simplify tests and add a new one more easily, I fixed the incomplete use of scenarios and an unexpected failure unveiled a bug.
When using a LXD remote, we don't correctly check if lxc is installed.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix for an incorrect LXD installation check when using a remote. The update also corrected incomplete scenario usage in tests, ensuring proper lxc validation during remote operations. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix for a bug where LXD installation was not correctly verified when using a remote. The update also corrected incomplete scenario usage in tests. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1916 lxd: initialize remote lazily | snapcraft | merged | Merged PR lazily initializes the LXD remote, resolving bug LP: #1746009. Approved after passing CI. A reviewer encountered an unrelated traceback during testing, which was noted for separate tracking but did not block the merge. | |
| #1400 lxd: Distingish FileNotFoundError if not installed | snapcraft | merged | Merged a fix distinguishing FileNotFoundError from CalledProcessError when LXD is not installed, resolving bug 1703344. Approved by one reviewer and merged despite minor unresolved comments and feedback on test fixture complexity. | |
| #1866 lxd: setup LXD remote for multipass | snapcraft | closed | Closed without merging. The change auto-configured an LXD remote for multipass but hit unresolved permission and mounting errors. Reviewers flagged inconsistent behavior, prompting closure pending fixes. | |
| #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. | |
| #2983 tests: add LXD spread test | snapcraft | merged | Merged after reviewer approval and passing CI checks. Added LXD spread test to verify LXD builds and installs. Modified two files with 29 added lines. | |
| #1771 lxd: suppress traceback when lxc launch / init fails | snapcraft | merged | Merged to suppress tracebacks when lxc launch or init fails, relying on lxc's native error output. Approved by one reviewer, modifying three files (+35/-15). | |
| #1629 lxd: fix the unit test for the user id map | snapcraft | merged | Merged a fix for an LXD unit test failure when run as root, introduced in #1588. Adjusts the user ID map test logic. Approved by two reviewers and passed CI. One remaining test error was noted but did not block the merge. | |
| #343 lxd should be a suggestion, not a direct dep | snapcraft | merged | Merged changes converting lxd from a direct dependency to a suggested package. Approved by reviewer and passed CI checks. | |
| #1488 lxd: always remove existing device for project folder | snapcraft | merged | 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. | |
| #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. |