Clean up the lxc building in travis
Metadata
Current evaluation
Merged as snapcraft#327 (merge 4f342a6b9, commit 026100eed "travis: clean up the lxd script"), simplifying the Travis LXC build after a rebase; kyrofa approved. Its follow-up bugfix landed in #340.
Resolution: closed resolved
Reason: PR #327 was merged into master via merge commit 4f342a6b9 (squashed as026100eed "travis: clean up the lxd script"), reducing the Travis LXD/LXC script dependencies (+6/-4 in .travis.yml). It was approved by kyrofa after the requested rebase, and the related bug the author reported separately was fixed in merged PR #340 (LP bug 1549880).
Evaluated by: mr-cal-bot
Issue body
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged as snapcraft#327 (merge 4f342a6b9, commit 026100eed "travis: clean up the lxd script"), simplifying the Travis LXC build after a rebase; kyrofa approved. Its follow-up bugfix landed in #340. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #327 via 4f342a6b9 (commit 026100eed), simplifying Travis LXC setup by removing privileged mode, narrowing permissions, deduplicating dependencies and dropping unused coverage packages. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged after rebase and maintainer approval, simplifying the Travis LXC build configuration and reducing its dependencies. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged after rebase and approval. The changes cleaned up the LXC build configuration in Travis CI, reducing dependencies. Originally an experiment, the commit was refined and integrated into the codebase. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged cleanup of LXC build configuration in Travis CI. Originally an experiment to reduce dependencies, the branch was rebased and approved by maintainers before integration. |
Update history
No update history recorded yet.
Related work
-
Related To:
snapcraft#340
(confidence 88%)
Follow-up bugfix PR opened by the same author (come-maiz/elopio) for the Travis LXC network-wait command after #327; explicitly referenced in the PR comments ('Let me report a bug').
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1807 tests: run test_cleanbuild in LXD on Travis | snapcraft | merged | Merged PR #1807 (commit 70df69b9a) by kalikiana, making test_cleanbuild run in LXD on Travis so it is no longer skipped when tests run as a snap; fixes LP #1738210. Reviewer asked to revert the last commit keeping LXD for cleanbuild only. | |
| #1411 tests: build the snapcraft snap in travis tests | snapcraft | merged | Merged #1411 via commit 92c1e43b1, adding Travis coverage that builds the Snapcraft snap after LXD test infrastructure landed in #1415. | |
| #1653685 clean up travis.yaml | snapcraft (launchpad) | closed | Resolved by merged snapcore/snapcraft PR #1012 (commit b66bdebe9), which cleaned up Travis and changed container build phases to use docker exec. | |
| #1415 tests: run the tests in travis using LXD containers | snapcraft | merged | Merged in snapcraft#1415 via commit 8c9a38cde, updating Travis CI to run the test suite in LXD containers. | |
| #1738210 container tests broken in travis when running from a snap | snapcraft (launchpad) | closed | Travis cleanbuild tests were fixed by running them in LXD, merged in snapcore/snapcraft#1807 (commit 70df69b9a). | |
| #1703344 snapcraft cleanbuild exits with a traceback when lxd is not installed | snapcraft (launchpad) | closed | Fixed cleanbuild’s traceback when LXD is absent by handling the missing lxc executable and reporting a clear error. Merged in snapcraft#1400. |