spread: restrict the files we package
Metadata
Current evaluation
Merged to restrict spread packaging to test/ and tools/ directories only. This reduces local run overhead by excluding hundreds of megabytes of unnecessary files. Approved by one reviewer, passed CI, and modified one file.
Suggested action: —
No scores available.
Issue body
The spread run only needs the 'test/' and 'tools/' dir. This change has little impact on CI runs because they start from a 'clean' checkout, but for local runs the behavior of adding everything can add hundreds of MB of local 'junk' (specially from the virtual environment). So this speeds up local runs a bit.
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged to restrict spread packaging to test/ and tools/ directories only. This reduces local run overhead by excluding hundreds of megabytes of unnecessary files. Approved by one reviewer, passed CI, and modified one file. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a change to restrict spread packaging to only the test/ and tools/ directories. This reduces local run overhead by excluding hundreds of megabytes of unnecessary files, speeding up development workflows. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged to restrict packaged files to only the test/ and tools/ directories. This reduces local junk from virtual environments and speeds up local spread runs with minimal CI impact. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1088 spread: only install docker for store tests | charmcraft | merged | Merged. Restricts Docker installation to store tests only via a +9/-5 line diff in the spread configuration. Approved by one reviewer with no comments. | |
| #62 spread: add a "big" spread test | rockcraft | merged | Merged a reusable spread test framework to house multiple checks without repeated setup. The initial test validates bug #44 regarding directory permissions. Approved by one reviewer and passed CI checks. | |
| #3491 spread: run cross compile tests on amd64 only | snapcraft | merged | Merged into master. Configures spread to run cross-compile tests exclusively on amd64. Approved by one reviewer, passed CI, and applied a minimal three-line change with no coverage impact. | |
| #1502 test(spread): disable centos spread test | charmcraft | merged | Merged a change disabling the CentOS spread test. The update modifies one file with three added lines, received approval from two reviewers, and was integrated without CI checks or comments. | |
| #573 ci: run spread tests on 'spread-installed' runners | rockcraft | merged | Merged a two-file, four-line CI configuration change to run spread tests on spread-installed runners. Approved by three reviewers and integrated. | |
| #13 test(spread): add more spread tests | snapcraft-rocks | merged | Successfully merged spread tests covering architectures, chisel slices, craftctl, package-repositories, and patchelf. Approved by two reviewers across 15 files (+206/-5 lines). | |
| #519 spread: reorganize spread tests | rockcraft | merged | Merged via squash-merge. Reorganizes spread tests to optimize CI resource usage by consolidating most tests into a jammy-based rockcraft suite and adding a foreign suite for cross-system testing. Changes affect 73 files. | |
| #1503 test(spread): disable centos spread test | charmcraft | merged | Merged to disable the CentOS spread test. Approved by two reviewers, the change modifies one file with three lines added and is now part of the main branch. | |
| #530 spread: don't cut dotnet slices unnecessarily | rockcraft | merged | Merged a one-line change replacing fragile dotnet slices with coreutils_bins in spread tests to avoid instability from ongoing time_t changes. Approved by three reviewers and passed CI. | |
| #2216 spread tests: keep sources local | snapcraft | merged | Merged PR #2216 to replace remote FTP sources in spread tests with in-tree tar archives, preventing test failures from server downtime. Approved by one reviewer, passed CI checks, and slightly increased code coverage. |