tests: run test_cleanbuild in LXD on Travis
Metadata
Current evaluation
Merged following CI errors and reviewer feedback. Adjusted to run test_cleanbuild in LXD on Travis, reverting a problematic commit to isolate the cleanbuild fix.
Suggested action: —
No scores available.
Issue body
- [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`?
-----
Attempt to sort out test_cleanbuild being skipped in #1803 when run as a snap
Fixes: [LP: #1738210](https://bugs.launchpad.net/snapcraft/+bug/1738210)
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged following CI errors and reviewer feedback. Adjusted to run test_cleanbuild in LXD on Travis, reverting a problematic commit to isolate the cleanbuild fix. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged after reverting a commit that broke test_container_builds. The final implementation configures Travis to run test_cleanbuild in LXD, resolving the skipped test issue when executed as a snap. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged after scope adjustment to run test_cleanbuild in LXD on Travis. Initial attempts hit LXD confinement and path errors. Maintainer advised reverting a problematic commit, finalizing the change to exclusively target cleanbuild tests. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #327 Clean up the lxc building in travis | snapcraft | merged | 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. | |
| #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. | |
| #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. | |
| #2149 travis: use LXD from 3.0 track | snapcraft | merged | Merged Travis CI configuration update switching to the LXD 3.0 track and removing the obsolete testbr0 setup. This resolves build failures caused by recent LXD networking changes. Approved by reviewer and passed CI checks. | |
| #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. | |
| #1411 tests: build the snapcraft snap in travis tests | snapcraft | merged | Merged after approval. Added Travis CI configuration to build the snapcraft snap, requiring dependency PR #1415. Reviewer recommended validating changes with spread using the lxd backend. | |
| #2983 tests: add LXD spread test | snapcraft | merged | Merged after reviewer approval and passing CI checks. Added LXD spread test to verify LXD builds and installs. Modified two files with 29 added lines. | |
| #1758 tests: move the ppa test trigger to lxd | snapcraft | merged | Merged changes moving the PPA test trigger from Docker to LXD to streamline future debugging. The original failure was valid, but the environment shift improves test maintainability. Approved by reviewers and passed all CI checks. | |
| #1087 Don't wait for lxd networking in cleanbuild test | snapcraft | merged | Merged to master. Removed an unnecessary LXD networking wait in the cleanbuild test, cutting runtime by five seconds. Approved by two reviewers, passed CI, and integrated successfully. |