← 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 PR #1677 removed unused LXD installation from CI; commits 2b568926 and 5282b96f record the change.

Resolution: closed resolved

Reason: PR #1677 was merged after approval, with commits 2b568926 and 5282b96f disabling unused LXD in CI. The maintainer confirmed the remaining test failures were caused by an unrelated Launchpad issue.

Evaluated by: mr-cal-bot

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
stealth/space-bunny-alpha — closed resolved Merged PR #1677 removed unused LXD installation from CI; commits 2b568926 and 5282b96f record the change.
stealth/space-bunny-alpha — closed resolved Merged the CI change removing unused LXD installation. The maintainer attributed the failing test-other-plugins jobs to an unrelated Launchpad issue.
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
#1978 ci: switch to stable lxd and unconfined containers snapcraft merged Merged PR #1978 as commit c2d5a4f68, switching CI to stable LXD and using unconfined containers.
75%
#1422 ci: remove the old script to run with lxc snapcraft merged Merged PR #1422 as commit 766daabb9, removing the obsolete LXC CI script and deleting 56 lines across two files.
75%
#280 ci: use setup-lxd craft-providers closed CI change to adopt setup-lxd was closed unmerged without review or CI activity.
72%
#3112 spread: remove dead code for lxd setup and add debug prints snapcraft merged Merged PR #3112 in commit cf2fec383, removing unreachable LXD setup code and adding debug output for intermittent LXD snap-install failures.
72%
#2557 ci: remove dependency on LXD from travis tests snapcraft merged Merged PR #2557 as commit c541fac1d, removing LXD setup and container execution from Travis tests to reduce networking errors.
71%
#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.
71%