← Back to issue list

lxd: introduce LXD for interacting with `lxd` command-line

View original Github issue

Metadata

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

Current evaluation

Merged PR #34 (commit 2a9e742, merge e181ccd) added an LXD class wrapping the `lxd` CLI plus is_supported_version() for minimum-version checks; it ships in craft_providers/lxd/lxd.py today.

Resolution: closed resolved

Reason: The PR was approved by 2 reviewers and merged into main as commit 2a9e742 ("lxd: introduce LXD for interacting with `lxd` command-line", +379/-0). Its is_supported_version() helper survives in craft_providers/lxd/lxd.py:78 and is consumed by craft_providers/lxd/installer.py:179.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Includes a `is_supported_version()` helper to ensure LXD version meets minimum required version by craft-providers. 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 PR #34 (commit 2a9e742, merge e181ccd) added an LXD class wrapping the `lxd` CLI plus is_supported_version() for minimum-version checks; it ships in craft_providers/lxd/lxd.py today.
qwen/qwen3.6-35b-a3b — closed resolved Merged PR introducing LXD command-line interaction support and an `is_supported_version()` helper to enforce minimum LXD version requirements.

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.
81%
#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.
73%
#71 lxd: account for minimum required version in checks (CRAFT-437) craft-providers merged Merged PR #71 updated LXD readiness checks to enforce the minimum required version and revised ensure_lxd_is_ready() error messages (implementation 9fcd49b).
70%