Repository navigation
[None][infra] Waive 1 failed cases for main in pre-merge 66075 - #20959
Conversation
Bug(s): 6988867 Requested by: @chzblych Signed-off-by: trtllm-agent <296075020+trtllm-agent@users.noreply.github.com>
|
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. |
|
/bot skip --comment "Test waive" |
|
No actionable comments were generated in the recent review.
 |
| Layer / File(s) | Summary |
|---|---|
Add test skip entry tests/integration/test_lists/waives.txt |
The skip list adds the GPT-OSS 120B Harmony test, linked to NV bug 6988867. |
Priority:
Estimated code review effort: 1 (Trivial) | ~3 minutes
Change: Other
Suggested labels: ci: full pre-merge approved
Suggested reviewers: brnguyen2
Merge Risk: 
27949
-
No actionable merge risk was identified; the waiver is narrowly scoped to the intended test.
Pre-merge checks: 
-
Check name Status Explanation Title check The title names the change as a waiver of one failed case on main for pre-merge 66075, which matches the changeset that adds one SKIP entry to tests/integration/test_lists/waives.txt. It follows the [… Description check The description follows the auto-generated waive PR format. It names the target branch (main), the bug (6988867, linked), the requester, and the single waive entry added. It does not fill in the templ… Docstring Coverage 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… 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: Title check
-
Explanation
The title names the change as a waiver of one failed case on main for pre-merge 66075, which matches the changeset that adds one SKIP entry to tests/integration/test_lists/waives.txt. It follows the [None][infra] template format. It omits the bug ID, but the template allows [None] when no ticket applies, and the bug ID appears in the description.
Full details: Description check
-
Explanation
The description follows the auto-generated waive PR format. It names the target branch (main), the bug (6988867, linked), the requester, and the single waive entry added. It does not fill in the template sections: Description, Test Coverage, and the PR Checklist are absent. For a one-line waive, the missing sections are low-impact, and the template's required title format is satisfied by the title.
Full details: Docstring Coverage
-
Explanation
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. (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
Comment @coderabbitai help to get the list of available commands.
|
PR_Github #80450 [ skip ] triggered by Bot. Commit: |
|
PR_Github #80450 [ skip ] completed with state |
Auto-generated Waive PR
Created by: TensorRT LLM CI Report (requested by @chzblych)
Target branch:
mainBug(s): 6988867
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 targeted
SKIPwaiver for the GPT-OSS 120B Harmony disaggregated test. It cites NV bug 6988867 and does not change source code or test behavior.QA Engineer Review
tests/integration/test_lists/waives.txtadds the waiver fordisaggregated/test_disaggregated.py::test_disaggregated_gpt_oss_120b_harmony[gpt_oss/gpt-oss-120b]. The test remains listed in both the CItest-db/l0_dgx_b200.ymllist and the manual-QAqa/llm_function_core.txtlist. Coverage verdict: sufficient for this test-list-only change. Test execution results were not provided.Per-File QA Perspective
tests/integration/test_lists/waives.txt: The entry marks the specified test to skip and links the waiver to NV bug 6988867. The test remains in the CI and manual-QA lists.