test: address a couple of non-flaky spread failures
Metadata
Current evaluation
Merged PR #926 updated three spread-test configuration files to address persistent failures; all CI checks passed and two reviewers approved.
Resolution: closed resolved
Reason: PR #926 was merged with a +3/-9 change across three files. It passed every listed CI check and resolved the targeted spread-test failures.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [ ] Have you followed the guidelines for contributing?
- [ ] 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 | Merged PR #926 updated three spread-test configuration files to address persistent failures; all CI checks passed and two reviewers approved. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #926 updated three spread-test configuration files, addressing persistent non-flaky test failures; all CI checks passed and two reviewers approved. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged canonical/rockcraft#926, updating test configurations across three files to fix persistent spread-test failures; all CI checks passed and two reviewers approved. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged after two approvals and passing all CI checks. Resolves non-flaky spread test failures by updating test configurations across three files, eliminating consistent failures. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged by maintainer tigarmo to fix non-flaky spread failures in the test suite. Changes were integrated into the main branch without additional discussion. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #534 test(spread): workaround for flaky test | fetch-service | merged | Merged PR #534 (commit 4f89bb7) worked around #533 by skipping the flaky snap names-query assertion while retaining the other spread-test checks. | |
| #3662 tests: update spread url | snapcraft | merged | Merged PR #3662 updated Spread tarball URLs in two test files via commit 507959d70. | |
| #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. | |
| #1600 test(spread): move flaky parallel-build test to weeklies | charmcraft | merged | Merged in commit a1d285c7, moving the flaky parallel-build Spread test from regular CI to weeklies as a workaround for #1332. | |
| #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). | |
| #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%. | |
| #415 test: fix typo in spread extension | fetch-service | closed | PR #415 was closed after approval was retracted; its typo fix was incorporated into merged PR #417 via commit 8363f15. | |
| #127 Fix spread tests | imagecraft | merged | Fixed and merged as commit cd43de1, updating 11 files (+49/-38); all listed CI checks, including spread, passed. | |
| #1962 Spread test the documentation examples | charmcraft | open | Documentation examples are now covered by dedicated Spread tests and CI through merged PR #2126; the still-open issue should be closed. | |
| #4529 Spread update | snapcraft | merged | Merged #4529, updating Spread test configuration across three files (+36/−40) after approval by two maintainers. |