Skip to content

[None][infra] Waive 1 failed cases for main in pre-merge 64375 - #20427

Merged
tburt-nv merged 1 commit into
NVIDIA:mainfrom
trtllm-agent:trtllm-ci-report/waive-20261015-153205
Oct 15, 2026
Merged

tburt-nv merged 1 commit into
NVIDIA:mainfrom
trtllm-agent:trtllm-ci-report/waive-20261015-153205

Conversation

@trtllm-agent

@trtllm-agent trtllm-agent commented Oct 15, 2026 •

Copy link
Copy Markdown
Collaborator

Auto-generated Waive PR

Created by: TensorRT LLM CI Report (requested by @tburt-nv)
Target branch: main
Bug(s): 6903683

Waive entries added

unittest/llmapi/test_llm_pytorch.py::test_llm_disagg_streaming_gen_cancelled[PYTHON] SKIP (https://nvbugs/6903683)

This PR was auto-generated by TensorRT LLM CI Report. Please review the waive entries before merging.

Dev Engineer Review

The change adds one SKIP waiver for test_llm_disagg_streaming_gen_cancelled[PYTHON], linked to NVBUG 6903683. No source code or public API changes are reported.

QA Engineer Review

tests/integration/test_lists/waives.txt adds a waiver for the streaming-generation cancellation test. The test function remains in tests/unittest/llmapi/test_llm_pytorch.py; the waiver suppresses its run. Test results are not provided. Coverage verdict: needs follow-up until the waiver is removed or the test is otherwise covered.

Per-File QA Perspective

  • tests/integration/test_lists/waives.txt: Adds the SKIP entry for test_llm_disagg_streaming_gen_cancelled[PYTHON]. QA should track the linked bug and confirm the test can run again after resolution.

Bug(s): 6903683
Requested by: @tburt-nv

Signed-off-by: trtllm-agent <296075020+trtllm-agent@users.noreply.github.com>
@tburt-nv

Copy link
Copy Markdown
Collaborator

/bot skip --comment "waive only"

@github-actions

Copy link
Copy Markdown

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.

@coderabbitai

coderabbitai Bot commented Oct 15, 2026

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository: NVIDIA/TensorRT-LLM/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Enterprise
  • Run ID: 75357966-8188-4faf-a0ef-94f5f88a7396
📥 Commits

Reviewing files that changed from the base of the PR and between ed87ab0 and 6b48458.

📒 Files selected for processing: 1
  • tests/integration/test_lists/waives.txt

Included review availability: This review used your included allowance. Your plan provides up to 12 included reviews per hour; 11 remain after this review.

Walkthrough

The integration test skip list now includes test_llm_disagg_streaming_gen_cancelled[PYTHON], with NVBUG 6903683 as its tracking reference.

Changes

Integration test waiver

Layer / File(s) Summary
Add cancellation test to skip list
tests/integration/test_lists/waives.txt
The skip list adds the Python-parameterized disaggregated streaming-generation cancellation test and its NVBUG tracking reference.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~2 minutes

Change: Other

Suggested reviewers: bowenfu

Merge Risk: ⚪ Minimal · up to 6b484

This change skips only the tracked Python cancellation test; it does not change product behavior, and no actionable merge risk remains.

Pre-merge checks: Passed 5
Passed Passed checks: 5 passed
Check name Status Explanation
Title check Passed The title clearly identifies the infrastructure change: waiving one failed case for the main branch pre-merge run. It follows the required ticket, type, and summary format.
Description check Passed The description explains the waiver, identifies the affected test, links NVBUG 6903683, and states the target branch and requester. It does not include explicit Test Coverage or checklist sections, bu…
Docstring Coverage Passed 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 Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
✨ 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.

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #78172 [ skip ] triggered by Bot. Commit: 6b48458 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #78172 [ skip ] completed with state SUCCESS. Commit: 6b48458
Skipping testing for commit 6b48458

Link to invocation

@tburt-nv
tburt-nv merged commit dac1641 into NVIDIA:main Oct 15, 2026
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants