← Back to issue list

tests: change default spread provider to lxd outside of travis

View original Github issue

Metadata

Project
snapcraft
Number
#2682
Type
pull request
State
merged
Author
cjp256
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #2682 (commit 8b41444a3) refactored runtests.sh, defaulted Spread to LXD outside Travis, and configured Travis to use google:.

Resolution: closed resolved

Reason: PR #2682 was approved and merged as commit 8b41444a3, implementing the test-runner cleanup and provider configuration changes.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Minor refactoring and cleanup for runtests script to make it easier to follow and remove some of the cruft. Modify travis config to specify google: for spread. Signed-off-by: Chris Patterson <chris.patterson@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 #2682 (commit 8b41444a3) refactored runtests.sh, defaulted Spread to LXD outside Travis, and configured Travis to use google:.
stealth/space-bunny-alpha — closed resolved Merged PR #2682 (commit 8b41444a3) by cjp256: cleaned up runtests.sh, made LXD the default Spread provider outside Travis, and set .travis.yml to use google:.
stealth/space-bunny-alpha — closed resolved Merged PR #2682 (commit 8b41444a3) refactored runtests.sh, made LXD the default Spread provider outside Travis, and configured Travis to use google:.
qwen/qwen3.6-35b-a3b — — Merged after approval and CI. Refactored runtests script for clarity and updated Travis config to use google spread provider, removing legacy code and simplifying non-Travis test execution.
qwen3.6-35b-a3b-mtp-q6 — — Merged refactoring of the runtests script and updated Travis config to set google as the default spread provider outside Travis. Code coverage upload failed, but changes were successfully integrated.

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)
80%
#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.
79%
#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.
78%
#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.
77%
#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.
74%
#4239 spread: improve resilience of lxd-type-base test snapcraft merged PR #4239 was squash-merged as ab2c13576, making the lxd-type-base spread test more resilient to frequent lxc stop timeouts on Google runners.
74%
#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.
73%
#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.
72%
#2149 travis: use LXD from 3.0 track snapcraft merged Merged PR #2149 via commit 0b84e215c, moving Travis to LXD's 3.0 track and removing obsolete testbr0 setup after LXD networking changes broke CI.
72%
#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.
72%