[None][infra] Waive 2 failed cases for main in pre-merge 51267 - #17182
[None][infra] Waive 2 failed cases for main in pre-merge 51267#17182trtllm-agent wants to merge 1 commit into
Conversation
Bug(s): 6550727 Requested by: @dominicshanshan Signed-off-by: trtllm-agent <296075020+trtllm-agent@users.noreply.github.com>
WalkthroughThe integration waiver list adds skipped entries for the ChangesIntegration test waivers
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 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 381-382: Update the modified waives.txt file with the required
NVIDIA copyright header, including the current year 2026, while preserving the
existing skip entries.
🪄 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: 30301f95-17e9-46bf-8a0d-1b45227456cd
📒 Files selected for processing (1)
tests/integration/test_lists/waives.txt
| unittest/_torch/executor SKIP (https://nvbugs/6550727) | ||
| unittest/_torch/executor/test_pytorch_model_engine.py::PyTorchModelEngineTestCase::test_promoted_context_precedes_speculative_overlap_generation SKIP (https://nvbugs/6550727) |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Add or update the NVIDIA copyright header.
The modified file has no visible NVIDIA copyright header or year. Add the required header with the current year, 2026.
As per coding guidelines, modified files must have the NVIDIA copyright year updated.
🤖 Prompt for 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.
In `@tests/integration/test_lists/waives.txt` around lines 381 - 382, Update the
modified waives.txt file with the required NVIDIA copyright header, including
the current year 2026, while preserving the existing skip entries.
Source: Coding guidelines
Auto-generated Waive PR
Created by: TensorRT LLM CI Report (requested by @dominicshanshan)
Target branch:
mainBug(s): 6550727
Waive entries added
This PR was auto-generated by TensorRT LLM CI Report. Please review the waive entries before merging.
Dev Engineer Review
tests/integration/test_lists/waives.txt.6550727.QA Engineer Review
tests/integration/test_lists/waives.txt._torch/executorunittest/_torch/executor/test_pytorch_model_engine.py::PyTorchModelEngineTestCase::test_promoted_context_precedes_speculative_overlap_generation