Skip to content

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

Merged
xinhe-nv merged 6 commits into
NVIDIA:mainfrom
tensorrt-cicd:trtllm-ci-report/waive-20260619-161606
Jun 23, 2026
Merged

[None][test] Waive 1 failed cases for main in QA CI#15499
xinhe-nv merged 6 commits into
NVIDIA:mainfrom
tensorrt-cicd:trtllm-ci-report/waive-20260619-161606

Conversation

@tensorrt-cicd

@tensorrt-cicd tensorrt-cicd commented Jun 19, 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/1557/
Bug(s): 6312828

Waive entries added

full:H100/disaggregated/test_disaggregated.py::test_disaggregated_stress_test[input8k-output1k-conc512-qwen3_32b_fp8_stress] SKIP (https://nvbugs/6312828)

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

Summary by CodeRabbit

  • Tests
    • Added a waiver for a failing stress test to improve test suite stability.

@xinhe-nv xinhe-nv marked this pull request as ready for review June 23, 2026 06:07
@xinhe-nv xinhe-nv force-pushed the trtllm-ci-report/waive-20260619-161606 branch from 0b958ca to 3469d98 Compare June 23, 2026 06:07
@xinhe-nv xinhe-nv force-pushed the trtllm-ci-report/waive-20260619-161606 branch from 3469d98 to 2e3a149 Compare June 23, 2026 06:09
@xinhe-nv

Copy link
Copy Markdown
Collaborator

/bot run --comment "Only waive tests"

@xinhe-nv xinhe-nv enabled auto-merge (squash) June 23, 2026 06:10
@coderabbitai

coderabbitai Bot commented Jun 23, 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: 23d70480-b919-4adc-b31f-63304e30dd5c

📥 Commits

Reviewing files that changed from the base of the PR and between 259b71f and 3469d98.

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

📝 Walkthrough

Walkthrough

A single SKIP waiver line is added to the integration test waiver list for the H100 disaggregated stress test variant test_disaggregated_stress_test[input8k-output1k-conc512-qwen3_32b_fp8_stress], referencing nvbugs 6312828.

Changes

H100 Disaggregated Stress Test Waiver

Layer / File(s) Summary
Add SKIP waiver entry
tests/integration/test_lists/waives.txt
Adds one new waiver line skipping the qwen3_32b_fp8_stress H100 disaggregated stress test variant, referencing nvbugs 6312828.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~1 minutes

Possibly related PRs

  • NVIDIA/TensorRT-LLM#15315: Adds an identical SKIP waiver for the same test (test_disaggregated_stress_test[input8k-output1k-conc512-qwen3_32b_fp8_stress]) and nvbugs 6312828, but targeting full:B200/disaggregated instead of full:H100/disaggregated.
  • NVIDIA/TensorRT-LLM#15505: Also modifies tests/integration/test_lists/waives.txt by adding SKIP entries for disaggregated test_disaggregated.py test cases.
  • NVIDIA/TensorRT-LLM#15511: Touches the same waiver file by adding/removing SKIP entries for specific parameterized integration tests, including Qwen3-related cases.

Suggested reviewers

  • crazydemo
  • jieli-matrix
  • StanleySun639
  • LarryXFly
  • xinhe-nv
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title follows the required template format with [None][test] prefix and clearly describes the main change: waiving one failed test case for the main branch in QA CI.
Description check ✅ Passed The description provides essential context (auto-generated status, target branch, Jenkins build, bug reference) and lists the waive entry added, though it lacks the standard PR template sections like explicit Description, Test Coverage, and PR Checklist.
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.

chienchunhung and others added 4 commits June 23, 2026 06:10
…IA#15471)

Signed-off-by: Chien-Chun Hung <2679986+chienchunhung@users.noreply.github.com>
Signed-off-by: GitLab CI Bot <gitlab-ci@nvidia.com>
Signed-off-by: GitLab CI Bot <gitlab-ci@nvidia.com>
Signed-off-by: GitLab CI Bot <gitlab-ci@nvidia.com>
Bug(s): 6312828
Requested by: qa@nvidia.com
Jenkins build: https://prod.blsm.nvidia.com/swqa-tensorrt-qa-test/job/LLM_FUNCTION_CLUSTER_TEST/1557/

Signed-off-by: tensorrt-cicd <90828364+tensorrt-cicd@users.noreply.github.com>
Signed-off-by: GitLab CI Bot <gitlab-ci@nvidia.com>
@xinhe-nv xinhe-nv force-pushed the trtllm-ci-report/waive-20260619-161606 branch from 3b2f479 to 92ccb48 Compare June 23, 2026 06:10
@tensorrt-cicd

Copy link
Copy Markdown
Collaborator Author

PR_Github #55184 Bot args parsing error: usage: /bot [-h]
{run,kill,skip,submit,reviewers,reuse-pipeline,reuse-review} ...
/bot: error: unrecognized arguments: --comment Only waive tests

Link to invocation

@xinhe-nv

Copy link
Copy Markdown
Collaborator

/bot skip --comment "Only waive tests"

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator Author

PR_Github #55186 [ skip ] triggered by Bot. Commit: ca3090e Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator Author

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

Link to invocation

@xinhe-nv xinhe-nv merged commit 2275d1e into NVIDIA:main Jun 23, 2026
7 checks passed
@xinhe-nv xinhe-nv deleted the trtllm-ci-report/waive-20260619-161606 branch June 23, 2026 06:37
xinhe-nv added a commit to tensorrt-cicd/TensorRT-LLM that referenced this pull request Jun 24, 2026
Signed-off-by: tensorrt-cicd <90828364+tensorrt-cicd@users.noreply.github.com>
Signed-off-by: GitLab CI Bot <gitlab-ci@nvidia.com>
Co-authored-by: xinhe-nv <200704525+xinhe-nv@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