ci: make manual spread label more cooperative
Metadata
Current evaluation
Merged in #5880 via commit ed2710863, improving manual Spread CI so labeled commits and dispatches run while label removal stops subsequent runs.
Resolution: closed resolved
Reason: PR #5880 was merged as commit ed2710863. It updated the workflow to support label-triggered runs, later commits, and manual dispatch while preventing runs after label removal.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [ ] Have you followed the [guidelines for contributing](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md)?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `make lint`?
- [ ] Have you successfully run `make test`?
---
I confirmed via manual testing that the following worked:
- Adding the label triggers the workflow
- Pushing a commit after the label is already added will run it
- Manual dispatch of the workflow works
- Pushing a commit after removing the label will not run it
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged in #5880 via commit ed2710863, improving manual Spread CI so labeled commits and dispatches run while label removal stops subsequent runs. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in #5880 (ed2710863): made manual spread-label CI cooperative, allowing labeled commits and manual dispatch while preventing runs after label removal. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged a CI workflow update improving manual spread label handling. The change ensures the workflow triggers on label addition, runs on subsequent commits, supports manual dispatch, and respects removal. Approved and merged. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a CI workflow update that improves handling of the PR: Run Manual Spread label. Verified that adding or removing the label correctly triggers or stops the workflow, and manual dispatch functions as expected. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1034 ci: make manual spread more cooperative | rockcraft | merged | Manual Spread CI was made cooperative in merged commit e4800e1d, running for labeled PRs, schedules, or manual dispatch while gating jobs through a predicate. | |
| #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. | |
| #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. | |
| #5434 ci: add check for each manual spread test job | snapcraft | merged | Merged PR #5434 in commit c6a67da45, adding checks so manual Spread test jobs no longer appear skipped when their label is absent. | |
| #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. | |
| #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. | |
| #524 ci: adjust tags for spread runs | rockcraft | closed | Unmerged CI tag proposal was superseded by #573, where commit c8ff5823 moved Spread tests to self-hosted runners labeled spread-installed. | |
| #455 ci(spread): fix runner label ('spread-enabled') | imagecraft | closed | Merged as commit 8d05774, updating the Spread CI runner label from spread-installed to spread-enabled. |