← Back to issue list

ci: reduce amount of artifacts to upload for spread

View original Github issue

Metadata

Project
snapcraft
Number
#3476
Type
pull request
State
merged
Author
sergiusens
Labels
—
Created
Updated
Closed

Current evaluation

Merged #3476 (f6fb10359), reducing Spread uploads to tests/ while preserving reset behavior; local runs now require the tested snap in that directory.

Resolution: closed resolved

Reason: PR #3476 was approved and merged as commit f6fb10359, limiting Spread artifacts to tests/ and keeping required files committed for reset behavior.

Evaluated by: mr-cal-bot

No scores available.

Issue body

This reduces the payload to upload for spread to just the tests directory. Everything is git commited to keep current reset behavior. This work implies that when running locally, the snap to be tested needs be copied into the test directory. Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com> - [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `./runtests.sh static`? - [ ] Have you successfully run `./runtests.sh tests/unit`? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged #3476 (f6fb10359), reducing Spread uploads to tests/ while preserving reset behavior; local runs now require the tested snap in that directory.
stealth/space-bunny-alpha — closed resolved Merged #3476 (f6fb10359), limiting Spread upload artifacts to tests/ and reducing CI payloads; local runs now require the tested snap in that directory.
stealth/space-bunny-alpha — closed resolved Merged #3476 (f6fb10359), limiting Spread upload artifacts to tests/ to reduce CI payloads; local runs must copy snaps into that directory.
qwen/qwen3.6-35b-a3b — — Merged a CI optimization that reduces spread upload artifacts to the tests directory, decreasing payload size. Local execution now requires copying the snap to the test directory. Approved and passed all checks.
qwen3.6-35b-a3b-mtp-q6 — — Merged. CI spread artifact uploads now only include the tests directory to reduce payload size. Local testing requires copying the snap into the test directory.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#432 ci: speed up spread testing workflow fetch-service merged Merged PR #432 (commit 9eec284), removing snap-test's dependency on snap-build so spread tests start sooner while snap-build still produces downloadable artifacts.
78%
#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.
72%
#4380 ci: download snap to tests dir snapcraft merged Merged PR #4380 as commit 0de1f0e1d, fixing scheduled Spread tests to download the snap into the tests/ directory.
72%
#107 spread: restrict the files we package rockcraft merged Merged PR #107 in commit bd878a14 restricted Spread packaging to the test/ and tools/ directories, reducing local overhead from virtual environments and other workspace files.
71%
#3026 spread tests: add core20 and cleanup systems snapcraft merged Merged #3026 in commit d006134d4, adding core20 spread tests, simplifying setup to install only the Snap, and cleaning up test systems.
71%
#6408 Re-organize spread tests snapcraft open Maintainer-filed, triaged enhancement to reorganize/dedupe Snapcraft's spread tests and drop low-value ones for faster, less flaky CI; no implementation plan or PR yet, tracked as SNAPCRAFT-1377.
70%
#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.
70%