Skip to content

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

Merged
crazydemo merged 1 commit into
NVIDIA:mainfrom
xinhe-nv:trtllm-ci-report/waive-20260517-144541
May 18, 2026
Merged

[None][test] Waive 1 failed cases for main in QA CI#14221
crazydemo merged 1 commit into
NVIDIA:mainfrom
xinhe-nv:trtllm-ci-report/waive-20260517-144541

Conversation

@xinhe-nv

@xinhe-nv xinhe-nv commented May 17, 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_CLUSTER_TEST/1423/
Bug(s): 6184575

Waive entries added

accuracy/test_disaggregated_serving.py::TestDeepSeekV3Lite::test_kv_cache_v2_nixl_python SKIP (https://nvbugs/6184575)

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

Summary by CodeRabbit

  • Tests
    • Updated test waiver list to skip a specific integration test case.

Review Change Stack

Bug(s): 6184575
Requested by: qa@nvidia.com
Jenkins build: https://prod.blsm.nvidia.com/swqa-tensorrt-qa-test/job/LLM_FUNCTION_CLUSTER_TEST/1423/

Signed-off-by: xinhe-nv <200704525+xinhe-nv@users.noreply.github.com>
@crazydemo
crazydemo force-pushed the trtllm-ci-report/waive-20260517-144541 branch from 2639d6e to 1ebb47f Compare May 18, 2026 04:28
@crazydemo
crazydemo marked this pull request as ready for review May 18, 2026 04:28
@crazydemo

Copy link
Copy Markdown
Collaborator

/bot run --stage-list ""

@coderabbitai

coderabbitai Bot commented May 18, 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: 75800a24-1a94-4374-a280-dcebc555e9ca

📥 Commits

Reviewing files that changed from the base of the PR and between 5e25977 and 1ebb47f.

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

📝 Walkthrough

Walkthrough

A single test case is added to the integration test waiver list for TestDeepSeekV3Lite::test_kv_cache_v2_nixl_python, marked as SKIP with a referenced NVBUG link.

Changes

Test Waiver Management

Layer / File(s) Summary
DeepSeekV3Lite KV Cache Test Waiver
tests/integration/test_lists/waives.txt
A new SKIP waiver entry is added for the accuracy/test_disaggregated_serving.py::TestDeepSeekV3Lite::test_kv_cache_v2_nixl_python test case with an NVBUG reference.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • NVIDIA/TensorRT-LLM#14001: Both PRs modify the same waiver file tests/integration/test_lists/waives.txt by adding SKIP entries for failing accuracy/... tests.
  • NVIDIA/TensorRT-LLM#14235: Both PRs modify tests/integration/test_lists/waives.txt to add SKIP entries for DeepSeekV3-related tests with NVBUG references.
  • NVIDIA/TensorRT-LLM#13991: Both PRs update tests/integration/test_lists/waives.txt by adding new SKIP waiver entries for TestDeepSeekV3Lite test cases.

Suggested reviewers

  • jieli-matrix
  • StanleySun639
  • LarryXFly
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is incomplete and does not follow the required template. It lacks required sections like Description, Test Coverage, and PR Checklist. Fill in the Description section explaining why the test is being waived, and complete the PR Checklist section to ensure compliance with contribution guidelines.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the main change: adding a test waiver entry for a failed test case in the QA CI 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 #48832 [ run ] triggered by Bot. Commit: 1ebb47f Link to invocation

@crazydemo

Copy link
Copy Markdown
Collaborator

/bot kill

@crazydemo
crazydemo enabled auto-merge (squash) May 18, 2026 08:31
@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #48873 [ kill ] triggered by Bot. Commit: 1ebb47f Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #48832 [ run ] completed with state ABORTED. Commit: 1ebb47f

Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #48873 [ kill ] completed with state SUCCESS. Commit: 1ebb47f
Successfully killed previous jobs for commit 1ebb47f

Link to invocation

@crazydemo

Copy link
Copy Markdown
Collaborator

/bot skip --comment "skip ci as check test list has passes"

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #48897 [ skip ] triggered by Bot. Commit: 1ebb47f Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #48897 [ skip ] completed with state SUCCESS. Commit: 1ebb47f
Skipping testing for commit 1ebb47f

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