lxd: launch instances from local instance instead of image
Metadata
Current evaluation
Merged PR #177 (commit cd70943) refactoring craft_providers/lxd/launcher.py to cache configured LXD environments as a base instance instead of an image, cutting launch time ~19-34s. Deprecates use_snapshots in favor of use_base_instance. (CRAFT-1339)
Resolution: closed resolved
Reason: PR #177 was approved and merged into main as commit cd70943, delivering the base-instance mechanism described in the description. Follow-ups #178 (test refactor), #187, and #202 (adopting use_base_instance) landed on top, and the deprecated parameter was later removed in c258e02.
Evaluated by: mr-cal-bot
Issue body
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #177 (commit cd70943) refactoring craft_providers/lxd/launcher.py to cache configured LXD environments as a base instance instead of an image, cutting launch time ~19-34s. Deprecates use_snapshots in favor of use_base_instance. (CRAFT-1339) | |
| qwen/qwen3.6-35b-a3b | — | closed resolved | Refactored craft_providers/lxd/launcher.py to cache configured LXD environments as a base instance instead of an image, reducing setup time and deprecating use_snapshots in favor of use_base_instance (CRAFT-1339). |
Update history
| Date | Change |
|---|---|
| created |
Related work
-
Likely Fixed By:
canonical/craft-providers#178
(confidence 85%)
Merged test refactor cited in PR #177 body as prerequisite; it enabled testing the new base-instance launcher.
-
Related To:
canonical/craft-providers#202
(confidence 80%)
Merged as commit fd23832; wires the new use_base_instance parameter through LXDProvider launches.
-
Related To:
canonical/craft-providers#187
(confidence 70%)
Merged as commit b0df595; adds base-instance assertions and an lxc.info() integration test for the merged behavior.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #202 lxd: use new `use_base_instance` parameter | craft-providers | merged | Merged in PR #202 as commit fd23832, forwarding use_base_instance for LXDProvider launches; included in craft-providers 1.8.0. |