← Back to issue list

ci: always discard spread runners

View original Github issue

Metadata

Project
charmcraft
Number
#1519
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Merged #1519 (commit fdf24383), adding post-job cleanup to the Spread workflow so temporary runners are always discarded after CI jobs.

Resolution: closed resolved

Reason: PR #1519 was approved and merged as commit fdf24383. It added the copied cleanup configuration to .github/workflows/spread.yml, ensuring Spread runners are discarded after jobs.

Evaluated by: mr-cal-bot

No scores available.

Issue body

A direct copy of: https://github.com/snapcore/snapcraft/blob/main/.github/workflows/spread.yml#L117

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged #1519 (commit fdf24383), adding post-job cleanup to the Spread workflow so temporary runners are always discarded after CI jobs.
stealth/space-bunny-alpha — closed resolved Merged PR #1519, copying Snapcraft's always-discard spread-runner cleanup into Charmcraft's CI workflow (+16 lines).
qwen/qwen3.6-35b-a3b — — Merged a CI workflow update that permanently discards spread runners by mirroring the snapcraft repository configuration. Approved by three reviewers, the change adds 16 lines to a single workflow file.
qwen3.6-35b-a3b-mtp-q6 — — Merged a CI workflow update to always discard spread runners. The change directly copies a configuration line from the snapcraft repository, standardizing runner cleanup across projects.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#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.
76%
#482 ci: always discard spread workers rockcraft merged PR #482 by lengau merged: added an always-run "Discard spread workers" cleanup step (spread -discard -reuse-pid) to .github/workflows/spread.yaml, copied from snapcraft's spread.yml.
75%
#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."
73%
#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.
73%
#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.
72%
#115 ci: always discard spread workers in build-rock tests (core22-8) snapcraft-rocks merged PR #115 merged (+8/-0): adds an always-run 'Discard spread workers' step to the spread-tests job in .github/workflows/build-rock.yaml for core22-8, so cancelled/crashed jobs no longer leak spread GCP VMs (SNAPCRAFT-1368).
71%
#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.
70%
#1526 ci: reduce spread 18.04 runners and increase 22.04 runners charmcraft merged PR #1526 merged as commit ccdd6015, rebalancing .github/workflows/spread.yaml to run fewer Ubuntu 18.04 spread runners and more 22.04 runners, matching where most tests now live.
70%
#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.
70%