← Back to issue list

lxd: account for minimum required version in checks (CRAFT-437)

View original Github issue

Metadata

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

Current evaluation

Merged PR #71 updated LXD readiness checks to enforce the minimum required version and revised ensure_lxd_is_ready() error messages (implementation 9fcd49b).

Resolution: closed resolved

Reason: PR #71 was merged by merge commit fc50a1b; implementation commit 9fcd49b added the minimum-version check and revised readiness error messages.

Evaluated by: mr-cal-bot

No scores available.

Issue body

While here, do a minor update for the error messages in ensure_lxd_is_ready() to meet the CLI guidelines. 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 #71 updated LXD readiness checks to enforce the minimum required version and revised ensure_lxd_is_ready() error messages (implementation 9fcd49b).
qwen/qwen3.6-35b-a3b — closed resolved Merged PR #71 in craft-providers to account for minimum required LXD version in checks and update error messages in ensure_lxd_is_ready().

Update history

Date Change
created

Related issues

Issue Project State Summary Similarity
#474 charmcraft: perform LXD checks before use (CRAFT-420) charmcraft merged Merged #474: added ensure_lxd_is_ready() before LXD use, including permission validation and clearer errors. Merge commit: 1dfb54b1.
74%
#69 lxd: ensure user is permitted to use lxd (CRAFT-374) craft-providers merged Merged PR #69 added LXD permission and readiness checks, resolving CRAFT-374 in commit 5478f63.
74%
#63 lxd: check if LXD is initialized prior to launch (CRAFT-348) craft-providers merged Merged LXD initialization preflight validation for CRAFT-348, adding is_initialized(), launch-time LXDError handling, and unit/integration tests in commit b00bea0.
74%
#535 Make LXD version check more robust craft-providers closed Merged in PR #810 (commit 99d694c), replacing unstable LXC version parsing with the LXD /1.0 API environment.server_version field.
71%
#34 lxd: introduce LXD for interacting with `lxd` command-line craft-providers merged 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.
70%