Skip to content

[TRTQA-3280][test] Waive 1 failed cases for main in QA CI#16599

Merged
xinhe-nv merged 1 commit into
NVIDIA:mainfrom
trtllm-agent:trtllm-ci-report/waive-20260720-061242
Jul 20, 2026
Merged

[TRTQA-3280][test] Waive 1 failed cases for main in QA CI#16599
xinhe-nv merged 1 commit into
NVIDIA:mainfrom
trtllm-agent:trtllm-ci-report/waive-20260720-061242

Conversation

@trtllm-agent

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

Copy link
Copy Markdown
Collaborator

Auto-generated Waive PR

Created by: TensorRT LLM CI Report (requested by qa@nvidia.com)
Target branch: main
Bug(s): 6327718, 6475621, 6475622

Waive entries added

full:H20/llmapi/test_llm_api_pytorch_moe_lora.py::test_qwen_moe_routed_expert_multi_lora_varying_ranks[eager] SKIP (https://nvbugs/6475621)

Already waived (skipped)

  • full:H20/accuracy/test_llm_api_pytorch_multimodal.py::TestNanoV3Omni::test_auto_dtype[fp8]
  • full:H20/accuracy/test_epd_disagg_multimodal.py::TestVideoMMEEPD::test_disaggregated_videomme[nemotron_nano_v3_omni_fp8]

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

Summary by CodeRabbit

  • Tests
    • Marked a known integration test as skipped and linked it to the relevant bug for tracking.

Bug(s): 6327718, 6475621, 6475622
Requested by: qa@nvidia.com

Signed-off-by: trtllm-agent <296075020+trtllm-agent@users.noreply.github.com>
@xinhe-nv
xinhe-nv enabled auto-merge (squash) July 20, 2026 06:13
@xinhe-nv xinhe-nv changed the title [None][test] Waive 1 failed cases for main in QA CI [TRTQA-3280][test] Waive 1 failed cases for main in QA CI Jul 20, 2026
@xinhe-nv

Copy link
Copy Markdown
Collaborator

/bot skip --comment "waive test only on H20"

@coderabbitai

coderabbitai Bot commented Jul 20, 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: b099aacc-914d-4d72-84aa-3146f8b1d2d3

📥 Commits

Reviewing files that changed from the base of the PR and between 447a628 and d6a5d2a.

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

📝 Walkthrough

Walkthrough

Adds a SKIP waiver for the H20 eager Qwen MoE routed-expert multi-LoRA integration test, including its linked bug reference.

Changes

Integration test waiver

Layer / File(s) Summary
Add Qwen MoE test waiver
tests/integration/test_lists/waives.txt
Adds a SKIP entry for test_qwen_moe_routed_expert_multi_lora_varying_ranks[eager].

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 includes the bug links and waived entry, but it omits the required Description, Test Coverage, and PR Checklist sections from the template. Add the missing template sections: a short Description, Test Coverage details, and the PR Checklist checkbox, while keeping the waive entries and bug references.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title is concise and accurately summarizes the PR's main change: waiving one failed QA CI test.
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.

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #60290 [ skip ] triggered by Bot. Commit: d6a5d2a Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #60290 [ skip ] completed with state SUCCESS. Commit: d6a5d2a
Skipping testing for commit d6a5d2a

Link to invocation

@xinhe-nv
xinhe-nv merged commit d42e391 into NVIDIA:main Jul 20, 2026
16 checks passed
@xinhe-nv
xinhe-nv deleted the trtllm-ci-report/waive-20260720-061242 branch July 20, 2026 06:27
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.

3 participants