Skip to content

[None][infra] Waive 1 failed cases for main in post-merge 2965 - #19334

Open
trtllm-agent wants to merge 1 commit into
NVIDIA:mainfrom
trtllm-agent:trtllm-ci-report/waive-20260917-065645
Open

trtllm-agent wants to merge 1 commit into
NVIDIA:mainfrom
trtllm-agent:trtllm-ci-report/waive-20260917-065645

Conversation

@trtllm-agent

@trtllm-agent trtllm-agent commented Sep 17, 2026

Copy link
Copy Markdown
Collaborator

Auto-generated Waive PR

Created by: TensorRT LLM CI Report (requested by @mzweilz)
Target branch: main
Bug(s): 6791735

Waive entries added

accuracy/test_disaggregated_serving.py::TestQwen3_5_4B::test_mismatched_block_reuse SKIP (https://nvbugs/6791735)

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

Dev Engineer Review

Adds one targeted waiver for accuracy/test_disaggregated_serving.py::TestQwen3_5_4B::test_mismatched_block_reuse, linked to bug 6791735. No source or API changes are included. The waiver can hide regressions until the bug is fixed.

QA Engineer Review

tests/integration/test_lists/waives.txt adds a SKIP entry for the failing integration test. No test implementation changed, and no test execution result is provided. No corresponding test-db/ or qa/ list update is included. Coverage verdict: needs follow-up.

Per-File QA Perspective

  • tests/integration/test_lists/waives.txt: The entry suppresses the specified failing test and records bug 6791735. QA should remove the waiver and verify the test after the bug is fixed.

Bug(s): 6791735
Requested by: @mzweilz

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

coderabbitai Bot commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 7b5ab432-1916-4f1d-a313-32dd10a05b3b

📥 Commits

Reviewing files that changed from the base of the PR and between 73c7063 and e6c6abf.

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

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


Walkthrough

The integration test waiver list now skips TestQwen3_5_4B::test_mismatched_block_reuse and references NVIDIA bug 6791735.

Changes

Integration test waiver

Layer / File(s) Summary
Add test waiver
tests/integration/test_lists/waives.txt
Adds a skip entry for the Qwen3.5 4B mismatched block reuse integration test and references bug 6791735.

Priority: ⬇️ Low

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

Change: Other

Suggested reviewers: brnguyen2

Merge Risk: ⚪ Minimal · up to e6c6a

This narrowly skips the reported failing integration case and does not introduce an identified production or data risk.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title uses the required ticket, type, and summary format. It clearly identifies an infrastructure change that waives one failed case for the main branch.
Description check ✅ Passed The description explains the waiver, identifies the affected test, links bug 6791735, and states the target branch and generation source. It does not include a separate Test Coverage section or comple…
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 PR with unit tests

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant