lxd: use built-in image streams.
Metadata
Current evaluation
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.
Suggested action: —
No scores available.
Issue body
Instead of setting up our own remote for each cleanbuild
invocation use the provided built-in image streams from
lxd.
LP: #1625787
Signed-off-by: Sergio Schvezov sergio.schvezov@ubuntu.com
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Replaced custom remote configuration for cleanbuilds with LXD's built-in image streams, eliminating redundant remotes and simplifying setup. Approved by @come-maiz. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Replaces custom remote setup for cleanbuild invocations with LXD's built-in image streams, resolving LP #1625787. Approved by @come-maiz. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #3271 build providers: use the releases endpoint for LXD | snapcraft | merged | Merged a two-file change switching the simplestreams endpoint from daily to releases for LXD image retrieval. Approved by one reviewer, passed CI checks, and completed despite noted unrelated test failures. | |
| #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. | |
| #1769 lxd: add an --image argument to cleanbuild | snapcraft | closed | Rejected and closed. Maintainers abandoned container-based builds in favor of VMs, rendering the LXD image argument obsolete. The PR was not merged. | |
| #1627 lxd: split container classes into different files | snapcraft | merged | Merged refactoring splitting LXD container classes from lxd.py into separate modules. Improves organization, moves Cleanbuilder methods, de-duplicates env vars, and fixes mocking imports. No functional changes. Approved by two reviewers 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. | |
| #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. | |
| #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. | |
| #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. | |
| #1519 lxd: use a unique temporary folder | snapcraft | merged | Merged to resolve parallel snapcraft cleanbuild failures by moving the temporary folder initialization to the constructor. Removed the is_snap condition, relocated tests, and passed CI review. Modified 5 files. | |
| #1230 lxd: refactor Cleanbuilder into Containerbuild and add Project. | snapcraft | merged | Merged. Refactored Cleanbuilder into a Containerbuild base class and introduced a Project class to manage persistent LXD containers for snap builds. Approved after code review and test fixes. |