Breadway/breadarr
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

ci: stop dev-release temp-dir cleanup from racing concurrent jobs #7

Merged
Breadway merged 1 commit from fix/eco38-tmp-race-glob into main 2026-09-29 07:54:03 +08:00
Conversation 0 Commits 1 Files changed 1 +1 -2
Breadway commented 2026-09-28 22:26:45 +08:00
Owner
Copy link

Same fix as breadbar, dev-release.yml only (check.yml already used the safe per-workflow mktemp prefix and never had the glob). Root cause confirmed from the forgejo-runner journal on Hestia: the clone bread-ecosystem (main) step's rm -rf /tmp/bread-ecosystem-ci-* glob deletes a concurrently running job's own bread-ecosystem clone on this shared self-hosted host executor.

Part of ECO-38.

Same fix as breadbar, dev-release.yml only (check.yml already used the safe per-workflow mktemp prefix and never had the glob). Root cause confirmed from the forgejo-runner journal on Hestia: the clone bread-ecosystem (main) step's rm -rf /tmp/bread-ecosystem-ci-* glob deletes a concurrently running job's own bread-ecosystem clone on this shared self-hosted host executor. Part of ECO-38.
ci: stop dev-release/check temp-dir cleanup from racing concurrent jobs
Some checks failed
check / check (push) Failing after 1m31s
Details
check / check (pull_request) Failing after 1s
Details
2664565fc1 
The clone bread-ecosystem (main) step used a preemptive
rm -rf /tmp/bread-ecosystem-ci-* before mktemp. /tmp is a single shared
namespace on the self-hosted host executor, and with runner capacity 2
plus several repos dev-release/check builds firing close together, one
job's rm -rf glob would delete another concurrently running job's
bread-ecosystem clone mid-use. That surfaced as "No such file or
directory" at regenerate dev index.json, or a git error partway through
clone bread-ecosystem itself, depending on timing. Root cause confirmed
from the forgejo-runner journal on Hestia.

Drop the preemptive rm -rf glob and give each workflow kind its own
mktemp prefix (bread-ecosystem-dev-release-, bread-ecosystem-check-),
matching the pattern already in use in breadhelp/breadarr check.yml,
which never hit this race. mktemp's own uniqueness is enough; a
crashed job leaves at most one small leftover directory instead of
nuking a sibling job's live one.

Part of ECO-38.
Breadway deleted branch fix/eco38-tmp-race-glob 2026-09-29 07:54:03 +08:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Breadway/breadarr!7
No description provided.