Skip to content

Prevent build-and-test container name collisions in XPU PR workflow - #239

Draft
jmunetong with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-build-and-test-job-again
Draft

Prevent build-and-test container name collisions in XPU PR workflow#239
jmunetong with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-build-and-test-job-again

Conversation

Copilot AI commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

PR Test (XPU) was intermittently failing in build-and-test before tests started because docker run reused a static container name on a shared self-hosted runner. This change isolates container identity per workflow run/attempt to remove cross-run conflicts.

  • Root cause

    • Run container used hardcoded --name ci_sglang_xpu, which conflicts when a prior container with that name still exists (e.g., retries/overlap on the same runner).
  • Workflow update

    • Added job-level env var:
      • CI_CONTAINER_NAME: ci_sglang_xpu_${{ github.run_id }}_${{ github.run_attempt }}
    • Replaced all direct ci_sglang_xpu references with ${CI_CONTAINER_NAME}.
  • Scope of replacement

    • docker run --name ...
    • all docker exec ...
    • all docker cp ...
    • cleanup docker rm -f ...
env:
  CI_CONTAINER_NAME: ci_sglang_xpu_${{ github.run_id }}_${{ github.run_attempt }}

# examples
docker run -dt --name "${CI_CONTAINER_NAME}" ...
docker exec "${CI_CONTAINER_NAME}" ...
docker cp "${CI_CONTAINER_NAME}":/path/in/container ./local.log
docker rm -f "${CI_CONTAINER_NAME}" || true

Copilot AI changed the title [WIP] Fix failing GitHub Actions job build-and-test Prevent build-and-test container name collisions in XPU PR workflow Jun 8, 2026
Copilot AI requested a review from jmunetong June 8, 2026 19:39
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