← Back to issue list

ci: switch to stable lxd and unconfined containers

View original Github issue

Metadata

Project
snapcraft
Number
#1978
Type
pull request
State
merged
Author
sergiusens
Labels
Created
Updated
Closed

Current evaluation

Merged CI configuration update switching to stable LXD and unconfined containers. Approved by two reviewers with a two-file, five-line change.

Suggested action:

No scores available.

Issue body

Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com> - [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)? - [x] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)? - [x] Have you successfully run `./runtests.sh static`? - [x] Have you successfully run `./runtests.sh unit`? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged CI configuration update switching to stable LXD and unconfined containers. Approved by two reviewers with a two-file, five-line change.
qwen3.6-35b-a3b-mtp-q6 CI configuration updated to use stable LXD and unconfined containers. The change was merged by maintainer sergiusens without additional comments.
qwen3.6-35b-a3b-mtp-q6 Merged CI configuration updates switching to stable LXD and unconfined containers. The maintainer passed tests, signed off the change, and the pull request was successfully integrated.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1677 ci: don't install lxd craft-parts merged 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.
78%
#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.
74%
#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.
72%
#1403 lxd: Only remove container if one exists snapcraft merged Merged following two reviewer approvals and passing CI. The update resolves duplicate exceptions during failed LXD container creation by adding a conditional check before attempting container removal.
72%
#1627 lxd: split container classes into different files snapcraft merged Merged refactoring splitting LXD container classes from lxd.py into separate modules. Improves organization, moves Cleanbuilder methods, de-duplicates env vars, and fixes mocking imports. No functional changes. Approved by two reviewers and passed CI.
72%
#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.
72%
#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.
71%
#6376 docs: update LXD docs URL snapcraft merged Merged documentation update redirecting LXD docs to canonical.com/lxd/docs/. Approved by one reviewer and passed most CI checks despite a failing OSV-scanner. Changed two files with minimal edits.
71%
#1644 lxd: fix the push in container builds snapcraft merged Merged pull request fixing LXD container push builds. Approved by one reviewer, the change refactored test cases for better coverage and updated build logic. Review comments highlighted the need for Travis integration and broader testing beyond unit tests.
71%
#1415 tests: run the tests in travis using LXD containers snapcraft merged Merged changes to run Travis CI tests using LXD containers. The update modifies five files, adding 166 lines and removing 12, after receiving reviewer approval.
70%