[https://nvbugs/6422432][fix] Unwaive testcase - #17947
Conversation
|
/bot run --extra-stage "GB200-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-2, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-1, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-2" |
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 12 included reviews per hour; 11 remain after this review. WalkthroughThe change updates ChangesIntegration test waiver maintenance
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: 🔵 Low · up to The PR re-enables a previously waived testcase. It is mergeable with owner awareness, but the affected GB200, DGX B300, and RTXPro6000D post-merge stages should be confirmed to ensure the waiver removal applies to the intended scope. Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Description checkExplanation The description includes the required template and a test command, but the Description section does not explain the issue or solution. The test coverage section also does not identify the relevant tests or expected validation. Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files.
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
PR_Github #67268 [ run ] triggered by Bot. Commit: |
|
PR_Github #67268 [ run ] completed with state
|
|
@QiJune Please add the following label so that I can verify whether the bug is reproducible: |
|
/bot run --extra-stage "GB200-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-2, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-1, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-2" --disable-fail-fast |
|
PR_Github #67453 [ run ] triggered by Bot. Commit: |
|
PR_Github #67453 [ run ] completed with state
|
|
Removed the "ci: post-merge approved" label because @YihuiLu512 could not be verified as an active member of NVIDIA/trt-llm-ci-approvers. Ask a member of that team to apply it. |
|
/bot run --extra-stage "GB200-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-2, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-1, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-2" --disable-fail-fast |
|
/bot kill |
|
PR_Github #67635 [ kill ] triggered by Bot. Commit: |
|
PR_Github #67635 [ kill ] completed with state |
|
/bot run --extra-stage "GB200-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-2, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-1, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-2" --disable-fail-fast |
|
PR_Github #67653 [ run ] triggered by Bot. Commit: |
|
PR_Github #67653 [ run ] completed with state
|
|
/bot run --extra-stage "GB200-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-2, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-1, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-2" --disable-fail-fast |
|
PR_Github #67748 [ run ] triggered by Bot. Commit: |
|
PR_Github #67748 [ run ] completed with state
|
|
/bot run --extra-stage "GB200-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-2, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-1, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-2" --disable-fail-fast |
|
PR_Github #68062 [ run ] triggered by Bot. Commit: |
|
/bot run --extra-stage "GB200-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-2, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-1, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-2" --disable-fail-fast |
|
PR_Github #68646 [ run ] completed with state |
|
PR_Github #68666 [ run ] completed with state
|
|
/bot run --extra-stage "GB200-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-2, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-1, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-2" --disable-fail-fast |
|
PR_Github #68748 [ run ] triggered by Bot. Commit: |
|
PR_Github #68748 [ run ] completed with state
|
a022956 to
3aa54cc
Compare
|
/bot run --extra-stage "GB200-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-2, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-1, RTXPro6000D-4_GPUs-PyTorch-Post-Merge-2" --disable-fail-fast |
|
PR_Github #68989 [ run ] triggered by Bot. Commit: |
|
/bot run --extra-stage "GB200-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-2" --disable-fail-fast |
|
PR_Github #68989 [ run ] completed with state
|
|
PR_Github #68996 [ run ] triggered by Bot. Commit: |
|
PR_Github #68996 [ run ] completed with state
|
3aa54cc to
02d0f85
Compare
|
/bot run --extra-stage "DGX_B300-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-2" --disable-fail-fast |
|
PR_Github #69254 [ run ] triggered by Bot. Commit: |
|
/bot kill |
|
PR_Github #69324 [ kill ] triggered by Bot. Commit: |
|
PR_Github #69254 [ run ] completed with state
|
|
PR_Github #69324 [ kill ] completed with state |
|
/bot run --stage-list "GB200-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-1, DGX_B300-4_GPUs-PyTorch-Post-Merge-2" --disable-fail-fast |
|
PR_Github #69330 [ run ] triggered by Bot. Commit: |
Signed-off-by: Yihui Lu <269394165+YihuiLu512@users.noreply.github.com>
02d0f85 to
1496608
Compare
|
/bot run --disable-fail-fast |
|
PR_Github #69399 [ run ] triggered by Bot. Commit: |
|
PR_Github #69330 [ run ] completed with state |
|
PR_Github #69399 [ run ] completed with state |
Dev Engineer Review
tests/integration/test_lists/waives.txt.QA Engineer Review
tests/integration/test_lists/waives.txt.test-db/orqa/files were modified.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.