[None][test] Waive 6 failed cases for main in QA CI - #16295
Conversation
Bug(s): 6410881, 6423866, 6424188, 6442594, 6445375 Requested by: qa@nvidia.com Signed-off-by: trtllm-agent <296075020+trtllm-agent@users.noreply.github.com>
📝 WalkthroughWalkthroughThe B300 waiver list was expanded and reorganized to cover additional DeepSeekV3Lite, DeepSeekV32, and MiniMaxM3 integration test configurations. ChangesB300 integration test waivers
Estimated code review effort: 1 (Trivial) | ~5 minutes 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 213-225: Update the header handling for waives.txt to use a
parser-safe # comment format while retaining the required copyright/year update.
Ensure waives_rule.py does not interpret the header lines as waiver test IDs;
only exempt the file from the header rule instead if that is the established
project convention.
🪄 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: 35216e65-cac0-4875-847b-e815213a15a8
📒 Files selected for processing (1)
tests/integration/test_lists/waives.txt
|
/bot skip --comment "Only waive tests" |
|
PR_Github #58869 [ skip ] triggered by Bot. Commit: |
|
PR_Github #58869 [ skip ] completed with state |
Auto-generated Waive PR
Created by: TensorRT LLM CI Report (requested by qa@nvidia.com)
Target branch:
mainBug(s): 6410881, 6423866, 6424188, 6442594, 6445375
Waive entries added
This PR was auto-generated by TensorRT LLM CI Report. Please review the waive entries before merging.
Summary by CodeRabbit