← Back to issue list

ci(spread): install spread

View original Github issue

Metadata

Project
charmcraft
Number
#1453
Type
pull request
State
closed
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

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.

Resolution: closed superseded

Reason: The author closed the PR on 2024-01-04 with "This isn't necessary, see #1463". Issue #1463 was resolved by merged PR #1510 (commit 9c98b399), which limited the spread workflow to [self-hosted, spread-enabled] runners rather than installing spread per job.

Evaluated by: mr-cal-bot

No scores available.

Issue body

No description provided.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed superseded 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.
stealth/space-bunny-alpha — closed superseded Closed unmerged as unnecessary; #1510 (9c98b399) instead restricted Spread CI to spread-installed runners, resolving #1463.
stealth/space-bunny-alpha — closed superseded Install-spread CI PR was closed as unnecessary; canonical/charmcraft#1510 (9c98b399) instead restricted spread tests to self-hosted runners labeled spread-installed.
stealth/space-bunny-alpha — closed superseded Closed unmerged as unnecessary. The alternative in #1463 shipped via PR #1510 (9c98b399), limiting spread tests to runners where spread is installed.
qwen/qwen3.6-35b-a3b — — The proposed CI spread installation was closed as unnecessary. The author deemed the change redundant and referenced issue #1463 as the correct approach.
qwen3.6-35b-a3b-mtp-q6 — — Rejected and closed by the author as unnecessary. Superseded by #1463, which fulfills the spread installation requirement.

Update history

No update history recorded yet.

Related work

  • Superseded By: charmcraft#1510 (confidence 92%)

    PR #1510 (9c98b399) implemented the spread-related CI change from issue #1463 using runner labels instead of installing spread.

  • Duplicate Of: charmcraft#1463 (confidence 85%)

    Maintainer comment pointed the PR at issue #1463, which tracked the spread CI problem and was closed by #1510.

  • Related To: charmcraft#1519 (confidence 40%)

    Follow-up commit fdf24383 adding spread worker discard steps to the same spread workflow.

Related issues

Issue Project State Summary Similarity
#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.
79%
#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."
78%
#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.
78%
#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.
78%
#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.
76%
#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.
76%
#490 ci(spread): run spread on all systems imagecraft closed PR #490 was superseded by PR #495 (commit 4c5701d), which merged the same change to run spread tests on all systems per backend.
75%
#495 ci(spread): run spread on all systems imagecraft merged Merged PR #495 as commit 4c5701d, replacing per-system spread matrix limits with backend-wide runs controlled by spread.yaml.
74%
#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.
73%
#1097 spread: make spread-large run all charm tests charmcraft closed PR #1097 was closed unmerged after 1,176 days of inactivity; the spread-large workflow change never landed.
72%