Skip to content

[None][infra] Waive 1 failed cases for release/1.3 in pre-merge 387 - #20691

Merged
rosong11 merged 1 commit into
NVIDIA:release/1.3from
trtllm-agent:trtllm-ci-report/waive-20261020-164854
Oct 23, 2026
Merged

rosong11 merged 1 commit into
NVIDIA:release/1.3from
trtllm-agent:trtllm-ci-report/waive-20261020-164854

Conversation

@trtllm-agent

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

Copy link
Copy Markdown
Collaborator

Auto-generated Waive PR

Created by: TensorRT LLM CI Report (requested by @tburt-nv)
Target branch: release/1.3
Bug(s): 6778221

Waive entries added

unittest/_torch/moe/test_moe_backend.py::test_trtllm_bf16_unquantized_moe[deepseekv3-relu2-8-separated_routing] SKIP (https://nvbugs/6778221)

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

Dev Engineer Review

  • Added one targeted SKIP waiver for the known BF16 unquantized MoE failure.
  • The waiver is linked to NVBug 6778221.
  • No source code, public API, or runtime behavior changed.
  • The waiver reduces CI coverage for one parameterized case. Remove it after the defect is fixed.

QA Engineer Review

  • Modified tests/integration/test_lists/waives.txt.
  • Added a waiver for unittest/_torch/moe/test_moe_backend.py::test_trtllm_bf16_unquantized_moe[deepseekv3-relu2-8-separated_routing].
  • The entry suppresses this known failure and does not change the test implementation.
  • Coverage verdict: needs follow-up.

Per-File QA Perspective

  • tests/integration/test_lists/waives.txt: Adds one NVBug-linked CI skip for a DeepSeekV3 MoE test. Verify that the waiver applies only to the intended parameterized case and remove it when NVBug 6778221 is resolved.

Bug(s): 6778221
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 20, 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: 8f07d66b-6325-4990-a900-27f2586f8d80
📥 Commits

Reviewing files that changed from the base of the PR and between 7f1b51c and 860f256.

📒 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; 10 remain after this review.

Walkthrough

The skipped-test list now includes test_trtllm_bf16_unquantized_moe, tracked by NVBug 6778221.

Changes

Integration test waiver

Layer / File(s) Summary
Add the test waiver
tests/integration/test_lists/waives.txt
Adds test_trtllm_bf16_unquantized_moe to the skipped-test list with NVBug 6778221 as its tracking reference.

Priority: ⬇️ Low

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

Change: Other

Suggested reviewers: yuxianq

Merge Risk: ⚪ Minimal · up to 860f2

The waiver is narrowly scoped and tracked by NVBug 6778221, so this change is mergeable.

Pre-merge checks: Passed 5
Check name Status Explanation
Title check Passed The title clearly identifies an infrastructure change that waives one failed case for the release/1.3 pre-merge run. It follows the required ticket and type format.
Description check Passed The description explains the automated origin, target branch, bug ID, and exact test case being waived. It does not reproduce the full template headings or checklist, but it provides the information r…
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.
Full details: Description check

Explanation

The description explains the automated origin, target branch, bug ID, and exact test case being waived. It does not reproduce the full template headings or checklist, but it provides the information required for this narrow waiver change.

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.

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #79315 [ skip ] triggered by Bot. Commit: 860f256 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #79315 [ skip ] completed with state SUCCESS. Commit: 860f256
Skipping testing for commit 860f256

Link to invocation

@BowenFu

BowenFu commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

Bug engineer ownership

@rosong11, as the engineer assigned to the linked bug, you are responsible for driving this bot PR to resolution. Please coordinate review, address feedback and required code changes, get CI passing, and verify the original failure before driving it to merge. If the PR is obsolete or superseded, close it with a reason. Keep this PR updated with blockers and the next step.

@rosong11
rosong11 merged commit 24d278c into NVIDIA:release/1.3 Oct 23, 2026
17 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.

5 participants