Skip to content

[https://nvbugs/6132301][infra] Waive 1 failed cases for main in pre-merge 36112#13679

Merged
zhenhuaw-me merged 1 commit intoNVIDIA:mainfrom
ZhanruiSunCh:trtllm-ci-report/waive-20260430-220700
May 1, 2026
Merged

[https://nvbugs/6132301][infra] Waive 1 failed cases for main in pre-merge 36112#13679
zhenhuaw-me merged 1 commit intoNVIDIA:mainfrom
ZhanruiSunCh:trtllm-ci-report/waive-20260430-220700

Conversation

@ZhanruiSunCh
Copy link
Copy Markdown
Collaborator

@ZhanruiSunCh ZhanruiSunCh commented Apr 30, 2026

Auto-generated Waive PR

Created by: TensorRT LLM CI Report (requested by @SimengLiu-nv)
Target branch: main
Bug(s): 6132301

Waive entries added

unittest/_torch/ray_orchestrator/multi_gpu/test_ops.py::test_cp_tp_broadcast_object[tp_cp_broadcast-list] SKIP (https://nvbugs/6132301)

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

Summary by CodeRabbit

  • Tests
    • Added a test waiver to skip a specific test case.

Note: This release contains no user-facing changes; the modification is internal testing infrastructure only.

@SimengLiu-nv SimengLiu-nv requested review from chzblych, xinhe-nv and zhenhuaw-me and removed request for chzblych and xinhe-nv April 30, 2026 22:07
@SimengLiu-nv SimengLiu-nv changed the title [None][infra] Waive 1 failed cases for main in pre-merge 36112 [https://nvbugs/6132301][infra] Waive 1 failed cases for main in pre-merge 36112 Apr 30, 2026
Bug(s): 6132301
Requested by: @SimengLiu-nv

Signed-off-by: ZhanruiSunCh <184402041+ZhanruiSunCh@users.noreply.github.com>
@SimengLiu-nv SimengLiu-nv force-pushed the trtllm-ci-report/waive-20260430-220700 branch from 57e7aae to 074b46d Compare April 30, 2026 22:08
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 30, 2026

📝 Walkthrough

Walkthrough

A single SKIP waiver line is added to the test list configuration to exclude a specific broadcast test case from execution, with a reference to a tracked issue.

Changes

Cohort / File(s) Summary
Test Waiver Configuration
tests/integration/test_lists/waives.txt
Added SKIP waiver for test_cp_tp_broadcast_object[tp_cp_broadcast-list] test case, referencing issue tracking reference.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly indicates this is an infrastructure change to waive a failed test case, with the specific issue reference (36112) and scope (main).
Description check ✅ Passed The description adequately explains the auto-generated waive PR, identifies the bug reference, and lists the specific test case being waived, though it deviates from the template structure.
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

Review rate limit: 9/10 reviews remaining, refill in 6 minutes.

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

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@tests/integration/test_lists/waives.txt`:
- Line 441: Update the waiver entry for the parametrized test so the
pytest-generated id order matches the parameters: change the existing waiver
line referencing
unittest/_torch/ray_orchestrator/multi_gpu/test_ops.py::test_cp_tp_broadcast_object[tp_cp_broadcast-list]
to use the correct id ordering produced by the decorators, i.e.
unittest/_torch/ray_orchestrator/multi_gpu/test_ops.py::test_cp_tp_broadcast_object[list-tp_cp_broadcast]
SKIP (https://nvbugs/6132301).
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 2dfe3617-b3b0-4051-98b0-5688fe660b82

📥 Commits

Reviewing files that changed from the base of the PR and between 4000e48 and 57e7aae.

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

Comment thread tests/integration/test_lists/waives.txt
@SimengLiu-nv
Copy link
Copy Markdown
Collaborator

/bot skip --comment "Waive only"

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #46446 [ skip ] triggered by Bot. Commit: 074b46d Link to invocation

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #46446 [ skip ] completed with state SUCCESS. Commit: 074b46d
Skipping testing for commit 074b46d

Link to invocation

@zhenhuaw-me zhenhuaw-me merged commit f7b01a3 into NVIDIA:main May 1, 2026
6 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