Skip to content

[None][test] Waive 1 failed cases for main in QA CI - #14503

Merged
jieli-matrix merged 2 commits into
NVIDIA:mainfrom
xinhe-nv:trtllm-ci-report/waive-20260524-042549
May 25, 2026
Merged

[None][test] Waive 1 failed cases for main in QA CI#14503
jieli-matrix merged 2 commits into
NVIDIA:mainfrom
xinhe-nv:trtllm-ci-report/waive-20260524-042549

Conversation

@xinhe-nv

@xinhe-nv xinhe-nv commented May 24, 2026

Copy link
Copy Markdown
Collaborator

Auto-generated Waive PR

Created by: TensorRT LLM CI Report (requested by qa@nvidia.com)
Target branch: main
Jenkins build: https://prod.blsm.nvidia.com/swqa-tensorrt-qa-test/job/LLM_FUNCTION_TEST/2192/
Bug(s): 6211880

Waive entries added

accuracy/test_llm_api_pytorch.py::TestGPTOSS::test_eagle3_4gpus[v2_kv_cache-cutlass-two_model-no_overlap_scheduler] SKIP (https://nvbugs/6211880)

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

Summary by CodeRabbit

  • Chores
    • Updated test suite configuration to skip a specific GPU-related performance test as part of ongoing test maintenance and infrastructure optimization.

Review Change Stack

Bug(s): 6211880
Requested by: qa@nvidia.com
Jenkins build: https://prod.blsm.nvidia.com/swqa-tensorrt-qa-test/job/LLM_FUNCTION_TEST/2192/

Signed-off-by: xinhe-nv <200704525+xinhe-nv@users.noreply.github.com>
@jieli-matrix
jieli-matrix force-pushed the trtllm-ci-report/waive-20260524-042549 branch from 4ddc9dc to 7d719d5 Compare May 25, 2026 03:05
@jieli-matrix
jieli-matrix marked this pull request as ready for review May 25, 2026 03:05
@jieli-matrix
jieli-matrix requested a review from dongfengy May 25, 2026 03:06
@coderabbitai

coderabbitai Bot commented May 25, 2026

Copy link
Copy Markdown
Contributor

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: 9507fa6e-4abc-4bd4-b25f-f3036bebfebf

📥 Commits

Reviewing files that changed from the base of the PR and between e45a8e3 and 7d719d5.

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

📝 Walkthrough

Walkthrough

A single test waiver entry is added to tests/integration/test_lists/waives.txt to skip a specific parameterized test variant of test_eagle3_4gpus within the TestGPTOSS class, marked with a referenced NVBug link.

Changes

Test Waiver Configuration

Layer / File(s) Summary
Eagle3 4GPUs v2_kv_cache test waiver
tests/integration/test_lists/waives.txt
Added SKIP waiver for accuracy/test_llm_api_pytorch.py::TestGPTOSS::test_eagle3_4gpus[v2_kv_cache-cutlass-two_model-no_overlap_scheduler] with NVBug reference.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • NVIDIA/TensorRT-LLM#13991: Adds SKIP waiver entries to tests/integration/test_lists/waives.txt for different accuracy/test_llm_api_pytorch.py test cases.
  • NVIDIA/TensorRT-LLM#14341: Modifies the same tests/integration/test_lists/waives.txt file by updating SKIP entries for specific test_llmapi/Eagle3 speculative-decoding-related tests.
  • NVIDIA/TensorRT-LLM#14450: Modifies tests/integration/test_lists/waives.txt to add new SKIP waivers for failing integration tests including eagle3_4gpus variants.

Suggested reviewers

  • crazydemo
  • StanleySun639
  • LarryXFly
  • jieli-matrix
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely indicates the main change: waiving 1 failed test case for the main branch QA CI pipeline, following the repository's title format [None][test].
Description check ✅ Passed The description includes all essential information: auto-generated status, CI context, target branch, Jenkins build link, bug reference, and the specific waive entry being added. While it lacks a traditional description/test coverage section, it provides adequate context for this auto-generated waive PR.
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.

@jieli-matrix

Copy link
Copy Markdown
Collaborator

/bot run --stage-list ""

@jieli-matrix
jieli-matrix enabled auto-merge (squash) May 25, 2026 03:12
@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #50132 [ run ] triggered by Bot. Commit: 7d719d5 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #50132 [ run ] completed with state SUCCESS. Commit: 7d719d5
/LLM/main/L0_MergeRequest_PR pipeline #39684 (Partly Tested) completed with status: 'SUCCESS'

CI Report

Link to invocation

@jieli-matrix

Copy link
Copy Markdown
Collaborator

/bot reuse-pipeline

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #50150 [ reuse-pipeline ] triggered by Bot. Commit: 45a6e7a Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #50150 [ reuse-pipeline ] completed with state SUCCESS. Commit: 45a6e7a
Reusing PR_Github #50132 (Partly Tested) for commit 45a6e7a

Link to invocation

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