fix(ci): restore verification after RustFS bind mounts - #61
Merged
Conversation
Summary: - run ICC setup before Docker creates local bind-mount directories - upgrade checkout from v4 to v7 and its Node 24 runtime Rationale: - Docker otherwise creates .local as root and blocks ICC setup in CI - checkout v7 removes the deprecated Node 20 action runtime warning Tests: - fresh-worktree setup with ICC creation before Docker startup - direnv exec . env SAKEKEEP_E2E_PORT=3200 bun run verify AI-Assisted: true AI-Agent: codex AI-Model: openai/gpt-5.6-sol
This was referenced Aug 18, 2026
Summary: - merge the current main branch into the CI workflow fix - retain main's ICC ordering comment and checkout v7 from the PR Rationale: - resolve the workflow overlap without changing either intended behavior - keep the PR diff limited to the checkout runtime upgrade Tests: - conflict-marker and staged whitespace checks - verified final PR diff against current main Related to #61 AI-Assisted: true AI-Agent: codex AI-Model: openai/gpt-5.6-sol
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
CI starts Docker before creating the ICC profile. The RustFS bind mounts therefore create
.localas root, and the followingsetup:iccstep fails withEACCES. The workflow also usesactions/checkout@v4, whose Node 20 action runtime now produces a deprecation warning.Solution
Create the ICC profile before starting Docker, matching the project setup order, and upgrade checkout to v7 with its Node 24 runtime.
Validation
.localThis unblocks the required checks on #57.
Built with Codex in T3 Code using gpt-5.6-sol.