← Back to issue list

ci: don't install lxd

View original Github issue

Metadata

Project
craft-parts
Number
#1677
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Merged a CI configuration update removing the unused LXD installation. Approved by three reviewers, the change was integrated despite unrelated test failures stemming from a Launchpad issue.

Suggested action: —

No scores available.

Issue body

We're not using it anyway - [x] Have you followed the guidelines for contributing? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `make lint && make test`? - [ ] Have you added an entry to the changelog (`docs/reference/changelog.rst`)? ---

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — Merged a CI configuration update removing the unused LXD installation. Approved by three reviewers, the change was integrated despite unrelated test failures stemming from a Launchpad issue.
qwen/qwen3.6-35b-a3b
Staleness: 0
Complexity: 5
Confidence: 95
needs review Removes unused lxd installation from CI configuration. Approved by 3 reviewers with zero unresolved comments. CI shows unrelated failures in slow test suites.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 5
Confidence: 90
needs review Removes unused LXD installation from CI configuration. Approved by two reviewers with zero unresolved comments. CI shows unrelated test failures acknowledged by the author.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 5
Confidence: 90
needs review Removes unused LXD installation from CI configuration. Approved by one reviewer. CI shows unrelated test failures noted by author.
qwen/qwen3.6-35b-a3b
Staleness: 0
Complexity: 5
Confidence: 95
needs review Removes unused LXD installation from CI configuration. PR is approved by one reviewer, all CI checks are passing, and it is ready for final review and merge.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 5
Confidence: 90
needs review Removes unused LXD installation from CI configuration. PR is fresh, all CI checks passing, awaiting maintainer review.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 5
Confidence: 90
needs review Removes unused lxd package installation from CI configuration. The PR is recent, CI checks are passing, and it awaits maintainer review.

Update history

Date Change
updated
updated
updated
updated
updated
updated
created

Related issues

Issue Project State Summary Similarity
#461 ci: use the default version of lxd for integration tests craft-application merged Merged after two reviewer approvals and passing CI. The change updates the CI configuration to use the default LXD version for integration tests, removing six lines of explicit version pinning across one file.
78%
#1978 ci: switch to stable lxd and unconfined containers snapcraft merged Merged CI configuration update switching to stable LXD and unconfined containers. Approved by two reviewers with a two-file, five-line change.
78%
#1422 ci: remove the old script to run with lxc snapcraft merged Merged a change that removes an outdated LXC CI script. The update deleted 56 lines across two files and was approved by two reviewers before integration.
75%
#1421 ci: wait for apt/dpkg lock when setting lxd up snapcraft merged Merged a CI fix that waits for the apt/dpkg lock during LXD setup. Resolves permission denied errors during apt-get update by preventing concurrent package manager access. Approved by two reviewers and passed all checks.
75%
#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.
74%
#1074 ci: fix lxd image name too long charmcraft merged Merged a fix for a CI configuration error where the LXD image name exceeded the maximum length. Approved by one reviewer, the change modified one file (+3/-2 lines) and was successfully integrated.
74%
#1419 ci: add tests that run in LXD containers craft-parts open Adds LXD container-based CI tests for interim/unreleased Ubuntu releases. 255 days old, no reviews or comments, all 16 LXD test jobs failing. Project has since migrated to starflow CI with use-lxd explicitly disabled.
74%
#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.
73%
#343 lxd should be a suggestion, not a direct dep snapcraft merged Merged changes converting lxd from a direct dependency to a suggested package. Approved by reviewer and passed CI checks.
73%
#2557 ci: remove dependency on LXD from travis tests snapcraft merged Merged into master, removing the LXD dependency from Travis CI tests to reduce networking errors. Approved, passed CI checks, and modified two files without impacting code coverage.
72%