Skip to content

[None][infra] Waive 1 failed cases for main in pre-merge 48013#16470

Merged
shuyixiong merged 1 commit into
NVIDIA:mainfrom
trtllm-agent:trtllm-ci-report/waive-20260716-050128
Jul 16, 2026
Merged

[None][infra] Waive 1 failed cases for main in pre-merge 48013#16470
shuyixiong merged 1 commit into
NVIDIA:mainfrom
trtllm-agent:trtllm-ci-report/waive-20260716-050128

Conversation

@trtllm-agent

@trtllm-agent trtllm-agent commented Jul 16, 2026

Copy link
Copy Markdown
Collaborator

Auto-generated Waive PR

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

Waive entries added

unittest/_torch/modules/moe/test_moe_module.py::test_configurable_moe_single_gpu -k "TRTLLM" SKIP (https://nvbugs/6464169)

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

Summary by CodeRabbit

  • Tests
    • Added a waived test entry for the configurable MoE single-GPU test when filtered for TRTLLM.
    • Documented the associated tracking issue and marked the test to be skipped.

Bug(s): 6464169
Requested by: @shuyixiong

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

coderabbitai Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Review 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: 3c87fe82-d704-4fec-8a48-9e89408bcb1f

📥 Commits

Reviewing files that changed from the base of the PR and between 6013944 and 0a9d31b.

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

📝 Walkthrough

Walkthrough

Adds a SKIP waiver for test_configurable_moe_single_gpu when filtered with TRTLLM, linked to nvbugs issue 6464169.

Changes

MoE test waiver

Layer / File(s) Summary
Add configurable MoE waiver
tests/integration/test_lists/waives.txt
Adds a SKIP entry for the configurable single-GPU MoE test with the TRTLLM filter and nvbugs reference 6464169.

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

Possibly related PRs

Suggested reviewers: bowenfu

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description covers the waiver and bug link, but it omits the required Test Coverage and PR Checklist sections. Add the missing Description, Test Coverage, and PR Checklist sections and briefly explain the waiver rationale and validation performed.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title matches the change: adding an infra waiver for one failed main pre-merge test case.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

@shuyixiong

Copy link
Copy Markdown
Collaborator

/bot skip --comment "Waive a test"

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #59616 [ skip ] triggered by Bot. Commit: 0a9d31b Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #59616 [ skip ] completed with state SUCCESS. Commit: 0a9d31b
Skipping testing for commit 0a9d31b

Link to invocation

@shuyixiong
shuyixiong merged commit 5006f4e into NVIDIA:main Jul 16, 2026
11 checks passed
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.

4 participants