Fix spread tests
Metadata
Current evaluation
Fixed and merged as commit cd43de1, updating 11 files (+49/-38); all listed CI checks, including spread, passed.
Resolution: closed resolved
Reason: The pull request was merged as commit cd43de1 after approval by two reviewers. All listed CI checks passed, including spread on Ubuntu 20.04, 22.04, and 24.04, with none failing or pending.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [x] Have you followed the guidelines for contributing?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `make lint && make test`?
---
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Fixed and merged as commit cd43de1, updating 11 files (+49/-38); all listed CI checks, including spread, passed. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged spread-test fixes across 11 files; approved by two reviewers, with all listed CI checks passing despite two unresolved review comments. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #127 to fix Spread tests across 11 files (+49/−38), with two approvals and all reported CI checks passing. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged after receiving approval from two reviewers and passing all CI checks. The changes fix spread tests across 11 files (+49/-38 lines). | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged by maintainer upils to fix failing spread tests. Changes were integrated into the main branch without additional review or requested modifications. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1287 tests: minor cleanups on the spread tests | snapcraft | merged | Merged into master as commit 1cf0f8c9f "tests: minor cleanups on the spread tests (#1287)" — a test-only tidy-up touching 4 spread test files (+16/-6), reviewed and approved by fgimenez, with coverage unchanged at 95.27%. | |
| #4529 Spread update | snapcraft | merged | Merged #4529, updating Spread test configuration across three files (+36/−40) after approval by two maintainers. | |
| #13 test(spread): add more spread tests | snapcraft-rocks | merged | Merged PR #13 added Spread tests for architectures, Chisel slices, craftctl, package repositories, and patchelf (+206/-5 across 15 files). | |
| #3663 tests: update spread url | snapcraft | merged | Merged via commit bcea80568, updating two Spread test tarball URLs (+2/-2 lines). | |
| #96 fix(test): fix spread tests on Fedora | snapcraft-rocks | merged | Merged PR #96 fixed Fedora 39 Spread tests by using Docker's default branch after the core18 branch closed; all CI checks passed. | |
| #573 ci: run spread tests on 'spread-installed' runners | rockcraft | merged | Merged PR #573 as commit c8ff5823, updating CI to run Spread tests on runners labeled "spread-installed." | |
| #926 test: address a couple of non-flaky spread failures | rockcraft | merged | Merged PR #926 updated three spread-test configuration files to address persistent failures; all CI checks passed and two reviewers approved. | |
| #3662 tests: update spread url | snapcraft | merged | Merged PR #3662 updated Spread tarball URLs in two test files via commit 507959d70. | |
| #291 Refactor spread testing to use spread extensions | fetch-service | closed | Fixed by merged PR #299 (commit 44d9906), which refactored spread tests to generate spread.yaml and use the new spread extension framework. | |
| #1083 ci: fix spread test missing PATH | charmcraft | merged | Merged PR #1083 as commit 13c96b83, fixing the missing PATH in Spread tests; configure and remove tests then passed. |