Skip to content

ci: migrate RL tests to test/registered/rl/#16417

Merged
Kangyan-Zhou merged 13 commits intomainfrom
ci/migrate-rl-tests
Jan 6, 2026
Merged

ci: migrate RL tests to test/registered/rl/#16417
Kangyan-Zhou merged 13 commits intomainfrom
ci/migrate-rl-tests

Conversation

@alisonshao
Copy link
Collaborator

Summary

1-GPU tests (stage-b-test-small-1-gpu)

Test CUDA Time AMD Time
test_fp32_lm_head.py 9s 15s
test_update_weights_from_tensor.py 195s N/A

2-GPU tests (stage-b-test-large-2-gpu / per-commit-2-gpu-amd)

Test CUDA Time AMD Time
test_update_weights_from_distributed.py 103s 103s

4-GPU tests (per-commit-4-gpu)

Test CUDA Time
test_return_routed_experts.py 300s

Note: The disabled test_update_weights_from_disk.py remains in test/srt/rl/ as it's currently disabled in CI (see #13998).

Test plan

  • Verify CI runs the migrated tests in the correct suites

Part of #13808

Migrate RL (reinforcement learning) tests from test/srt/rl/ to the new
test/registered/ CI system with proper registration decorators:

1-GPU tests (stage-b-test-small-1-gpu):
- test_fp32_lm_head.py (CUDA=9s, AMD=15s)
- test_update_weights_from_tensor.py (CUDA=195s)

2-GPU tests (stage-b-test-large-2-gpu):
- test_update_weights_from_distributed.py (CUDA=103s, AMD=103s)

4-GPU tests (per-commit-4-gpu):
- test_return_routed_experts.py (CUDA=300s)

The disabled test_update_weights_from_disk.py remains in test/srt/rl/
as it's currently disabled in CI.
@gemini-code-assist
Copy link
Contributor

Warning

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

@Kangyan-Zhou
Copy link
Collaborator

/tag-and-rerun-ci

@github-actions github-actions bot added the run-ci label Jan 5, 2026
@alisonshao

This comment was marked as resolved.

@alisonshao
Copy link
Collaborator Author

alisonshao commented Jan 5, 2026

@Kangyan-Zhou
Copy link
Collaborator

@alisonshao
Copy link
Collaborator Author

@alisonshao
Copy link
Collaborator Author

/rerun-stage stage-b-test-small-1-gpu-amd

@github-actions
Copy link
Contributor

github-actions bot commented Jan 6, 2026

✅ Triggered stage-b-test-small-1-gpu-amd to run independently (skipping dependencies).

It will not be shown in this page. Check the Actions tab for progress.

@alisonshao
Copy link
Collaborator Author

alisonshao commented Jan 6, 2026

amd 1 gpu passed: https://github.com/sgl-project/sglang/actions/runs/20735509755

@alisonshao
Copy link
Collaborator Author

/rerun-stage stage-b-test-large-2-gpu-amd

@github-actions
Copy link
Contributor

github-actions bot commented Jan 6, 2026

✅ Triggered stage-b-test-large-2-gpu-amd to run independently (skipping dependencies).

It will not be shown in this page. Check the Actions tab for progress.

@alisonshao
Copy link
Collaborator Author

alisonshao commented Jan 6, 2026

amd 2 gpu passed: https://github.com/sgl-project/sglang/actions/runs/20736307829

@alisonshao
Copy link
Collaborator Author

/rerun-stage stage-c-test-large-4-gpu

@github-actions
Copy link
Contributor

github-actions bot commented Jan 6, 2026

✅ Triggered stage-c-test-large-4-gpu to run independently (skipping dependencies).

It will not be shown in this page. Check the Actions tab for progress.

@alisonshao
Copy link
Collaborator Author

alisonshao commented Jan 6, 2026

nv 4 gpu passed: https://github.com/sgl-project/sglang/actions/runs/20736583928

@alisonshao
Copy link
Collaborator Author

/rerun-stage stage-b-test-large-2-gpu
/rerun-stage stage-b-test-small-1-gpu

@github-actions
Copy link
Contributor

github-actions bot commented Jan 6, 2026

✅ Triggered stage-b-test-large-2-gpu to run independently (skipping dependencies).

It will not be shown in this page. Check the Actions tab for progress.

@alisonshao
Copy link
Collaborator Author

@alisonshao
Copy link
Collaborator Author

/rerun-stage stage-b-test-small-1-gpu

@github-actions
Copy link
Contributor

github-actions bot commented Jan 6, 2026

✅ Triggered stage-b-test-small-1-gpu to run independently (skipping dependencies).

It will not be shown in this page. Check the Actions tab for progress.

@alisonshao
Copy link
Collaborator Author

@Kangyan-Zhou Kangyan-Zhou merged commit 861a35f into main Jan 6, 2026
33 of 57 checks passed
@Kangyan-Zhou Kangyan-Zhou deleted the ci/migrate-rl-tests branch January 6, 2026 06:13
jamesjxliu pushed a commit to jamesjxliu/sglang that referenced this pull request Jan 6, 2026
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