← Back to issue list

ci: adjust tags for spread runs

View original Github issue

Metadata

Project
rockcraft
Number
#524
Type
pull request
State
closed
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Unmerged CI tag proposal was superseded by #573, where commit c8ff5823 moved Spread tests to self-hosted runners labeled spread-installed.

Resolution: closed superseded

Reason: Merged PR #573 contains commit c8ff5823, "ci: run spread tests on 'spread-installed' runners," implementing the tag change described in #524. No #524 merge commit appears in the workflow history, so the landed follow-up replaced the closed proposal.

Evaluated by: mr-cal-bot

No scores available.

Issue body

This ensures we can use any machine with the spread-installed tag. Necessary for upcoming changes to Canonical's self-hosted runners. Spread will fail until a repo admin adjusts the runner tags. - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed superseded Unmerged CI tag proposal was superseded by #573, where commit c8ff5823 moved Spread tests to self-hosted runners labeled spread-installed.
stealth/space-bunny-alpha — closed resolved Merged PR #524 to use the spread-installed runner tag, enabling Canonical self-hosted runners; repository admins still needed to configure that tag.
stealth/space-bunny-alpha — closed resolved Merged CI tag adjustments for spread runs in PR #524, enabling use of Canonical self-hosted runners; a repository admin must still configure the runner tags.
qwen/qwen3.6-35b-a3b — — Merged a configuration update adjusting spread runner tags to support Canonical's self-hosted runners. Approved by two reviewers with passing CI. The change modifies tags across two files to prevent execution failures.
qwen3.6-35b-a3b-mtp-q6 — — Closed without merging to adjust CI spread runner tags for Canonical's self-hosted infrastructure, pending admin configuration.

Update history

No update history recorded yet.

Related work

  • Superseded By: canonical/rockcraft#573 (confidence 90%)

    Commit c8ff5823 implemented the equivalent spread-installed runner-tag migration.

Related issues

Issue Project State Summary Similarity
#4715 ci: adjust tags for spread runs snapcraft merged Merged PR #4715 updated Spread CI runner tags to allow any machine with the spread-installed tag, preparing Canonical's self-hosted runners.
91%
#1628 ci: use the correct tags for large spread runs charmcraft merged Merged PR #1628 (commit ad16fb2f), correcting runner tags for large Spread test runs with a one-line CI workflow change.
84%
#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."
82%
#571 ci: use self-hosted runners for ci workflows rockcraft closed Closed unmerged and superseded by PR #573, whose commit c8ff5823 moved spread tests to self-hosted runners labeled spread-enabled.
80%
#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.
79%
#1453 ci(spread): install spread charmcraft closed PR #1453 (add spread install step to spread CI) was closed unmerged by lengau as unnecessary; #1463 was instead solved by PR #1510 (9c98b399), which restricted spread jobs to self-hosted runners labelled spread-enabled.
76%
#1463 Limit spread tests to runners that have spread charmcraft closed Spread CI was restricted to self-hosted runners labeled spread-installed by merged PR #1510 (commit 9c98b399), fixing #1463.
76%
#1510 ci(spread): limit github action runners to spread-installed label charmcraft merged Merged PR #1510 (commit 9c98b399) restricted Spread CI to self-hosted runners labeled spread-installed, fixing #1463.
75%
#127 Fix spread tests imagecraft merged Fixed and merged as commit cd43de1, updating 11 files (+49/-38); all listed CI checks, including spread, passed.
73%
#2126 ci(docs): spread test documentation charmcraft merged Merged PR #2126 in commit f1b73a6e, adding CI for documentation spread tests and adapting spread tasks for Google-hosted runners.
72%