← Back to issue list

spread tests: enable LXD build provider tests

View original Github issue

Metadata

Project
snapcraft
Number
#2631
Type
pull request
State
merged
Author
sergiusens
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #2631 enabled LXD build-provider spread tests and reused early-prepared LXD setup, reducing repeated setup and image-download overhead.

Resolution: closed resolved

Reason: PR #2631 was merged into master by commit edfe970ac, enabling the test suite while moving common declarations from task.yaml to spread.yaml.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Move LXD setup to early prepare and do not remove at the end of each test to avoid the setup overhead (including image download). Additionally, move the common declarations from task.yaml to spread.yaml for the build-providers suite. Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com> - [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `./runtests.sh static`? - [ ] Have you successfully run `./runtests.sh tests/unit`? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #2631 enabled LXD build-provider spread tests and reused early-prepared LXD setup, reducing repeated setup and image-download overhead.
stealth/space-bunny-alpha — closed resolved Merged PR #2631 enabled LXD build-provider spread tests, reusing early-prepared LXD to reduce setup and image-download overhead.
qwen/qwen3.6-35b-a3b — — 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%.
qwen3.6-35b-a3b-mtp-q6 — — Merged into master. Optimized LXD build provider tests by moving setup to early prepare and consolidating declarations in spread.yaml to reduce overhead. Slightly increased overall code coverage.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#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)
87%
#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.
84%
#269 tests: add initial spread testing fetch-service merged Merged PR #269 (commit 8fe0091) added initial Spread integration tests to fetch-service: spread.yaml plus snap-config, basic-inspection, update-config and upgrade-revert suites; all 10 lxd-vm tasks passed, resolving #226.
79%
#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.
79%
#485 test(spread): test uc-prepare in lxd imagecraft merged Merged PR adding spread tests for uc-prepare on LXD across core22, core24, and core26 base images.
78%
#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:.
78%
#1133 test(spread): make LXD refresh in prepare more reliable craft-application closed Closed unmerged; its LXD auto-refresh race fix was superseded by merged PR #1170 (commit fea3e835), which waits for active snap refreshes during spread prepare.
78%
#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.
77%
#487 test(spread): run the simple pack test in all providers imagecraft merged Merged PR #487 adds variants of the simple pack test to run across LXD, multipass, and host/destructive providers, fixing #480.
77%
#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.
76%