Skip to content

[None][test] Waive 2 failed cases for main in QA CI - #15579

Merged
xinhe-nv merged 3 commits into
NVIDIA:mainfrom
tensorrt-cicd:trtllm-ci-report/waive-20260624-074641
Jun 24, 2026
Merged

[None][test] Waive 2 failed cases for main in QA CI#15579
xinhe-nv merged 3 commits into
NVIDIA:mainfrom
tensorrt-cicd:trtllm-ci-report/waive-20260624-074641

Conversation

@tensorrt-cicd

@tensorrt-cicd tensorrt-cicd commented Jun 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_CLUSTER_TEST/1567/
Bug(s): 6344883, 6369254

Waive entries added

full:GB200/accuracy/test_llm_api_pytorch.py::TestQwen3_5_4B::test_dflash SKIP (https://nvbugs/6344883)
full:GB200/disaggregated/test_ad_disagg.py::test_async_eagle3_full_model_handoff SKIP (https://nvbugs/6369254)

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

Summary by CodeRabbit

  • Tests
    • Updated integration test waivers for GB200-related cases to account for two additional scenarios.
    • Preserved the existing nearby waiver entry while adjusting the list order.

@xinhe-nv
xinhe-nv marked this pull request as ready for review June 24, 2026 07:52
@xinhe-nv
xinhe-nv force-pushed the trtllm-ci-report/waive-20260624-074641 branch from f1eca64 to 277c757 Compare June 24, 2026 07:54
@xinhe-nv
xinhe-nv requested review from a team as code owners June 24, 2026 07:54
@xinhe-nv
xinhe-nv requested a review from a team June 24, 2026 07:54
@xinhe-nv
xinhe-nv requested review from a team as code owners June 24, 2026 07:54
Signed-off-by: GitLab CI Bot <gitlab-ci@nvidia.com>
Signed-off-by: GitLab CI Bot <gitlab-ci@nvidia.com>
Bug(s): 6344883, 6369254
Requested by: qa@nvidia.com
Jenkins build: https://prod.blsm.nvidia.com/swqa-tensorrt-qa-test/job/LLM_FUNCTION_CLUSTER_TEST/1567/

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-20260624-074641 branch from 277c757 to 3e443b4 Compare June 24, 2026 07:55
@xinhe-nv
xinhe-nv enabled auto-merge (squash) June 24, 2026 07:55
@xinhe-nv

Copy link
Copy Markdown
Collaborator

/bot skip --comment "only waive test"

@coderabbitai

coderabbitai Bot commented Jun 24, 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: 963a393f-6eea-4ff0-bd76-fd68e3712a9d

📥 Commits

Reviewing files that changed from the base of the PR and between 65d1bfb and 3e443b4.

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

📝 Walkthrough

Walkthrough

Two SKIP waiver entries are added to tests/integration/test_lists/waives.txt for the GB200 platform: one for full:GB200/accuracy/test_llm_api_pytorch.py::TestQwen3_5_4B::test_dflash and one for full:GB200/disaggregated/test_ad_disagg.py::test_async_eagle3_full_model_handoff.

Changes

GB200 Waiver Additions

Layer / File(s) Summary
GB200 SKIP waivers for test_dflash and test_async_eagle3_full_model_handoff
tests/integration/test_lists/waives.txt
Two new SKIP entries inserted for GB200: TestQwen3_5_4B::test_dflash in the accuracy LLM API PyTorch suite, and test_async_eagle3_full_model_handoff in the disaggregated suite.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • NVIDIA/TensorRT-LLM#15506: Adds/updates SKIP waiver entries for Qwen test_dflash cases in the same accuracy/test_llm_api_pytorch.py path within waives.txt, directly overlapping with the TestQwen3_5_4B::test_dflash waiver added here.
  • NVIDIA/TensorRT-LLM#15505: Adds SKIP/waiver entries for GB200/B200 CI test failures in the same waives.txt file, using the same mechanism.
  • NVIDIA/TensorRT-LLM#15499: Modifies tests/integration/test_lists/waives.txt by adding new SKIP entries for disaggregated test cases, directly related to the same waiver-file mechanism.

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 is concise, correctly formatted, and summarizes the waive update.
Description check ✅ Passed The description explains the waive request and bugs, but it omits the template's Test Coverage and PR Checklist sections.
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.

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator Author

PR_Github #55452 [ skip ] triggered by Bot. Commit: 3e443b4 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator Author

PR_Github #55452 [ skip ] completed with state SUCCESS. Commit: 3e443b4
Skipping testing for commit 3e443b4

Link to invocation

@xinhe-nv
xinhe-nv merged commit d3c0d83 into NVIDIA:main Jun 24, 2026
11 checks passed
@xinhe-nv
xinhe-nv deleted the trtllm-ci-report/waive-20260624-074641 branch June 24, 2026 08:12
BrianLi23 pushed a commit to BrianLi23/TensorRT-LLM that referenced this pull request Jul 9, 2026
Signed-off-by: GitLab CI Bot <gitlab-ci@nvidia.com>
Signed-off-by: tensorrt-cicd <90828364+tensorrt-cicd@users.noreply.github.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.

2 participants