ci: scope RC release cleanup to its own checkout #50

New issue

Closed

opened 2026-10-01 15:20:23 +08:00 by Breadway · 3 comments

Breadway commented

2026-10-01 15:20:23 +08:00

Owner

Child of ECO-28. The current .forgejo/workflows/rc-release.yml removes /tmp/bread-ecosystem-ci-* before creating its own mktemp checkout. That wildcard can delete an active checkout used by a concurrent workflow. Remove the wildcard and retain exact invocation-specific EXIT cleanup. Verify the embedded shell for successful and failing index generation while a sibling active checkout remains intact.

Child of ECO-28. The current `.forgejo/workflows/rc-release.yml` removes `/tmp/bread-ecosystem-ci-*` before creating its own mktemp checkout. That wildcard can delete an active checkout used by a concurrent workflow. Remove the wildcard and retain exact invocation-specific EXIT cleanup. Verify the embedded shell for successful and failing index generation while a sibling active checkout remains intact.

Breadway added the

label

2026-10-01 15:20:24 +08:00

Breadway referenced this issue from a pull request that will close it,

2026-10-01 15:23:09 +08:00

ci: scope RC release cleanup to its own checkout #51

Breadway commented

2026-10-01 15:27:32 +08:00

Author

Owner

PR: #51

The beta index step now uses a prefix older wildcard cleaners cannot match and removes only its own clone via EXIT trap. YAML parsing and the embedded step passed simulated success/failure checks while a legacy-prefix sibling checkout remained intact. Forgejo CI is running; keeping the issue in DOING until it succeeds.

PR: https://git.breadway.dev/Breadway/breadbar/pulls/51 The beta index step now uses a prefix older wildcard cleaners cannot match and removes only its own clone via EXIT trap. YAML parsing and the embedded step passed simulated success/failure checks while a legacy-prefix sibling checkout remained intact. Forgejo CI is running; keeping the issue in DOING until it succeeds.

Breadway commented

2026-10-01 15:33:44 +08:00

Author

Owner

Forgejo CI for PR #51 completed successfully in 5m46s. The PR is ready for independent source review and remains in DOING pending that review.

Breadway commented

2026-10-01 15:37:46 +08:00

Author

Owner

Reviewed PR #51 against BAR-50:

The beta index workflow no longer performs broad shared-prefix cleanup.
It clones into a unique /tmp/bread-index-XXXXXX directory and an EXIT trap removes only that job's directory: .forgejo/workflows/rc-release.yml:53-56.
The clone path is outside the legacy /tmp/bread-ecosystem-ci-* prefix, so old cleanup jobs cannot remove it.

The PR reports simulated index success and failure checks that preserve a sibling checkout. CI is successful at head 886a439944d6550a3bc001d9a1a0f32dcf81453d, and Forgejo reports the PR mergeable.

Reviewed PR #51 against BAR-50: 1. The beta index workflow no longer performs broad shared-prefix cleanup. 2. It clones into a unique `/tmp/bread-index-XXXXXX` directory and an EXIT trap removes only that job's directory: `.forgejo/workflows/rc-release.yml:53-56`. 3. The clone path is outside the legacy `/tmp/bread-ecosystem-ci-*` prefix, so old cleanup jobs cannot remove it. The PR reports simulated index success and failure checks that preserve a sibling checkout. CI is successful at head `886a439944d6550a3bc001d9a1a0f32dcf81453d`, and Forgejo reports the PR mergeable.

Breadway closed this issue

2026-10-01 15:38:11 +08:00