spread: improve resilience of lxd-type-base test
Metadata
Current evaluation
PR #4239 was squash-merged as ab2c13576, making the lxd-type-base spread test more resilient to frequent lxc stop timeouts on Google runners.
Resolution: closed resolved
Reason: PR #4239 was squash-merged into main as commit ab2c13576, resolving the LXD timeout flakiness tracked by issue #4236.
Evaluated by: mr-cal-bot
No scores available.
Issue body
This test often times out on `lxc stop` on Google runners.
- [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] Have you successfully run `make lint`?
- [x] Have you successfully run `pytest tests/unit`?
-----
#4236
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | PR #4239 was squash-merged as ab2c13576, making the lxd-type-base spread test more resilient to frequent lxc stop timeouts on Google runners. | |
| stealth/space-bunny-alpha | — | closed resolved | Squash-merged #4239 in commit ab2c13576, improving the lxd-type-base spread test and reducing frequent lxc stop timeouts on Google runners. | |
| stealth/space-bunny-alpha | — | closed resolved | PR #4239 was squash-merged as ab2c13576, improving the lxd-type-base spread test’s resilience to lxc stop timeouts on Google runners. | |
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Squash merged into main. The change improves resilience of the lxd-type-base test, preventing frequent timeouts on Google runners. |
Update history
No update history recorded yet.
Related work
-
Related To:
snapcore/snapcraft#4236
(confidence 100%)
Linked flaky-test issue closed by this change.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #4236 lxd-type-base spread test is flaky due to lxd timeouts | snapcraft | closed | Flaky lxd-type-base spread test (LXC stop 'context deadline exceeded') fixed by PR #4239, squash-merged as commit ab2c13576 'spread: improve resilience of lxd-type-base test'. | |
| #4140 Flaky spread tests | snapcraft | closed | Flaky LXD spread-test timeouts were addressed by PR #4141 and commit 4b20c8435, which improved the resilience of the core18 lxd-type-base test. | |
| #5856 fix(spread): pack with lxd for core20 base tests | snapcraft | merged | Merged PR #5856 (commit 66b9ae9f4) updated Spread core20-base tests to pack with LXD. | |
| #2983 tests: add LXD spread test | snapcraft | merged | Merged PR #2983 as 4b8522a14, adding spread coverage that builds the LXD package with LXD and destructive/host providers; installation checks were deferred. | |
| #2631 spread tests: enable LXD build provider tests | snapcraft | merged | Merged PR #2631 enabled LXD build-provider spread tests and reused early-prepared LXD setup, reducing repeated setup and image-download overhead. | |
| #2682 tests: change default spread provider to lxd outside of travis | snapcraft | merged | Merged PR #2682 (commit 8b41444a3) refactored runtests.sh, defaulted Spread to LXD outside Travis, and configured Travis to use google:. | |
| #187 tests: improve lxd launcher tests for base instances | craft-providers | merged | Merged in b0df595: expanded LXD base-instance tests, mocked _is_valid(), added lxc.info() integration coverage, and clarified an error message. | |
| #467 test(spread): use LXD provider for spread tests | imagecraft | closed | PR closed without merge; spread LXD provider changes were implemented across follow-up PRs (#483, #485, #486, #487, #488, #495) | |
| #694 tests(spread): use LXD from `5.21/candidate` | rockcraft | merged | Merged PR #694 in commit 90bc9b34, pinning spread tests to LXD 5.21/candidate to restore main and strengthen LTS regression coverage. | |
| #262 tests: install docker after lxd | rockcraft | merged | Merged PR #262 as 64c5c01b, changing spread-test setup to install Docker after LXD and avoid order-dependent failures. |