Skip to content

[None][infra] Waive 1 failed cases for main in pre-merge 43656 - #15439

Merged
2ez4bz merged 1 commit into
NVIDIA:mainfrom
ZhanruiSunCh:trtllm-ci-report/waive-20260617-031050
Jun 17, 2026
Merged

[None][infra] Waive 1 failed cases for main in pre-merge 43656#15439
2ez4bz merged 1 commit into
NVIDIA:mainfrom
ZhanruiSunCh:trtllm-ci-report/waive-20260617-031050

Conversation

@ZhanruiSunCh

@ZhanruiSunCh ZhanruiSunCh commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

Auto-generated Waive PR

Created by: TensorRT LLM CI Report (requested by @2ez4bz)
Target branch: main
Bug(s): 6335726

Waive entries added

full:H100_PCIe/unittest/llmapi/test_llm_pytorch.py::test_qwen_moe_routed_expert_multi_lora_varying_ranks SKIP (https://nvbugs/6335726)

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

Summary by CodeRabbit

  • Chores
    • Updated test infrastructure configuration to skip a specific test case.

Bug(s): 6335726
Requested by: @2ez4bz

Signed-off-by: ZhanruiSunCh <184402041+ZhanruiSunCh@users.noreply.github.com>
@2ez4bz
2ez4bz enabled auto-merge (squash) June 17, 2026 03:11
@2ez4bz

2ez4bz commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

/bot skip --comment "Test has timed out multiple times across multiple PRs"

@coderabbitai

coderabbitai Bot commented Jun 17, 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: a7e400b2-d161-43a0-8fc1-d6711cd98457

📥 Commits

Reviewing files that changed from the base of the PR and between fd5de7e and a719ac6.

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

📝 Walkthrough

Walkthrough

A single SKIP waiver entry is added to the integration test waiver list for the test_qwen_moe_routed_expert_multi_lora_varying_ranks test targeting the full:H100_PCIe configuration, referencing nvbug 6335726.

Changes

Test Waiver Addition

Layer / File(s) Summary
SKIP entry for Qwen MoE multi-LoRA varying ranks
tests/integration/test_lists/waives.txt
Adds a waiver line skipping test_qwen_moe_routed_expert_multi_lora_varying_ranks for the full:H100_PCIe target, linked to nvbug 6335726.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • NVIDIA/TensorRT-LLM#13648: Modifies the same tests/integration/test_lists/waives.txt file by adding a SKIP/waiver entry using the same mechanism.
  • NVIDIA/TensorRT-LLM#15273: Modifies tests/integration/test_lists/waives.txt by adding SKIP/waiver entries in the same format and file.
  • NVIDIA/TensorRT-LLM#15373: Modifies tests/integration/test_lists/waives.txt by adding SKIP waivers including entries for related unittest/_torch targets.

Suggested reviewers

  • 2ez4bz
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description is an auto-generated waiver entry with clear context, but lacks explicit sections following the repository template (Description, Test Coverage, PR Checklist). Consider expanding the description to include sections from the template, such as detailed description of why the waiver is needed and relevant test information.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The PR title clearly describes the change: adding one waive entry for a failing test case in the pre-merge pipeline.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #54737 [ skip ] triggered by Bot. Commit: a719ac6 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

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

Link to invocation

@2ez4bz
2ez4bz merged commit 9b77c8e into NVIDIA:main Jun 17, 2026
11 of 12 checks passed
xinhe-nv pushed a commit to tensorrt-cicd/TensorRT-LLM that referenced this pull request Jun 23, 2026
…A#15439)

Signed-off-by: ZhanruiSunCh <184402041+ZhanruiSunCh@users.noreply.github.com>
Signed-off-by: GitLab CI Bot <gitlab-ci@nvidia.com>
xinhe-nv pushed a commit to tensorrt-cicd/TensorRT-LLM that referenced this pull request Jun 24, 2026
…A#15439)

Signed-off-by: ZhanruiSunCh <184402041+ZhanruiSunCh@users.noreply.github.com>
Signed-off-by: GitLab CI Bot <gitlab-ci@nvidia.com>
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