Skip to content

[CICD] Migrate CI runners to FlagCICD runner set #51

Open
Darryl233 wants to merge 14 commits intoflagos-ai:mainfrom
Darryl233:test_arc
Open

[CICD] Migrate CI runners to FlagCICD runner set #51
Darryl233 wants to merge 14 commits intoflagos-ai:mainfrom
Darryl233:test_arc

Conversation

@Darryl233
Copy link
Copy Markdown
Collaborator

@Darryl233 Darryl233 commented Mar 20, 2026

ci: Migrate CI runners to FlagCICD runner set and fix git workspace mount in containers

Changes across 5 workflow files (qa-l0-pytorch-wheel, qa-l0-te-cpp-unittest-pytorch-lint, qa-l1-te-cpp-pytorch-tests, qa-l3-te-pytorch-fa-versions-test, te-plugin-tests):

  • Runner migration: Replace runs-on: [self-hosted, Linux, X64, nvidia, gpu-8] with [dx4-nvidia-8gpu-v1] to adopt the new compute platform.
  • Workspace volume mount: Add -v /home/runner/_work/${{ github.run_id }}:${{ github.workspace }} to container options so the host workspace is correctly mounted inside the container.
  • Fix git safe.directory: Add a git config --global --add safe.directory '*' step before checkout to resolve git failures caused by directory ownership mismatches inside the container.

@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ Darryl233
❌ liyuzhuo


liyuzhuo seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@Darryl233 Darryl233 changed the title [wip] Test arc [CICD] Migrate CI runners to FlagCICD runner set Mar 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants