[TRTLLMINF-69][infra] Migrate A100X-FMHA-Post-Merge-1 and A100X-Triton-Post-Merge-[1,2] to SLURM - #14921
Conversation
…SLURM Signed-off-by: Matt Lefebvre <mlefebvre@nvidia.com>
📝 WalkthroughWalkthroughThis PR migrates A100X post-merge test stages from Kubernetes-based to SLURM-based execution in Jenkins configuration. The stages ( ChangesA100X Test Stage SLURM Migration
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
/bot run --disable-fail-fast --stage-list "A100X-Triton-Post-Merge-1,A100X-Triton-Post-Merge-2,A100X-FMHA-Post-Merge-1" |
|
PR_Github #51920 [ run ] triggered by Bot. Commit: |
|
PR_Github #51920 [ run ] completed with state |
Co-authored-by: Yanchao Lu <yanchaol@nvidia.com> Signed-off-by: Matt Lefebvre <matthewelefebvre@gmail.com>
|
/bot run --disable-fail-fast --stage-list "DGX_A100-Triton-Post-Merge-1,DGX_A100-Triton-Post-Merge-2,DGX_A100-FMHA-Post-Merge-1" |
|
PR_Github #52105 [ run ] triggered by Bot. Commit: |
|
PR_Github #52105 [ run ] completed with state |
|
/bot skip --comment "DGX A100 tests passing" |
|
PR_Github #52135 [ skip ] triggered by Bot. Commit: |
|
PR_Github #52135 [ skip ] completed with state |
…n-Post-Merge-[1,2] to SLURM (NVIDIA#14921) Signed-off-by: Matt Lefebvre <mlefebvre@nvidia.com> Signed-off-by: Matt Lefebvre <matthewelefebvre@gmail.com> Co-authored-by: Yanchao Lu <yanchaol@nvidia.com> Signed-off-by: NVFB <186336021+NVFB@users.noreply.github.com>
…n-Post-Merge-[1,2] to SLURM (NVIDIA#14921) Signed-off-by: Matt Lefebvre <mlefebvre@nvidia.com> Signed-off-by: Matt Lefebvre <matthewelefebvre@gmail.com> Co-authored-by: Yanchao Lu <yanchaol@nvidia.com>
… before MTP one-model merge) Squashed probe equal to the tree of upstream commit 910826b ([TRTLLMINF-69][infra] Migrate A100X-FMHA/Triton Post-Merge to SLURM #14921), the commit immediately before the suspected culprit 8e5d9e2, rebased onto its parent a8c4007 so CI builds exactly that commit's tree. Authored/signed-off by chenfeiz to satisfy DCO. Signed-off-by: chenfeiz <chenfeiz@nvidia.com>
Summary by CodeRabbit
Description
Test Coverage
PR Checklist
Please review the following before submitting your PR:
PR description clearly explains what and why. If using CodeRabbit's summary, please make sure it makes sense.
PR Follows TRT-LLM CODING GUIDELINES to the best of your knowledge.
Test cases are provided for new code paths (see test instructions)
If PR introduces API changes, an appropriate PR label is added - either
api-compatibleorapi-breaking. Forapi-breaking, includeBREAKINGin the PR title.Any new dependencies have been scanned for license and vulnerabilities
CODEOWNERS updated if ownership changes
Documentation updated as needed
Update tava architecture diagram if there is a significant design change in PR.
The reviewers assigned automatically/manually are appropriate for the PR.
Please check this after reviewing the above items as appropriate for this PR.
GitHub Bot Help
To see a list of available CI bot commands, please comment
/bot help.