ci: add check for each manual spread test job
Metadata
Current evaluation
Merged PR #5434 in commit c6a67da45, adding checks so manual Spread test jobs no longer appear skipped when their label is absent.
Resolution: closed resolved
Reason: PR #5434 was approved and merged as commit c6a67da45, adding the required checks to the manual Spread test workflow.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [x] Have you followed the [guidelines for contributing](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `make lint`?
- [x] Have you successfully run `make test`?
---
We thought we could get away without these extra lines, but we need them.
These extra checks needs to be present or else these jobs show as "skipped" when the manual spread label isn't added.
For example:
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #5434 in commit c6a67da45, adding checks so manual Spread test jobs no longer appear skipped when their label is absent. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #5434 in commit c6a67da45, adding six checks so manual spread-test jobs do not appear skipped when their manual label is absent. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #5434 (commit c6a67da45) added 6 lines to .github/workflows/spread.yml so each manual spread-test job runs at least one step and no longer reports as "skipped" without the manual label. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #5434 in commit c6a67da45, adding checks so manual spread-test jobs are not reported as skipped when the label is absent. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #5434 (c6a67da45), adding explicit checks to manual spread test jobs so they no longer appear as skipped when the manual spread label is absent. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #5434, adding explicit checks so manual Spread test jobs no longer appear skipped when the manual label is absent. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #5434 (commit c6a67da45), adding explicit checks to each manual Spread test job so unlabeled runs do not appear skipped. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged CI configuration changes adding explicit checks for manual spread test jobs. This prevents jobs from incorrectly showing as skipped when the manual label is absent. Approved by reviewers and passed all CI checks. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged changes add CI checks for each manual spread test job. These checks prevent jobs from incorrectly showing as skipped when the manual spread label is absent, ensuring accurate test execution tracking. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #5880 ci: make manual spread label more cooperative | snapcraft | merged | Merged in #5880 via commit ed2710863, improving manual Spread CI so labeled commits and dispatches run while label removal stops subsequent runs. | |
| #5714 ci: update label for manual spread tests | snapcraft | merged | Merged #5714 (951647c5a), updating the manual Spread job to use the new Terraform-managed runner label. | |
| #2487 ci: add manual spread label, match other crafts | charmcraft | merged | PR #2487 by bepri merged as commit e0c8f7a8, adding a 'PR: Run Manual Spread' label trigger to .github/workflows/spread-manual.yaml so large/flaky spread tasks run on label, schedule or dispatch like other crafts. | |
| #6335 ci: always discard spread workers in manual tests (main) | snapcraft | merged | Merged #6335 in commit b36b3c325, adding always-run Spread worker cleanup to manual CI jobs so cancelled or crashed tests do not leak GCP VMs. | |
| #395 ci: add manual spread workflow with per-suite labels | imagecraft | closed | PR #395 (CI: add spread-manual.yaml with per-suite manual labels, IMAGECRAFT-176) was closed unmerged after ~43 days idle: review still pending, 0 reviewers, 2 unresolved comments, 'predicate' and 'Canonical CLA' checks failing. | |
| #996 fix(ci): restore the manual spread label | rockcraft | merged | Merged PR #996 as commit 8846d864, restoring the manual spread label needed by #993 despite a non-blocking snap-tests failure. | |
| #573 ci: run spread tests on 'spread-installed' runners | rockcraft | merged | Merged PR #573 as commit c8ff5823, updating CI to run Spread tests on runners labeled "spread-installed." |