Skip to content

Fix AMD CI suite names in pr-test-amd.yml#16710

Closed
alisonshao wants to merge 2 commits intomainfrom
fix-amd-ci-suite-names
Closed

Fix AMD CI suite names in pr-test-amd.yml#16710
alisonshao wants to merge 2 commits intomainfrom
fix-amd-ci-suite-names

Conversation

@alisonshao
Copy link
Copy Markdown
Collaborator

@alisonshao alisonshao commented Jan 8, 2026

Summary

  • Fix unit-test-backend-1-gpu-amd-mi35x: Changed from per-commit-amd-mi35x to per-commit-amd
  • Fix unit-test-backend-2-gpu-amd: Changed from per-commit-2-gpu-amd to use new test runner with stage-b-test-large-2-gpu-amd

Test plan

  • CI should pass for AMD jobs that were previously failing with "invalid choice" errors

Example failure: https://github.com/sgl-project/sglang/actions/runs/20805568675/job/59760722246?pr=16458

Fix two broken CI jobs that were using non-existent suite names:

1. unit-test-backend-1-gpu-amd-mi35x: Changed from per-commit-amd-mi35x
   (doesn't exist) to per-commit-amd (the standard 1-GPU AMD suite)

2. unit-test-backend-2-gpu-amd: Changed from per-commit-2-gpu-amd
   (doesn't exist) to use the new test runner with
   stage-b-test-large-2-gpu-amd suite

These suite names don't exist in either test/srt/run_suite.py (old runner)
or test/run_suite.py (new runner), causing CI failures.
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@github-actions github-actions bot added the amd label Jan 8, 2026
@alisonshao
Copy link
Copy Markdown
Collaborator Author

/rerun-stage unit-test-backend-1-gpu-amd

@bingxche
Copy link
Copy Markdown
Collaborator

bingxche commented Jan 8, 2026

Hi @alisonshao, thanks for your attention. We are working on this problem #16675.

@alisonshao alisonshao closed this Jan 8, 2026
@zhyncs zhyncs deleted the fix-amd-ci-suite-names branch January 8, 2026 21:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants