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

ci: isolate RC release index cleanup #12

Closed
Breadway wants to merge 1 commit from fix/mon-11-rc-cache-cleanup into main
pull from: fix/mon-11-rc-cache-cleanup
merge into: Breadway:main
Conversation 5 Commits 1 Files changed 1 +4 -5
Breadway commented 2026-10-01 15:36:50 +08:00
Owner
Copy link

Removes the shared wildcard cleanup from the beta index workflow, uses a prefix that older workflows do not match, and cleans only this job's clone on exit.

Verified workflow YAML and shell syntax. Simulated success and failure index generation; the job clone was cleaned and a sibling legacy-prefix checkout remained intact.

Closes #11

Removes the shared wildcard cleanup from the beta index workflow, uses a prefix that older workflows do not match, and cleans only this job's clone on exit. Verified workflow YAML and shell syntax. Simulated success and failure index generation; the job clone was cleaned and a sibling legacy-prefix checkout remained intact. Closes #11
fix: scope breadmon RC cache cleanup
All checks were successful
check / check (push) Successful in 5m45s
Details
2b9f1cfe68
Breadway commented 2026-10-01 15:38:02 +08:00
Author
Owner
Copy link

Synced from #12, closing MON-11. The isolated success/failure acceptance and diff checks pass. Forgejo CI is running, so this PR card remains TODO until CI and independent source review complete.

Synced from https://git.breadway.dev/Breadway/breadmon/pulls/12, closing MON-11. The isolated success/failure acceptance and diff checks pass. Forgejo CI is running, so this PR card remains TODO until CI and independent source review complete.
Breadway commented 2026-10-01 16:22:01 +08:00
Author
Owner
Copy link

PR #12 passes local YAML and shell checks plus success/failure cleanup simulations with a legacy-prefix sibling preserved. Forgejo CI was queued at last check. Moving to REVIEW for independent review; if CI fails, return it to TODO with the failure recorded.

PR #12 passes local YAML and shell checks plus success/failure cleanup simulations with a legacy-prefix sibling preserved. Forgejo CI was queued at last check. Moving to REVIEW for independent review; if CI fails, return it to TODO with the failure recorded.
Breadway commented 2026-10-01 16:25:55 +08:00
Author
Owner
Copy link

Forgejo CI passed in 5m45s on head 2b9f1cfe68900ed8dbc21d111b7c71420e2679fb. The acceptance harness passed for success and failure, preserving the active legacy-prefix sibling. Moving this linked PR card to REVIEW.

Forgejo CI passed in 5m45s on head `2b9f1cfe68900ed8dbc21d111b7c71420e2679fb`. The acceptance harness passed for success and failure, preserving the active legacy-prefix sibling. Moving this linked PR card to REVIEW.
Breadway commented 2026-10-02 12:05:06 +08:00
Author
Owner
Copy link

Not merged: obsolete. main has deprecated breadmon and retired its release jobs (#13), so .forgejo/workflows/rc-release.yml no longer exists and this branch conflicts (modify/delete). No code change needed; suggest closing this PR and the MON-11 card.

Not merged: obsolete. `main` has deprecated breadmon and retired its release jobs (#13), so `.forgejo/workflows/rc-release.yml` no longer exists and this branch conflicts (modify/delete). No code change needed; suggest closing this PR and the MON-11 card.
Breadway commented 2026-10-02 17:10:20 +08:00
Author
Owner
Copy link

Closing as obsolete. MON-13 (breadmon #13) retired breadmon's release jobs, and main now keeps only .forgejo/workflows/check.yml: rc-release.yml and release.yml are gone, and no workflow on main references bread-ecosystem-ci temp paths any more. The shared-cleanup race this PR fixed is no longer reachable in this repository, so the breadmon child of ECO-56 resolves by removal instead of by this PR.

The branch was based on the pre-deprecation main, where the race still existed; the change itself (distinct /tmp/bread-index-* prefix plus an EXIT trap, deliberately outside the legacy wildcard's match set) was the strongest of the per-repo variants reviewed, so nothing is lost by retiring it with the workflows it targeted.

Closing as obsolete. MON-13 (breadmon #13) retired breadmon's release jobs, and main now keeps only `.forgejo/workflows/check.yml`: `rc-release.yml` and `release.yml` are gone, and no workflow on main references `bread-ecosystem-ci` temp paths any more. The shared-cleanup race this PR fixed is no longer reachable in this repository, so the breadmon child of ECO-56 resolves by removal instead of by this PR. The branch was based on the pre-deprecation main, where the race still existed; the change itself (distinct `/tmp/bread-index-*` prefix plus an EXIT trap, deliberately outside the legacy wildcard's match set) was the strongest of the per-repo variants reviewed, so nothing is lost by retiring it with the workflows it targeted.
Breadway closed this pull request 2026-10-02 17:10:21 +08:00
All checks were successful
check / check (push) Successful in 5m45s
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
ci
No labels
ci
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/breadmon!12
No description provided.