ci: Split L5 GPU workflows from L4 - #164
Merged
Merged
Conversation
added 4 commits
April 13, 2026 18:21
Performance Comparison
|
Contributor
There was a problem hiding this comment.
Pull request overview
Splits GPU CI coverage into separate workflows for L4 golden checkpoint parity and L5 end-to-end generation tests to improve clarity and scheduling flexibility.
Changes:
- Refactors existing
gpu_tests.ymlto run only L4 golden parity and updates Codecov flags/artifact naming accordingly. - Adds
gpu_l4_golden_parity.ymlto run L4 golden parity as its own GPU workflow. - Adds
gpu_l5_generation_e2e.ymlto run L5 generation E2E as its own GPU workflow.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| .github/workflows/gpu_tests.yml | Updated to L4-only GPU golden parity, with renamed workflow/job metadata and uploads. |
| .github/workflows/gpu_l4_golden_parity.yml | New dedicated L4 GPU golden parity workflow (currently overlaps with gpu_tests.yml). |
| .github/workflows/gpu_l5_generation_e2e.yml | New dedicated L5 GPU generation E2E workflow with its own schedule and uploads. |
|
The author of this PR, tadani3, is not an activated member of this organization on Codecov. |
justinchuby
approved these changes
Apr 13, 2026
added 2 commits
April 14, 2026 20:28
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
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.
This pull request restructures the GPU CI workflows by splitting the previous combined L4/L5 GPU test workflow into two distinct workflows: one for L4 Golden Checkpoint Parity tests and another for L5 End-to-End Generation tests. This improves clarity, maintainability, and scheduling flexibility for GPU-based tests.
Workflow restructuring and separation:
.github/workflows/gpu_l4_golden_parity.ymldedicated to running L4 Golden Checkpoint Parity tests on GPU, including setup, caching, test execution, and result uploads..github/workflows/gpu_l5_generation_e2e.ymlfor L5 End-to-End Generation tests on GPU, structured similarly to the L4 workflow but targeting generation correctness.Modifications to the existing GPU test workflow:
.github/workflows/gpu_tests.ymlis now deleted. The new L4 and L5 workflows have also been updated to run on a nightly schedule.