test(spread): test sector writing in all providers
Metadata
Current evaluation
Merged PR adding spread tests to verify sector writing across all providers, addressing CRAFT-5332.
Resolution: closed resolved
Reason: The pull request was merged by the maintainer, introducing spread tests to validate sector writing functionality across all supported providers, fulfilling the requirements of CRAFT-5332.
Evaluated by: mr-cal-bot
No scores available.
Issue body
CRAFT-5332
---
- [ ] I've followed the [contribution guidelines](https://github.com/canonical/imagecraft/blob/main/CONTRIBUTING.md).
- [ ] 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 |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | closed resolved | Merged PR adding spread tests to verify sector writing across all providers, addressing CRAFT-5332. | |
| ~deepseek/deepseek-pro-latest |
Impact:
30
Quick Win:
17
Complexity:
25
Confidence:
70
Actionability:
75
|
needs review | Adds a spread test for writing data to sectors outside partitions and runs it across all providers; CI mostly green, spread jobs pending. |
Update history
| Date | Change |
|---|---|
| merged | |
| updated | |
| updated | |
| updated | |
| updated | |
| created |
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #3877 tests: enable provider spread tests for core22 | snapcraft | merged | Merged as commit 7612a3a87: renamed the spread test dir build-providers to providers and moved legacy-only tests into providers/legacy, enabling provider spread tests on core22 (CRAFT-1266). Prerequisite for #3875. | |
| #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). | |
| #16 tests: add spread support | rockcraft | merged | Merged PR #16 added Spread support for CRAFT-865 via commit 3aeba2b3 (merge commit 5bf2680a). | |
| #1325 test(spread): consolidate basic init/build tests | charmcraft | merged | Merged PR #1325 as commit 90e25caf, consolidating basic Spread init/build tests for CRAFT-2127 and reducing duplicated setup across four files. | |
| #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. | |
| #762 tests(spread): add a test for the fetch-service | rockcraft | merged | Merged PR #762 in commit f4cff41c, adding 51 lines of spread-test coverage for the fetch service across two files. | |
| #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. |