tests: move the ppa test trigger to lxd
Metadata
Current evaluation
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.
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`?
-----
I spent a long time trying to understand why this was failing. On docker, everything was hard, so I moved it to lxd to debug in there. In the end, the error was correct, the PPA failed to build yesterday's snap. But now I have this in lxd, which will be useful if I have to debug an error again.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged changes moving the PPA test trigger from Docker to LXD. The original Docker failure was a valid PPA build error, but the LXD environment will streamline future debugging and test maintenance. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged changes moving the PPA test trigger from Docker to LXD to streamline debugging. The original failure was a legitimate PPA build issue, but the LXD environment provides better isolation for diagnosing similar test failures. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #262 tests: install docker after lxd | rockcraft | merged | Merged test changes to install Docker after LXD, resolving order-dependent failures and a race condition. A separate PR will address the remaining flaky spread test. | |
| #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. | |
| #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. | |
| #2631 spread tests: enable LXD build provider tests | snapcraft | merged | Merged into master. Enabled LXD build provider tests by relocating setup to the early prepare phase to reduce overhead and consolidating declarations from task.yaml to spread.yaml. Approved, passed CI, and increased coverage by 0.01%. | |
| #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. | |
| #1807 tests: run test_cleanbuild in LXD on Travis | snapcraft | merged | 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. | |
| #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. | |
| #302 test(spread): switch to testing in destructive mode | imagecraft | merged | Switched spread tests to destructive mode and merged. This addresses an upcoming losetup requirement that temporarily breaks the LXD backend. Approved by two reviewers with passing CI checks. | |
| #4239 spread: improve resilience of lxd-type-base test | snapcraft | merged | Merged via squash to improve resilience of the lxd-type-base test, fixing frequent lxc stop timeouts on Google runners. Approved by reviewers with passing CI checks. | |
| #73 tests: remove duplicated unit tests | rockcraft | merged | Merged removal of duplicated unit tests from test_provider.py and test_providers.py, artifacts from a prior LXD and Multipass test consolidation. Approved, passed CI, and reduced two files by 60 lines. |