tests: move the ppa test trigger to lxd
Metadata
Current evaluation
Merged snapcore/snapcraft#1758 (commit 3524514e9), moving the PPA test trigger from Docker to LXD after debugging showed the PPA correctly failed to build the prior day's snap.
Resolution: closed resolved
Reason: PR #1758 was merged and recorded as commit 3524514e9. It replaced Docker with LXD for the PPA test trigger, making future PPA build failures easier to debug.
Evaluated by: mr-cal-bot
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 |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged snapcore/snapcraft#1758 (commit 3524514e9), moving the PPA test trigger from Docker to LXD after debugging showed the PPA correctly failed to build the prior day's snap. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in #1758 via 3524514e9, replacing the PPA test trigger's Docker environment with LXD. Debugging confirmed the prior failure was a valid PPA build failure. | |
| 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. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. | |
| #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. | |
| #483 test(spread): run snap-preseed tests in lxd | imagecraft | merged | Merged PR #483 to update snap-preseed plugin tests to run in LXD across all systems, addressing CRAFT-5332. | |
| #202 Run the tests in a xenial LXC. | snapcraft | merged | PR #202 by come-maiz moved snapcraft's test suite into a Xenial LXC, fixing LP bug #1532213. Merged in Jan 2016 (commit 4e8829dc3) after +1 from sergiusens. | |
| #3957 tests: retry installing lxd snap | snapcraft | merged | PR #3957 merged as 57cd1d97b, adding snapd-testing-tools retries for LXD installation in spread tests to work around intermittent snapd bug 1870201. | |
| #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). | |
| #3725 tests: use lxd in clean provider test | snapcraft | merged | Merged PR #3725 (commit 61aaff6fc) into snapcraft/7.0, enabling LXD in provider-cleaning unit tests so they match destructive-mode test runs. Approved by one maintainer reviewer. | |
| #1364 lxd: Inject snapcraft and core snaps into the container | snapcraft | merged | Merged PR #1364 as 99f5598ac, injecting the host's Snapcraft and core snaps into LXD containers so builds no longer use a mismatched Xenial archive version. | |
| #3341 repo: move apt ppa helpers into apt_ppa module | snapcraft | merged | Merged in canonical/snapcraft#3341 as commit 68ec9d3c9, extracting apt PPA helpers from _deb and expanding focused unit-test coverage. | |
| #1849 tests: add snap not found tests | snapcraft | merged | Merged missing-snap test coverage; snapcraft PR #1849 landed as commit 54cf2bd9e. |