test: fix spread tests
Metadata
Current evaluation
Merged PR #6350 as a18c7f485, fixing all non-store Spread tests and OSV scanning, including pinning Black 26.3.1 to avoid patchelf failures.
Resolution: closed resolved
Reason: PR #6350 was approved and merged as commit a18c7f485. Its non-store Spread and OSV-scanner checks passed, with the Black pin avoiding patchelf silent failures and segfaults.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Fixes all the non-store spread tests and the OSV scanner.
---
- [x] I've followed the [contribution guidelines](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md).
- [x] I've signed the [CLA](http://www.ubuntu.com/legal/contributors/).
- [ ] I've successfully run `make lint && make test`.
- [ ] I've added or updated any relevant documentation.
- [ ] In documents I changed, I [added a meta description](https://canonical-starflow.readthedocs-hosted.com/how-to/add-a-page-meta-description/) if one was missing.
- [ ] I've updated the relevant release notes.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #6350 as a18c7f485, fixing all non-store Spread tests and OSV scanning, including pinning Black 26.3.1 to avoid patchelf failures. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #6350 repaired non-store spread tests and OSV-scanner setup, pinning black 26.3.1 to avoid patchelf silent failures and segfaults. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged after fixing non-store spread tests and the OSV scanner. Pinned black to 26.3.1 to resolve patchelf segfaults and silent failures. Approved by two reviewers and passed CI checks. | |
| qwen3.6-35b-a3b-mtp-q6 |
Staleness:
0
Complexity:
10
Confidence:
90
|
needs review | Fixes several spread test failures. Opened by a maintainer, currently awaiting CI validation and initial review. |
Update history
| Date | Change |
|---|---|
| updated |
Related work
-
Related To:
canonical/snapcraft#6347
(confidence 90%)
Merged PR #6347, commit 5bb3aff7d, separately fixed OSV scanning for documentation inputs.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2417 fix: fix a series of breaking spread tests | charmcraft | merged | Merged PR #2417 at commit 3bee888e, fixing multiple spread-test failures and unblocking the core24 migration; three listed tests remained broken but non-blocking. | |
| #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). | |
| #1366 tests(spread): make release test less flaky | charmcraft | merged | Merged #1366 at commit 05b18a03, reducing flakiness in the Spread release test and resolving #1362. | |
| #97 fix(test): fix spread tests on Fedora | snapcraft-rocks | merged | Merged PR #97 switched Spread tests from the closed core18 Docker branch to the default branch, fixing them on Fedora 39. | |
| #54 Add basic spread tests | imagecraft | merged | PR #54 merged as e5eb5f4, adding amd64 and arm64 spread tests for 20.04/22.04/24.04 (commits 819603b, 514868e). Later fixed by #127 (cd43de1) for 20.04 failures. | |
| #1460 fix(tests): make store spread tests less flaky | charmcraft | merged | Merged PR #1460 as fd5bd6ba, making store spread tests less timing-sensitive and fixing flaky upload/release checks in #1429. | |
| #3242 tests: migrate legacy classic patchelf tests to spread | snapcraft | merged | Merged PR #3242 as e76a07747, migrating legacy classic patchelf tests to Spread and removing 186 net lines across 13 files. | |
| #1290 test(spread): clean up store tests | charmcraft | merged | Merged in commit 34ab4bdd, cleaning up store spread tests by deleting 28 lines across four files; the remaining library-test failure was fixed separately in #1292. | |
| #2763 fix(tests/spread): fix reactive spread tests | charmcraft | merged | Merged #2763 as 6e775f4a: split reactive spread coverage by legacy/Resolute, enabled binary wheels, and removed obsolete build flags. | |
| #6202 test(spread): create core26 spread test suite | snapcraft | merged | Merged PR #6202 as a9a5b4750, adding core26 Spread tests and CI coverage while generalizing common tests for core22, core24, and core26. |