Repository navigation
[None][infra] Waive 1 failed cases for main in pre-merge 67464 - #21378
Conversation
Bug(s): 7024242 Requested by: @HuiGao-NV Signed-off-by: trtllm-agent <296075020+trtllm-agent@users.noreply.github.com>
|
/bot skip --comment "waive one main break case" |
|
Automatically added "ci: full pre-merge approved" because this PR has satisfied the required GitHub review approvals. Unresolved review conversations and other required checks remain independent merge requirements. |
|
No actionable comments were generated in the recent review.
 |
| Layer / File(s) | Summary |
|---|---|
Add test waiver tests/integration/test_lists/waives.txt |
The waiver list adds the environment-scope test module with issue reference 7024242. |
Estimated code review effort: 1 (Trivial) | ~3 minutes
Change: Other
Possibly related PRs
- NVIDIA/TensorRT-LLM#21345: Adds the waived test module and its environment restoration behavior.
Suggested reviewers: xinhe-nv
Merge Risk: 
6f952
-
The waiver change is limited in scope and is ready to merge.
Pre-merge checks: 
-
Check name Status Explanation Title check The title follows the required [ticket][type] format and clearly identifies the infrastructure change: waiving one failed case for the main branch. Description check The description explains the automated waiver, identifies the affected test, links bug 7024242, and states the target branch. It does not include the template headings or checklist, but the relevant i… Docstring Coverage No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat… Linked Issues check Check skipped because no linked issues were found for this pull request. Out of Scope Changes check Check skipped because no linked issues were found for this pull request. Full details: Description check
-
Explanation
The description explains the automated waiver, identifies the affected test, links bug 7024242, and states the target branch. It does not include the template headings or checklist, but the relevant information is complete for this small waiver change.
Full details: Docstring Coverage
-
Explanation
No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)
-
Finishing Touches
-
Generate unit tests (beta)
-
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
|
PR_Github #81819 [ skip ] triggered by Bot. Commit: |
|
PR_Github #81819 [ skip ] completed with state |
Auto-generated Waive PR
Created by: TensorRT LLM CI Report (requested by @HuiGao-NV)
Target branch:
mainBug(s): 7024242
Waive entries added
This PR was auto-generated by TensorRT LLM CI Report. Please review the waive entries before merging.
Dev Engineer Review
The change adds one
SKIPwaiver intests/integration/test_lists/waives.txtforunittest/_torch/modeling_v2/test_modeling_v2_env_scope.py, linked to issue 7024242. It does not change source code or public APIs.QA Engineer Review
The change affects only a test-list entry. It waives the environment-scope test, which is also registered in
tests/integration/test_lists/test-db/l0_cpu.yml. No test function or test ID changed. Coverage verdict: needs follow-up because the waiver prevents this test from running.Per-File QA Perspective
tests/integration/test_lists/waives.txt: Adds a skip waiver for the environment-scope test. The test remains in the CI listtests/integration/test_lists/test-db/l0_cpu.yml; the waiver suppresses its execution.