ci(arm): arm-cargo-host concurrency + full src/ purge (DAK-7483)#135
Closed
ferhimedamine wants to merge 1 commit into
Closed
ci(arm): arm-cargo-host concurrency + full src/ purge (DAK-7483)#135ferhimedamine wants to merge 1 commit into
ferhimedamine wants to merge 1 commit into
Conversation
Three ARM runners (dakera-arm, cli-arm, mcp-arm) share the same physical host and ~/.cargo/registry/src/. Concurrent cargo builds race on C++ source extraction, leaving partial directories (aws-lc-sys, librocksdb-sys) that cause "No such file or directory" compile errors on subsequent jobs. Fix: - arm-cargo-host concurrency group on all ARM jobs (check/clippy/test/ integration-test): serializes builds org-wide across all three runners - Purge step expanded: rm -rf ~/.cargo/registry/src/ (was aws-lc-sys-* only) so all partially-extracted C++ crates are force-re-extracted from .crate cache Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Contributor
Author
|
Superseded by DAK-7484 infra fix. The root cause (shared /root/.cargo across all ARM runners) has been fixed at the systemd level: each of the 13 runners now has an isolated CARGO_HOME=/root/actions-runner-{repo}-arm/.cargo via their service unit EnvironmentFile. This eliminates the race at source — no rm -rf needed. This PR's full-src/ purge was also actively causing corruption in concurrent jobs. Closing in favor of a clean workflow PR (DAK-7484). |
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.
Summary
Fixes concurrent ARM runner C++ source extraction failures (DAK-7483). The three ARM runners (
actions-runner-cli-arm,actions-runner-mcp-arm,actions-runner-dakera-arm) share the same physical host and~/.cargo/registry/src/. Under concurrent builds, C++ crates likeaws-lc-sysget partially extracted, causing "No such file or directory" errors.Changes:
arm-cargo-hostconcurrency group on all ARM jobs (check/clippy/test/integration-test) — serializes org-wide, same as PR#788 in dakera reporm -rf ~/.cargo/registry/src/instead ofrm -rf ~/.cargo/registry/src/aws-lc-sys-*— forces full re-extraction of all C++ crates from .crate cache filesThis unblocks CTO merging the 8 dependabot PRs via DAK-7474.
Test plan
🤖 Generated with Claude Code