← Back to issue list

lxd: introduce LXDInstance

View original Github issue

Metadata

Project
craft-providers
Number
#33
Type
pull request
State
merged
Author
cjp256
Labels
—
Created
Updated
Closed

Current evaluation

Merged as commit 260c038, introducing LXDInstance, exposing the lxc fixture to LXD integration tests, and adding unit and integration coverage.

Resolution: closed resolved

Reason: PR #33 was approved and merged as commit 260c038 (lxd: introduce LXDInstance). The change delivered the implementation and test coverage described in the PR.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- make `lxc` fixture available to all lxd integration tests. - add unit & integration tests Signed-off-by: Chris Patterson <chris.patterson@canonical.com> - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged as commit 260c038, introducing LXDInstance, exposing the lxc fixture to LXD integration tests, and adding unit and integration coverage.
qwen/qwen3.6-35b-a3b — closed resolved Merged into main, introducing the LXDInstance class, making the lxc fixture available to all LXD integration tests, and adding unit and integration tests.

Update history

Date Change
created

Related issues

Issue Project State Summary Similarity
#31 lxd: introduce LXC to interface with lxc craft-providers merged Merged PR adding the LXC wrapper class to craft_providers/lxd/lxc.py for all lxc CLI interaction (+2627/-0, 3 files). Merged as 38f233f (feature commit 0d400f6); integration tests landed separately in #32.
74%
#187 tests: improve lxd launcher tests for base instances craft-providers merged Merged in b0df595: expanded LXD base-instance tests, mocked _is_valid(), added lxc.info() integration coverage, and clarified an error message.
73%
#36 lxd: introduce installation support craft-providers merged Merged PR #36 added LXD installation support, including installer logic and tests; implementation commit aab8fd0 was merged via 1a1212c.
72%