[None][test] Waive 7 failed cases for main in QA CI - #16949
Conversation
Bug(s): 6525007, 6525008, 6525009, 6525010, 6525011, 6526186 Requested by: qa@nvidia.com Signed-off-by: trtllm-agent <296075020+trtllm-agent@users.noreply.github.com>
WalkthroughExpanded the B200 integration-test waiver list with seven new SKIP entries covering additional model accuracy, quantization, contention, and block-reuse variants. ChangesB200 integration waivers
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@tests/integration/test_lists/waives.txt`:
- Around line 141-154: Add the standard NVIDIA copyright header at the beginning
of the modified waives.txt file, including the current copyright year, while
preserving all existing test waiver entries unchanged.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Enterprise
Run ID: 75a6ce51-189b-4390-8321-83556a542f92
📒 Files selected for processing (1)
tests/integration/test_lists/waives.txt
Signed-off-by: Jie Li <76780849+jieli-matrix@users.noreply.github.com>
|
/bot skip --comment "Only waive tests" |
|
PR_Github #63336 [ skip ] triggered by Bot. Commit: |
Signed-off-by: Jie Li <lijie@nvidia.com>
|
/bot skip --comment "Only waive tests" |
|
PR_Github #63338 [ skip ] triggered by Bot. Commit: |
|
PR_Github #63336 [ skip ] completed with state |
|
PR_Github #63338 [ skip ] completed with state |
Auto-generated Waive PR
Created by: TensorRT LLM CI Report (requested by qa@nvidia.com)
Target branch:
mainBug(s): 6525007, 6525008, 6525009, 6525010, 6525011, 6526186
Waive entries added
This PR was auto-generated by TensorRT LLM CI Report. Please review the waive entries before merging.
Dev Engineer Review
Updated
tests/integration/test_lists/waives.txtwith seven B200SKIPwaivers tied to bugs 6525007–6525011 and 6526186. The entries follow the existing format and scope; no public API or code changes are included. No obvious duplicate or malformed entries were identified.QA Engineer Review
Only
tests/integration/test_lists/waives.txtwas modified. Seven B200 accuracy test cases were added to the waiver list; no entries were removed. CBTS coverage data was not provided.Verdict: needs follow-up